fix(util): force check id when both have id (SlimefunGuguProject#912)

mcchampions · Jul 5, 2024 · fe1a0c7 · fe1a0c7
1 parent 4390b5e
commit fe1a0c7
Showing 1 changed file with 15 additions and 5 deletions.
diff --git a/src/main/java/io/github/thebusybiscuit/slimefun4/utils/SlimefunUtils.java b/src/main/java/io/github/thebusybiscuit/slimefun4/utils/SlimefunUtils.java
@@ -361,9 +361,7 @@ public static boolean isItemSimilar(
                         .getItemData(possibleSfItemMeta)
                         .orElse(null);
                 // Prioritize SlimefunItem id comparison over ItemMeta comparison
-                if (id != null && id.equals(possibleItemId)) {
-                    Debug.log(TestCase.CARGO_INPUT_TESTING, "  Item IDs matched!");
-
+                if (id != null && possibleItemId != null) {
                     /*
                      * PR #3417
                      *
@@ -374,14 +372,20 @@ public static boolean isItemSimilar(
                     if (optionalDistinctive.isPresent()) {
                         return optionalDistinctive.get().canStack(possibleSfItemMeta, itemMeta);
                     }
-                    return true;
+
+                    var match = id.equals(possibleItemId);
+
+                    Debug.log(TestCase.CARGO_INPUT_TESTING, "  Use Item ID match: {}", match);
+
+                    return match;
                 } else {
                     Debug.log(
                             TestCase.CARGO_INPUT_TESTING,
-                            "  Item IDs don't match, checking meta {} == {} (lore: {})",
+                            "  one of item have no Slimefun ID, checking meta {} == {} (lore: {})",
                             itemMeta,
                             possibleSfItemMeta,
                             checkLore);
+
                     return equalsItemMeta(itemMeta, possibleSfItemMeta, checkLore, checkCustomModelData);
                 }
             } else if (sfitem.hasItemMeta()) {
@@ -462,20 +466,24 @@ private static boolean equalsItemMeta(
             boolean checkLore,
             boolean checkCustomModelCheck) {
         if (itemMeta.hasDisplayName() != sfitemMeta.hasDisplayName()) {
+            Debug.log(TestCase.CARGO_INPUT_TESTING, "  Comparing has display name failed");
             return false;
         } else if (itemMeta.hasDisplayName()
                 && sfitemMeta.hasDisplayName()
                 && !itemMeta.getDisplayName().equals(sfitemMeta.getDisplayName())) {
+            Debug.log(TestCase.CARGO_INPUT_TESTING, "  Comparing display name failed");
             return false;
         } else if (checkLore) {
             boolean hasItemMetaLore = itemMeta.hasLore();
             boolean hasSfItemMetaLore = sfitemMeta.hasLore();
 
             if (hasItemMetaLore && hasSfItemMetaLore) {
                 if (!equalsLore(itemMeta.getLore(), sfitemMeta.getLore())) {
+                    Debug.log(TestCase.CARGO_INPUT_TESTING, "  Comparing lore failed");
                     return false;
                 }
             } else if (hasItemMetaLore != hasSfItemMetaLore) {
+                Debug.log(TestCase.CARGO_INPUT_TESTING, "  Comparing has lore failed");
                 return false;
             }
         }
@@ -497,6 +505,8 @@ private static boolean equalsItemMeta(
             return ((PotionMeta) itemMeta).getBasePotionType().equals(((PotionMeta) sfitemMeta).getBasePotionType());
         }
 
+        Debug.log(TestCase.CARGO_INPUT_TESTING, "  All meta checked.");
+
         return true;
     }