mirrors/PocketMine-MP
1
0
Fork 0
mirror of https://github.com/pmmp/PocketMine-MP.git synced 2025-04-22 16:51:42 +00:00
Code Issues Projects Releases Wiki Activity

BaseInventory: fixed passing NULL slots to InventoryListener->onContentChange()

Browse Source 
as per the documentation, InventoryListener->onContentChange() does not expect to receive NULL in the given array.
This commit is contained in:
Dylan K. Taylor 2020-12-05 01:11:30 +00:00
parent 070d8efda3
commit eca0e88471
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/inventory/BaseInventory.php
View File

@ -27,6 +27,7 @@ use Ds\Set;
use pocketmine\item\Item;
use pocketmine\item\ItemFactory;
use pocketmine\player\Player;
use function array_map;
use function array_slice;
use function count;
use function max;
@ -95,7 +96,9 @@ abstract class BaseInventory implements Inventory{
$items = array_slice($items, 0, $this->getSize(), true);
}
$oldContents = $this->slots->toArray();
$oldContents = array_map(function(?Item $item) : Item{
return $item ?? ItemFactory::air();
}, $this->slots->toArray());
$listeners = $this->listeners->toArray();
$this->listeners->clear();