Added double chest fix #2744

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@prankard

Added a fix for the double chest Issue #2493

The data wasn't always going into the doublechest, and didn't appear to be combining correctly upon logout/login of server.

Double Chest Changes:

The doublechest, now starts with the items created from left/right in it's constructor.

Then in it's setContent's function. It now doesn't call parent (as the slot[] check is invalid and won't check both chests) but now has a similar pattern using the slots[] of left and right.

Minor improvement to base inventory:

When you set contents to an inventory, I now clear if the 'SetItem' fails.

@prankard prankard Added double chest fix
78a3ce4
@shoghicp
Member
shoghicp commented Mar 9, 2015

Could you follow the Contribution Guidelines? Specifically the code formatting, so it follows the project's formatting

@shoghicp shoghicp added a commit that closed this pull request Mar 17, 2015
@shoghicp shoghicp Implemented double chest fix, closes #2744, fixes #2493 37bc127
@shoghicp shoghicp closed this in 37bc127 Mar 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment