Fix subsequent calls to InventoryContents#set() not working

MinusKube / SmartInvs

Advanced Inventory API for your Minecraft Bukkit plugins.

https://minuskube.gitbook.io/smartinvs/

Apache License 2.0

263 stars 65 forks source link

Fix subsequent calls to InventoryContents#set() not working #186

Closed 2008Choco closed 3 years ago

2008Choco commented 3 years ago

smartInventory.getManager().getOpenedPlayers(smartInventory) returns a List<Player>, against which InventoryContents#update() calls contains() with a UUID, causing any subsequent calls to InventoryContents#set() to not update in the inventory.

Caused by 7382db3da7988a4d2fa790fbf59f4f9332457620

Relevant bug: #179

This PR aims to resolve this issue.

MinusKube commented 3 years ago

Great catch, thanks!