satyrnidae / wolf-armor-and-storage-legacy

Minecraft mod that improves vanilla wolves, no custom entities required.
https://minecraft.curseforge.com/projects/wolf-armor-and-storage
MIT License
12 stars 8 forks source link

[1.12] [Duplicate] Strange/Unexpected Behavior When Removing A Chest From a Wolf #166

Closed PiggiesGoSqueal closed 4 years ago

PiggiesGoSqueal commented 4 years ago

Issue Description Storing items in the wolf's chest then removing the chest will not drop the items that were stored in the chest. It saves that information for when the chest is put back on the wolf. I mean, it doesn't delete the items so that is a plus but that is very weird behavior and not how it should logically behave. Players would likely think it deleted their items (as I did at first) and if they have a lot of dogs it would be very easy to mix up which to put the chest back on - even if they knew about this feature. Furthermore, it basically acts as a way to 'hide' items from other people (including staff of a server) which is problematic.

Reproduction Steps Steps to reproduce the behavior:

  1. Tame a wolf.
  2. Place a chest on the wolf.
  3. Store items in the chest.
  4. Open the wolf's inventory and click the "Remove Chest" button.
  5. Watch as it only drops the chest and not the items.
  6. Open the wolf's inventory to confirm that there are no items stored there (because it has no chest on it).
  7. Put the same chest back on the wolf and open the wolf's inventory.
  8. Stare in confusion when the items are magically back in the storage of the wolf. :P

Expected Behavior When the player removes the chest from the wolf I'd expect it to drop the items that were being stored in the chest.

Screenshots / Videos GIF: https://gyazo.com/c5d96c66d2e1b4142d9765d9e33d846b

Version Information:

Additional Context

Hope this gets fixed! Awesome mod other than this issue! :)

satyrnidae commented 4 years ago

Fixed this in #160, but I guess i haven't released that version yet, oops

PiggiesGoSqueal commented 4 years ago

Fixed this in #160, but I guess i haven't released that version yet, oops

Ah okay. Thanks!