Fixed NSM on CRAFTING TABLE usage at mc 1.8.8

Author: Tigerpanzer02

.github/CHANGELOG.md

@@ -1,9 +1,12 @@
-### 1.4.3 Release (07.2025)
+### 1.4.3 Release (10.07.2025)
 * Added 1.21.6-7 support!
 * Updated XSeries to 13.3.3
 * Updated FastBoard to 2.1.5
 * Updated FastInv to 3.1.2
 * Fixed NPE on ServerListPing if plugin isn't fully started / arenaregistry not ready
+* Fixed NSM on CRAFTING TABLE usage at mc 1.8.8
+* Internal Code Cleanup such as prefixes and MiscUtils
+* Changed Hex code support to only support 6 digit long hex code!
 
 ### 1.4.2 Release (09.06.2025)
 * Added BLOCK_IN_GAME_INTERACTIONS option to block interactions whileas ingame

MiniGamesBox Classic/src/main/java/plugily/projects/minigamesbox/classic/events/Events.java

@@ -44,6 +44,7 @@
 import plugily.projects.minigamesbox.classic.PluginMain;
 import plugily.projects.minigamesbox.classic.handlers.language.MessageBuilder;
 import plugily.projects.minigamesbox.classic.utils.version.ServerVersion;
+import plugily.projects.minigamesbox.classic.utils.version.VersionUtils;
 import plugily.projects.minigamesbox.classic.utils.version.events.api.PlugilyPlayerInteractEvent;
 import plugily.projects.minigamesbox.classic.utils.version.events.api.PlugilyPlayerSwapHandItemsEvent;
 
@@ -189,7 +190,7 @@ public void onCraft(PlugilyPlayerInteractEvent event) {
       return;
     }
     if(event.getPlayer().getTargetBlock(null, 7).getType() == XMaterial.CRAFTING_TABLE.parseMaterial()) {
-      if(event.getAction().isRightClick()) {
+      if(VersionUtils.isRightClick(event.getAction())) {
         event.setCancelled(true);
       }
     }

MiniGamesBox Classic/src/main/java/plugily/projects/minigamesbox/classic/utils/version/VersionUtils.java

@@ -36,6 +36,7 @@
 import org.bukkit.entity.HumanEntity;
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
+import org.bukkit.event.block.Action;
 import org.bukkit.event.entity.CreatureSpawnEvent;
 import org.bukkit.inventory.EquipmentSlot;
 import org.bukkit.inventory.ItemStack;
@@ -445,7 +446,6 @@ public static double getMaxHealth(LivingEntity entity) {
     }
 
 
-
     java.util.Optional<org.bukkit.attribute.AttributeInstance> at = MiscUtils.getEntityAttribute(entity, XAttribute.MAX_HEALTH.get());
     return at.map(AttributeInstance::getValue).orElse(20D);
   }
@@ -643,16 +643,22 @@ public static Entity spawnEntity(Location location, EntityType entityType) {
   }
 
   public static BannerMeta setBaseColor(BannerMeta meta, DyeColor color) {
-      BlockStateMeta bsm = (BlockStateMeta) meta;
-      BlockState state = ((BlockStateMeta) meta).getBlockState();
-      if (state instanceof Banner) {
-          ((Banner) state).setBaseColor(color);
-          state.update(true);
-      }
-      bsm.setBlockState(state);
+    BlockStateMeta bsm = (BlockStateMeta) meta;
+    BlockState state = ((BlockStateMeta) meta).getBlockState();
+    if(state instanceof Banner) {
+      ((Banner) state).setBaseColor(color);
+      state.update(true);
+    }
+    bsm.setBlockState(state);
     return meta;
+  }
 
+  public static boolean isLeftClick(Action action) {
+    return action == Action.LEFT_CLICK_AIR || action == Action.LEFT_CLICK_BLOCK;
+  }
 
-}
+  public static boolean isRightClick(Action action) {
+    return action == Action.RIGHT_CLICK_AIR || action == Action.RIGHT_CLICK_BLOCK;
+  }
 
 }