Fixed part of the issues, still splattermap and renaming from GUI to fix

pom.xml

@@ -39,7 +39,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>fr.moribus</groupId>
     <artifactId>ImageOnMap</artifactId>
-    <version>4.1.2</version>
+    <version>4.2.0</version>
     <packaging>jar</packaging>
 
     <properties>

src/main/java/fr/moribus/imageonmap/commands/maptool/NewCommand.java

@@ -121,7 +121,7 @@ public class NewCommand extends IoMCommand {
             }
             scaling = resizeMode();
         }
-        if (width == 0 || height == 0) {
+        if (width < 0 || height < 0) {
             throwInvalidArgument(I.t("You need to specify a valid size. e.g. resize 4 5"));
             return;
         }

src/main/java/fr/moribus/imageonmap/gui/MapDetailGui.java

@@ -46,6 +46,7 @@ import fr.zcraft.quartzlib.components.gui.Gui;
 import fr.zcraft.quartzlib.components.gui.GuiAction;
 import fr.zcraft.quartzlib.components.gui.PromptGui;
 import fr.zcraft.quartzlib.components.i18n.I;
+import fr.zcraft.quartzlib.tools.PluginLogger;
 import fr.zcraft.quartzlib.tools.items.ItemStackBuilder;
 import fr.zcraft.quartzlib.tools.runners.RunTask;
 import org.apache.commons.lang.ArrayUtils;
@@ -208,32 +209,35 @@ public class MapDetailGui extends ExplorerGui<Integer> {
             return;
         }
 
-        PromptGui.prompt(getPlayer(), newName -> {
+        try {
-            if (!Permissions.RENAME.grantedTo(getPlayer())) {
+            PromptGui.prompt(getPlayer(), newName -> {
-                I.sendT(getPlayer(), "{ce}You are no longer allowed to do that.");
+                if (!Permissions.RENAME.grantedTo(getPlayer())) {
-                return;
+                    I.sendT(getPlayer(), "{ce}You are no longer allowed to do that.");
-            }
+                    return;
+                }
 
-            if (newName == null || newName.isEmpty()) {
+                if (newName == null || newName.isEmpty()) {
-                I.sendT(getPlayer(), "{ce}Map names can't be empty.");
+                    I.sendT(getPlayer(), "{ce}Map names can't be empty.");
-                return;
+                    return;
-            }
+                }
-            if (newName.equals(map.getName())) {
+                if (newName.equals(map.getName())) {
-                return;
+                    return;
-            }
+                }
 
-            map.rename(newName);
+                map.rename(newName);
-            I.sendT(getPlayer(), "{cs}Map successfully renamed.");
+                I.sendT(getPlayer(), "{cs}Map successfully renamed.");
 
-            if (getParent() != null) {
+                if (getParent() != null) {
-                RunTask.later(() -> Gui.open(getPlayer(), this), 1L);
+                    RunTask.later(() -> Gui.open(getPlayer(), this), 1L);
-
-            } else {
-                close();
-            }
-        }, map.getName(), this);
 
+                } else {
+                    close();
+                }
+            }, map.getName(), this);
 
+        } catch (IllegalStateException e) {
+            PluginLogger.info("error while renaming");
+        }
     }
 
     @GuiAction("delete")

src/main/java/fr/moribus/imageonmap/ui/SplatterMapManager.java

@@ -137,6 +137,7 @@ public abstract class SplatterMapManager {
      * @return True if the attribute was detected.
      */
     public static boolean hasSplatterAttributes(ItemStack itemStack) {
+
         try {
             final NBTCompound nbt = NBT.fromItemStack(itemStack);
             if (!nbt.containsKey("Enchantments")) {

src/main/resources/plugin.yml

@@ -1,6 +1,6 @@
 name: ImageOnMap
 main: fr.moribus.imageonmap.ImageOnMap
-version: "4.1.2"
+version: "4.2.0"
 api-version: "1.13"