Merge branch 'Slimefun:master' into master

Slimefun · Jan 21, 2024 · 948ead0 · 948ead0
2 parents f423de8 + cd3672c
commit 948ead0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/main/java/io/github/thebusybiscuit/slimefun4/api/items/SlimefunItem.java b/src/main/java/io/github/thebusybiscuit/slimefun4/api/items/SlimefunItem.java
@@ -1199,7 +1199,7 @@ public final int hashCode() {
      *            The {@link ItemStack} to check
      * @return The {@link Optional} {@link SlimefunItem} associated with this {@link ItemStack} if present, otherwise empty
      */
-    public @Nonnull Optional<SlimefunItem> getOptionalByItem(@Nullable ItemStack item) {
+    public static @Nonnull Optional<SlimefunItem> getOptionalByItem(@Nullable ItemStack item) {
         return Optional.ofNullable(getByItem(item));
     }
 }
diff --git a/src/main/java/io/github/thebusybiscuit/slimefun4/utils/ArmorStandUtils.java b/src/main/java/io/github/thebusybiscuit/slimefun4/utils/ArmorStandUtils.java
@@ -21,6 +21,8 @@
  */
 public class ArmorStandUtils {
 
+    private ArmorStandUtils() {}
+
     /**
      * Spawns an {@link ArmorStand} at the given {@link Location} with the given custom name
      * <br>
@@ -51,15 +53,14 @@ public class ArmorStandUtils {
         // The consumer method was moved from World to RegionAccessor in 1.20.2
         // Due to this, we need to use a rubbish workaround to support 1.20.1 and below
         // This causes flicker on these versions which sucks but not sure a better way around this right now.
-        if (PaperLib.getMinecraftVersion() <= 20
-            && PaperLib.getMinecraftPatchVersion() < 2
-        ) {
+        if (PaperLib.getMinecraftVersion() < 20 ||
+                (PaperLib.getMinecraftVersion() == 20 && PaperLib.getMinecraftPatchVersion() < 2)) {
             ArmorStand armorStand = location.getWorld().spawn(location, ArmorStand.class);
             setupArmorStand(armorStand);
             return armorStand;
         }
 
-        return location.getWorld().spawn(location, ArmorStand.class, armorStand -> setupArmorStand(armorStand));
+        return location.getWorld().spawn(location, ArmorStand.class, ArmorStandUtils::setupArmorStand);
     }
 
     private static void setupArmorStand(ArmorStand armorStand) {