EncryptSL / LiteEco

Minecraft Economy Plugin works with vault.
26 stars 9 forks source link

/money top command does not work properly #9

Closed DJIronic closed 1 year ago

DJIronic commented 1 year ago

Command /money top is not reporting the top players in the game.

This is how the comand result looks: image

But when I look in the DB, there are players with 100 000+ in the currency, so they should be up on the top: image

We also tried to remove all the money of players and set only few of us a balance to see, if the plugin will work it out with smaller numbers, but that not helped. As you can see, names in the baltop are repeating, but with zero balance.

image

and here is the DB with two of us having money: image

Also placeholder for the top player, top balance are not working, it will show the wrong numbers, the same listed in money top command.

We would love to see this fixed.

PS: Můžeme komunikovat i česky :)

EncryptSL commented 1 year ago

Po pravdě, chyba by někde mohla být neříkám že ne. Testováním chyb jsem ale neodhalil příčinu co popisuješ.

Testování jsem provedl s čistou tabulkou a databází v SQL modu.

S výpisem okolo 20 účtů, při čemž jsem každému z nich přidělil random hodnoty. Výsledek řazení ale ukázal že by mělo fungovat, image

Pokusím se tohle chování ještě otestovat přímo ve hře, případně ho nějak opravit.

EncryptSL commented 1 year ago

Doplňující otázka používáš doufám verzi 1.1.4-SNAPSHOT.

Řešení by mohlo být v posledním příspěvku.

EncryptSL commented 1 year ago

Podařilo se mi problém replikovat až jsem nahodil tuhle verzi - 1.1.4-SNAPSHOT Při téhle vývojové verzi se mi tohle nestalo, https://drive.google.com/file/d/1BC-ad5o9jjtU_UTDpMHaUO2KEI0Jc9CL/view

Můžeš tohle potvrdit. Případně dej vědět.

DJIronic commented 1 year ago

Nahodím a vyzkouším

EncryptSL commented 1 year ago

Opraveno zmíněno v #10