1.16 Inventory Packet Fixes

[AJ-Ferguson]

WIP PR to finish implementing inventory related packets for 1.16.

To-do:

• [x] Fix SwapStackRequestActionData. Wrong fields.
• [x] Fix ItemStackResponseSerializer. Packet data ends if success is false.
• [x] Fix InventoryContentPacket. The current implementation is wrong and shouldn't be using a map. #55
• [x] Implement toString, equals, and hashCode in the StackRequestActionData classes.
• [x] Map out container ids used in ItemStackResponsePacket and ItemStackRequestPacket
• [x] Refactor CreativeContentPacket to use the net id stored in ItemData instead of a map key.

[Heath123]

Implement toString, equals, and hashCode in the StackRequestActionData classes.

Lombok can do that for you: https://projectlombok.org/features/ToString https://projectlombok.org/features/EqualsAndHashCode

[AJ-Ferguson]

Ready for review.

Notes:

• All crafting requests seem to use Container.CREATIVE_OUTPUT as the source when transferring output into the inventory. Maybe it should be renamed?
• There might be more StackRequestActionDatas where it would make sense to implement a common interface.
• Not sure if Container was the best name for the new enum as there is already a ContainerId class. Maybe NetContainer