mirror of
https://github.com/pmmp/PocketMine-MP.git
synced 2025-06-21 19:04:15 +00:00
c70b80c273
Given the various limitations and flexibilities posed by EntityItemPickupEvent, I settled on this as the simplest way to deal with the problem. - EntityItemPickupEvent may have its destination inventory changed, so we can't cache the result of getAddableItemQuantity() to use after the event. - The item itself may have changed, so even if we thought we could add some items before the change, we might not be able to afterwards. Considering the above facts, it's better to just give the whole itemstack to EntityItemPickupEvent, and let plugins use getAddableItemQuantity() on their own to decide if their chosen inventory can accommodate the item or not. If it can't, then we'll just drop it on the ground. This also fixes a potential issue where plugins changing the item to a custom one might end up with their items and the actual items both just vanishing if the target inventory was full. closes #4499