Fix shulker boxes unstacking items upon an explosion (Fixes #85)

Author: Mikedeejay2

src/main/java/com/mikedeejay2/simplestack/Simplestack.java

@@ -40,7 +40,7 @@ public void onEnable()
     {
         super.onEnable();
 
-        setPrefix("&b[&9" + this.getDescription().getName() + "&b]&r");
+        setPrefix("&b[&9" + this.getDescription().getName() + "&b]&r ");
 
         this.commandManager = new CommandManager(this, "simplestack");
         commandManager.setDefaultSubCommand("help");
@@ -77,6 +77,8 @@ public void onEnable()
             registerEvent(new InventoryPickupItemListener(this));
             registerEvent(new PlayerItemConsumeListener(this));
             registerEvent(new ItemSpawnListener(this));
+            registerEvent(new BlockExplodeListener(this));
+            registerEvent(new EntityExplodeListener(this));
             if(MinecraftVersion.getVersionShort() >= 16)
             {
                 registerEvent(new PrepareSmithingListener(this));

src/main/java/com/mikedeejay2/simplestack/listeners/BlockExplodeListener.java

@@ -0,0 +1,45 @@
+package com.mikedeejay2.simplestack.listeners;
+
+import com.mikedeejay2.simplestack.Simplestack;
+import com.mikedeejay2.simplestack.util.CancelUtils;
+import com.mikedeejay2.simplestack.util.MoveUtils;
+import com.mikedeejay2.simplestack.util.ShulkerBoxes;
+import org.bukkit.block.Block;
+import org.bukkit.entity.Player;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.EventPriority;
+import org.bukkit.event.Listener;
+import org.bukkit.event.block.BlockBreakEvent;
+import org.bukkit.event.block.BlockExplodeEvent;
+
+/**
+ * Listens for block explode events
+ *
+ * @author Mikedeejay2
+ */
+public class BlockExplodeListener implements Listener
+{
+    private final Simplestack plugin;
+
+    public BlockExplodeListener(Simplestack plugin)
+    {
+        this.plugin = plugin;
+    }
+
+    /**
+     * BlockExplodeEvent
+     * This is needed for when a shulker box breaks, because by default Minecraft
+     * unstacks items inside of a shulker box automatically
+     *
+     * @param event The event being activated
+     */
+    @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true)
+    public void blockExplodeEvent(BlockExplodeEvent event)
+    {
+        for(Block block : event.blockList())
+        {
+            if(!ShulkerBoxes.isShulkerBox(block.getType())) continue;
+            MoveUtils.preserveShulkerBox(block);
+        }
+    }
+}

src/main/java/com/mikedeejay2/simplestack/listeners/EntityExplodeListener.java

@@ -0,0 +1,43 @@
+package com.mikedeejay2.simplestack.listeners;
+
+import com.mikedeejay2.simplestack.Simplestack;
+import com.mikedeejay2.simplestack.util.MoveUtils;
+import com.mikedeejay2.simplestack.util.ShulkerBoxes;
+import org.bukkit.block.Block;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.EventPriority;
+import org.bukkit.event.Listener;
+import org.bukkit.event.block.BlockExplodeEvent;
+import org.bukkit.event.entity.EntityExplodeEvent;
+
+/**
+ * Listens for block explode events
+ *
+ * @author Mikedeejay2
+ */
+public class EntityExplodeListener implements Listener
+{
+    private final Simplestack plugin;
+
+    public EntityExplodeListener(Simplestack plugin)
+    {
+        this.plugin = plugin;
+    }
+
+    /**
+     * BlockExplodeEvent
+     * This is needed for when a shulker box breaks, because by default Minecraft
+     * unstacks items inside of a shulker box automatically
+     *
+     * @param event The event being activated
+     */
+    @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true)
+    public void blockExplodeEvent(EntityExplodeEvent event)
+    {
+        for(Block block : event.blockList())
+        {
+            if(!ShulkerBoxes.isShulkerBox(block.getType())) continue;
+            MoveUtils.preserveShulkerBox(block);
+        }
+    }
+}

src/main/java/com/mikedeejay2/simplestack/listeners/player/BlockBreakListener.java

@@ -41,6 +41,7 @@ public void blockBreakEvent(BlockBreakEvent event)
         if(!ShulkerBoxes.isShulkerBox(block.getType())) return;
         if(CancelUtils.cancelPlayerCheck(plugin, player)) return;
 
-        MoveUtils.preserveShulkerBox(event, block);
+        MoveUtils.preserveShulkerBox(block);
+        event.setCancelled(true);
     }
 }

src/main/java/com/mikedeejay2/simplestack/util/MoveUtils.java

@@ -13,6 +13,7 @@
 import org.bukkit.entity.Player;
 import org.bukkit.event.Cancellable;
 import org.bukkit.event.block.BlockBreakEvent;
+import org.bukkit.event.block.BlockExplodeEvent;
 import org.bukkit.event.inventory.InventoryDragEvent;
 import org.bukkit.inventory.*;
 import org.bukkit.inventory.meta.BlockStateMeta;
@@ -366,7 +367,7 @@ public static boolean combineItemInternal(Simplestack plugin, ItemStack itemInSl
      * @param event Event that this method is being run in
      * @param block The block to check
      */
-    public static void preserveShulkerBox(BlockBreakEvent event, Block block)
+    public static void preserveShulkerBox(Block block)
     {
         Location location = block.getLocation();
         World world = location.getWorld();
@@ -385,7 +386,6 @@ public static void preserveShulkerBox(BlockBreakEvent event, Block block)
 
         world.dropItemNaturally(location, item);
         block.setType(Material.AIR);
-        event.setCancelled(true);
     }