Lukario45 / MCThunder

Open Source Minecraft Server Software
GNU General Public License v3.0
6 stars 2 forks source link

Keeping track of Inventory #9

Open pupnewfster opened 9 years ago

pupnewfster commented 9 years ago

The Inventory still needs some work when using hotkeys to assign items to a new slot to actually update where in the Inventory they are serverside.

Lukario45 commented 9 years ago

Also stakable items need to be fixed, You /give yourself someone once to get 64, if you do it again it shows -128 in your bar

Evadable commented 9 years ago

I have a fix for this

pupnewfster commented 9 years ago

https://github.com/Lukario45/MCThunder/blob/master/src/main/java/net/mcthunder/inventory/ItemStack.java line 529 is the function that can be modified to fix it It just needs to return false if it is over max stack size reason I didn't do it originally is I have not added in for all the materials max stack size aka we could at least set it for 64 I guess for now but for swords they dont stack about one so I originally left it as is. EDIT: Also when 63+2 or something to have it split will need to be done as well at some point but is not as large an issue as it will return false if it adds to over max size anyways