Inventory: remove redundant return values

src/pocketmine/inventory/ArmorInventory.php

@@ -60,19 +60,19 @@ class ArmorInventory extends BaseInventory{
 		return $this->getItem(self::SLOT_FEET);
 	}
 
-	public function setHelmet(Item $helmet) : bool{
+	public function setHelmet(Item $helmet) : void{
-		return $this->setItem(self::SLOT_HEAD, $helmet);
+		$this->setItem(self::SLOT_HEAD, $helmet);
 	}
 
-	public function setChestplate(Item $chestplate) : bool{
+	public function setChestplate(Item $chestplate) : void{
-		return $this->setItem(self::SLOT_CHEST, $chestplate);
+		$this->setItem(self::SLOT_CHEST, $chestplate);
 	}
 
-	public function setLeggings(Item $leggings) : bool{
+	public function setLeggings(Item $leggings) : void{
-		return $this->setItem(self::SLOT_LEGS, $leggings);
+		$this->setItem(self::SLOT_LEGS, $leggings);
 	}
 
-	public function setBoots(Item $boots) : bool{
+	public function setBoots(Item $boots) : void{
-		return $this->setItem(self::SLOT_FEET, $boots);
+		$this->setItem(self::SLOT_FEET, $boots);
 	}
 }

src/pocketmine/inventory/BaseInventory.php

@@ -117,9 +117,7 @@ abstract class BaseInventory implements Inventory{
 					$this->clear($i, false);
 				}
 			}else{
-				if(!$this->setItem($i, $items[$i], false)){
+				$this->setItem($i, $items[$i], false);
-					$this->clear($i, false);
-				}
 			}
 		}
 
@@ -134,7 +132,7 @@ abstract class BaseInventory implements Inventory{
 		}
 	}
 
-	public function setItem(int $index, Item $item, bool $send = true) : bool{
+	public function setItem(int $index, Item $item, bool $send = true) : void{
 		if($item->isNull()){
 			$item = ItemFactory::air();
 		}else{
@@ -145,8 +143,6 @@ abstract class BaseInventory implements Inventory{
 
 		$this->slots[$index] = $item->isNull() ? null : $item;
 		$this->onSlotChange($index, $oldItem, $send);
-
-		return true;
 	}
 
 	public function contains(Item $item) : bool{
@@ -330,8 +326,8 @@ abstract class BaseInventory implements Inventory{
 		return $itemSlots;
 	}
 
-	public function clear(int $index, bool $send = true) : bool{
+	public function clear(int $index, bool $send = true) : void{
-		return $this->setItem($index, ItemFactory::air(), $send);
+		$this->setItem($index, ItemFactory::air(), $send);
 	}
 
 	public function clearAll(bool $send = true) : void{

src/pocketmine/inventory/CraftingGrid.php

@@ -61,14 +61,9 @@ class CraftingGrid extends BaseInventory{
 		throw new \BadMethodCallException("Cannot change the size of a crafting grid");
 	}
 
-	public function setItem(int $index, Item $item, bool $send = true) : bool{
+	public function setItem(int $index, Item $item, bool $send = true) : void{
-		if(parent::setItem($index, $item, $send)){
+		parent::setItem($index, $item, $send);
 		$this->seekRecipeBounds();
-
-			return true;
-		}
-
-		return false;
 	}
 
 	/**

src/pocketmine/inventory/DoubleChestInventory.php

@@ -58,13 +58,10 @@ class DoubleChestInventory extends ChestInventory implements InventoryHolder{
 		return $index < $this->left->getSize() ? $this->left->getItem($index) : $this->right->getItem($index - $this->left->getSize());
 	}
 
-	public function setItem(int $index, Item $item, bool $send = true) : bool{
+	public function setItem(int $index, Item $item, bool $send = true) : void{
 		$old = $this->getItem($index);
-		if($index < $this->left->getSize() ? $this->left->setItem($index, $item, $send) : $this->right->setItem($index - $this->left->getSize(), $item, $send)){
+		$index < $this->left->getSize() ? $this->left->setItem($index, $item, $send) : $this->right->setItem($index - $this->left->getSize(), $item, $send);
 		$this->onSlotChange($index, $old, $send);
-			return true;
-		}
-		return false;
 	}
 
 	public function getContents(bool $includeEmpty = false) : array{

src/pocketmine/inventory/FurnaceInventory.php

@@ -70,28 +70,22 @@ class FurnaceInventory extends ContainerInventory{
 
 	/**
 	 * @param Item $item
-	 *
-	 * @return bool
 	 */
-	public function setResult(Item $item) : bool{
+	public function setResult(Item $item) : void{
-		return $this->setItem(2, $item);
+		$this->setItem(2, $item);
 	}
 
 	/**
 	 * @param Item $item
-	 *
-	 * @return bool
 	 */
-	public function setFuel(Item $item) : bool{
+	public function setFuel(Item $item) : void{
-		return $this->setItem(1, $item);
+		$this->setItem(1, $item);
 	}
 
 	/**
 	 * @param Item $item
-	 *
-	 * @return bool
 	 */
-	public function setSmelting(Item $item) : bool{
+	public function setSmelting(Item $item) : void{
-		return $this->setItem(0, $item);
+		$this->setItem(0, $item);
 	}
 }

src/pocketmine/inventory/Inventory.php

@@ -56,15 +56,12 @@ interface Inventory{
 
 	/**
 	 * Puts an Item in a slot.
-	 * If a plugin refuses the update or $index is invalid, it'll return false
 	 *
 	 * @param int  $index
 	 * @param Item $item
 	 * @param bool $send
-	 *
-	 * @return bool
 	 */
-	public function setItem(int $index, Item $item, bool $send = true) : bool;
+	public function setItem(int $index, Item $item, bool $send = true) : void;
 
 	/**
 	 * Stores the given Items in the inventory. This will try to fill
@@ -182,10 +179,8 @@ interface Inventory{
 	 *
 	 * @param int  $index
 	 * @param bool $send
-	 *
-	 * @return bool
 	 */
-	public function clear(int $index, bool $send = true) : bool;
+	public function clear(int $index, bool $send = true) : void;
 
 	/**
 	 * Clears all the slots

src/pocketmine/inventory/PlayerInventory.php

@@ -119,16 +119,10 @@ class PlayerInventory extends BaseInventory{
 	 * Sets the item in the currently-held slot to the specified item.
 	 *
 	 * @param Item $item
-	 *
-	 * @return bool
 	 */
-	public function setItemInHand(Item $item) : bool{
+	public function setItemInHand(Item $item) : void{
-		if($this->setItem($this->getHeldItemIndex(), $item)){
+		$this->setItem($this->getHeldItemIndex(), $item);
 		$this->sendHeldItem($this->holder->getViewers());
-			return true;
-		}
-
-		return false;
 	}
 
 	/**