8.12.2 release

CHANGELOG.md

@@ -1,4 +1,7 @@
 # Changelog
+## 8.12.2
+- Fixes NullPointerException when using EssentialsX' /ec command in Minecraft 1.12
+
 ## 8.12.0
 - Changed name of command /chestsort to /sort. You can still use /chestsort though.
 - Fixed weird config updater problem on systems that don't properly support UTF-8 (like Windows)

pom.xml

@@ -9,7 +9,7 @@
 	<name>JeffChestSort</name>
 	<url>https://www.chestsort.de</url>
 	<description>Automatically sorts your chests!</description>
-	<version>8.12.1</version>
+	<version>8.12.2</version>
 	<packaging>jar</packaging>
 
 	<properties>

src/main/java/de/jeff_media/ChestSort/ChestSortOrganizer.java

@@ -18,6 +18,7 @@ import org.bukkit.inventory.meta.EnchantmentStorageMeta;
 import org.bukkit.inventory.meta.ItemMeta;
 import org.bukkit.inventory.meta.PotionMeta;
 import org.bukkit.potion.PotionData;
+import org.jetbrains.annotations.NotNull;
 
 import java.io.File;
 import java.io.FileNotFoundException;
@@ -399,7 +400,7 @@ public class ChestSortOrganizer {
     }
 
     // Sort an inventory only between startSlot and endSlot
-    void sortInventory(Inventory inv, int startSlot, int endSlot) {
+    void sortInventory(@NotNull Inventory inv, int startSlot, int endSlot) {
         if(inv==null) return;
         Class<? extends Inventory> invClass = inv.getClass();
         try {
@@ -414,7 +415,10 @@ public class ChestSortOrganizer {
                     return;
                 }
             }
-        } catch (NoSuchMethodException | SecurityException | IllegalStateException e) {
+        } catch (Throwable throwable) {
+            //System.out.println("Exception cought");
+            // I dont know, but this inv.getLocation() causes a NullPointerException when using EssentialsX's /ec command in 1.12
+            // Don't ask me why, but everything still works as expected if we catch that Exception.
             // TODO Auto-generated catch block
 
         }

src/main/resources/plugin.yml

@@ -1,6 +1,6 @@
 main: de.jeff_media.ChestSort.ChestSortPlugin
 name: ChestSort
-version: 8.12.1
+version: 8.12.2
 api-version: "1.13"
 description: Allows automatic chest sorting
 author: mfnalex