1.0.8

README.md

@@ -5,7 +5,7 @@ ItemCasePE
 
 * Summary: An implementation of Bukkit's ItemCase
 * Dependency Plugins: n/a
-* PocketMine-MP version: 1.4 - API 1.10.0
+* PocketMine-MP version: 1.4 - API 1.10.0, 1.6 - API 1.13.0
 * OptionalPlugins:
 * Categories: General
 * Plugin Access: Tile Entities, Items/Blocks
@@ -59,9 +59,16 @@ wave** mode is experimental and has not been fully tested.
 * itemcase.cmd: allow players access to the itemcase command
 * itemcase.destroy: allow players destroy cases
 
-Changes
--------
+## Changes
 
+* 1.0.8 : No AIR cases
+  - Do not allow to place cases with AIR only.  Reported by @Pub4Game.
+    Closes #30.
+* 1.0.7 : Update for PM1.6dev
+  - Checks which function to call (isPlaceable/canBePlaced) without having
+    to check version.
+* 1.0.6 : Update for PM1.6dev
+  - changed isPlaceable for canBePlaced
 * 1.0.5 : BugFix
   - Fixed a small bug related to new wave mode.
 * 1.0.4 : new wave vs classic
@@ -92,4 +99,3 @@ Copyright
 
     You should have received a copy of the GNU General Public License
     along with this program.  If not, see <http://www.gnu.org/licenses/>.
-

plugin.yml

@@ -1,10 +1,10 @@
 main: aliuly\itemcasepe\Main
-api: 1.10.0
+api: [1.10.0, 1.13.0]
 load: POSTWORLD
 
 name: ItemCasePE
 description: An implementation of Bukkit's ItemCase
-version: 1.0.5
+version: 1.0.8
 author: aliuly
 
 commands:

src/aliuly/itemcasepe/Main.php

@@ -302,6 +302,11 @@ public function onPlayerInteract(PlayerInteractEvent $ev){
 		}
 		$cid = implode(":",[$bl->getX(),$bl->getY(),$bl->getZ()]);
 		$item = $pl->getInventory()->getItemInHand();
+		if ($item->getId() === Item::AIR) {
+			$pl->sendMessage("You must be holding an item!");
+			$ev->setCancelled();
+			return;
+		}
 
 		if (!$this->addItemCase($bl->getLevel(),$cid,
 										implode(":",[$item->getId(),$item->getDamage()]),
@@ -312,8 +317,13 @@ public function onPlayerInteract(PlayerInteractEvent $ev){
 		}
 		unset($this->touches[$pl->getName()]);
 		$ev->setCancelled();
-		if ($ev->getItem()->isPlaceable())
-			$this->places[$pl->getName()] = $pl->getName();
+		if (is_callable([$ev->getItem(),"canBePlaced"])) {
+			if ($ev->getItem()->canBePlaced())
+				$this->places[$pl->getName()] = $pl->getName();
+		} elseif (is_callable([$ev->getItem(),"isPlaceable"])) {
+			if ($ev->getItem()->isPlaceable())
+				$this->places[$pl->getName()] = $pl->getName();
+		}
 	}
 	public function onBlockPlace(BlockPlaceEvent $ev){
 		$pl = $ev->getPlayer();