mirrors/PocketMine-MP
1
0
Fork 0
mirror of https://github.com/pmmp/PocketMine-MP.git synced 2025-09-05 09:26:06 +00:00
Code Issues Projects Releases Wiki Activity

InventoryManager: ensure the windowID is valid before attempting to remove any window

Browse Source 
this is currently a harmless bug, since remove() isn't currently doing any heavy lifting.
This commit is contained in:
Dylan K. Taylor
2022-09-28 01:01:42 +01:00
parent ed7c95549d
commit def2f8c145
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/network/mcpe/InventoryManager.php
View File

@ -259,7 +259,7 @@ class InventoryManager{
public function onClientRemoveWindow(int $id) : void{ public function onClientRemoveWindow(int $id) : void{
if($id === $this->lastInventoryNetworkId){ if($id === $this->lastInventoryNetworkId){
if($id !== $this->pendingCloseWindowId){ if(isset($this->windowMap[$id]) && $id !== $this->pendingCloseWindowId){
$this->remove($id); $this->remove($id);
$this->player->removeCurrentWindow(); $this->player->removeCurrentWindow();
} }