Player: Stop abusing getItemInHand() when we already have it stored

This commit is contained in:
Dylan K. Taylor 2017-12-10 17:08:56 +00:00
parent 2e62c084a4
commit 44e06f6a32

View File

@ -2485,8 +2485,8 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
}
return true;
}elseif($this->inventory->getItemInHand()->getId() === Item::BUCKET and $this->inventory->getItemInHand()->getDamage() === 1){ //Milk!
$this->server->getPluginManager()->callEvent($ev = new PlayerItemConsumeEvent($this, $this->inventory->getItemInHand()));
}elseif($slot->getId() === Item::BUCKET and $slot->getDamage() === 1){ //Milk!
$this->server->getPluginManager()->callEvent($ev = new PlayerItemConsumeEvent($this, $slot));
if($ev->isCancelled()){
$this->inventory->sendContents($this);
@ -2494,7 +2494,6 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
}
if($this->isSurvival()){
$slot = $this->inventory->getItemInHand();
--$slot->count;
$this->inventory->setItemInHand($slot);
$this->inventory->addItem(ItemFactory::get(Item::BUCKET, 0, 1));