From eee039a4904f9aeb811a0e3a0fe4252caff2941b Mon Sep 17 00:00:00 2001
From: "Dylan K. Taylor" <odigiman@gmail.com>
Date: Wed, 26 Jun 2019 15:32:57 +0100
Subject: [PATCH] Player: don't fire events on equipItem() when sending the
 same slot

---
 src/pocketmine/player/Player.php | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/pocketmine/player/Player.php b/src/pocketmine/player/Player.php
index f62428432..2553c43bf 100644
--- a/src/pocketmine/player/Player.php
+++ b/src/pocketmine/player/Player.php
@@ -1652,6 +1652,9 @@ class Player extends Human implements CommandSender, ChunkLoader, ChunkListener,
 		if(!$this->inventory->isHotbarSlot($hotbarSlot)){ //TODO: exception here?
 			return false;
 		}
+		if($hotbarSlot === $this->inventory->getHeldItemIndex()){
+			return true;
+		}
 
 		$ev = new PlayerItemHeldEvent($this, $this->inventory->getItem($hotbarSlot), $hotbarSlot);
 		$ev->call();