search

ksafin / DynamicEconomy

A Virtual Economy for your Minecraft Server
http://dev.bukkit.org/server-mods/dynamiceconomy-v-01/
12 stars 7 forks source link

/sell <item> all now leaves 1 in inventory, and still gives money #20

Closed jayyx2 closed 12 years ago

jayyx2 commented 12 years ago

I am using the latest version. v .8.03 for CB 1.2.3-R0.2 and this CraftBukkit version: http://dl.bukkit.org/downloads/craftbukkit/view/00923_1.2.3-R0.3/

this is affecting all items. Simply get one item in your inventory, then /sell all -- you are still given money and the item is not removed from your inventory. Edit: Also, if you have 64+ of the item in your inventory all but 1 is sold.

angelofdev commented 12 years ago

I'm also having this issue on version .8.03

ksafin commented 12 years ago

Fixed the issue, v .8.04 incoming. I had some odd statement that decremented the saleAmount when the user uses keyword "all".

I think there was a point in time that the bukkit build was unstable and the method I used for counting items in inventory always returned one too little or too much, leading me to add or subtract 1 from various item amounts.

Seems like it's all good now.