tickleman / RealShop

Shop using chests for Minecraft Game
http://mc.thgbm.net/wiki/En:RealShop
11 stars 3 forks source link

case sensitivity problems #1

Closed sunkid closed 13 years ago

sunkid commented 13 years ago

When I use '/mny set ' the player's account isn't correctly updated if the player has upper case letters in the user name. Instead, a new entry with all lower case is created in plugins/RealShop/accounts.cfg.

tickleman commented 13 years ago

I will study this in detail. But it's hard to test because I have no player account containing an uppercase letter. Could you give me some in a private message ? (I promise I will forget its password as soon as I've made a correction to my bug).

Otherwise I will work with it "in theory".

sunkid commented 13 years ago

My own username is lower case only as well, sorry.

May I suggest you simply use player.getName() everywhere and eliminate all 'toLowerCase()' conversions?

tickleman commented 13 years ago

You're right ! Job done for the next 0.41 release.

sunkid commented 13 years ago

great, thanks!