new ItemStack => ItemStack.of

Author: Intybyte

src/main/java/io/github/thebusybiscuit/slimefun4/api/events/PlayerRightClickEvent.java

@@ -92,13 +92,13 @@ public PlayerInteractEvent getInteractEvent() {
     /**
      * This method returns the {@link ItemStack} that was held in the hand of the {@link Player}.
      * It will never return null, should there be no {@link ItemStack} then it will return
-     * {@code new ItemStack(Material.AIR)}.
+     * {@code ItemStack.of(Material.AIR)}.
      * 
      * @return The {@link ItemStack} that the {@link Player} right clicked with
      */
     @Nonnull
     public ItemStack getItem() {
-        return itemStack.orElse(new ItemStack(Material.AIR));
+        return itemStack.orElse(ItemStack.of(Material.AIR));
     }
 
     /**

src/main/java/io/github/thebusybiscuit/slimefun4/api/items/SlimefunItemStack.java

@@ -78,7 +78,7 @@ public SlimefunItemStack(@Nonnull String id, @Nonnull ItemStack item, @Nonnull C
     }
 
     public SlimefunItemStack(@Nonnull String id, @Nonnull Material type, @Nonnull Consumer<ItemMeta> consumer) {
-        this(id, new ItemStack(type), consumer);
+        this(id, ItemStack.of(type), consumer);
     }
 
     public SlimefunItemStack(@Nonnull String id, @Nonnull Material type, @Nullable String name, @Nonnull Consumer<ItemMeta> consumer) {
@@ -109,7 +109,7 @@ public SlimefunItemStack(@Nonnull String id, @Nonnull ItemStack item, @Nullable
     }
 
     public SlimefunItemStack(@Nonnull String id, @Nonnull Material type, @Nullable String name, String... lore) {
-        this(id, new ItemStack(type), name, lore);
+        this(id, ItemStack.of(type), name, lore);
     }
 
     public SlimefunItemStack(@Nonnull String id, @Nonnull Material type, @Nonnull Color color, @Nullable String name, String... lore) {
@@ -284,7 +284,7 @@ public void lock() {
 
     private static @Nonnull ItemStack getSkull(@Nonnull String id, @Nonnull String texture) {
         if (Slimefun.getMinecraftVersion() == MinecraftVersion.UNIT_TEST) {
-            return new ItemStack(Material.PLAYER_HEAD);
+            return ItemStack.of(Material.PLAYER_HEAD);
         }
 
         PlayerSkin skin = PlayerSkin.fromBase64(getTexture(id, texture));

src/main/java/io/github/thebusybiscuit/slimefun4/api/recipes/RecipeType.java

@@ -113,7 +113,7 @@ public RecipeType(NamespacedKey key, ItemStack item) {
     }
 
     public RecipeType(MinecraftRecipe<?> recipe) {
-        this.item = new ItemStack(recipe.getMachine());
+        this.item = ItemStack.of(recipe.getMachine());
         this.machine = "";
         this.key = NamespacedKey.minecraft(recipe.getRecipeClass().getSimpleName().toLowerCase(Locale.ROOT).replace("recipe", ""));
     }

src/main/java/io/github/thebusybiscuit/slimefun4/core/guide/options/GuideModeOption.java

@@ -47,7 +47,7 @@ public Optional<ItemStack> getDisplayItem(Player p, ItemStack guide) {
 
         if (current.isPresent()) {
             SlimefunGuideMode selectedMode = current.get();
-            ItemStack item = new ItemStack(Material.AIR);
+            ItemStack item = ItemStack.of(Material.AIR);
 
             if (selectedMode == SlimefunGuideMode.SURVIVAL_MODE) {
                 item.setType(Material.CHEST);

src/main/java/io/github/thebusybiscuit/slimefun4/core/networks/cargo/ItemStackAndInteger.java

@@ -42,7 +42,7 @@ public void add(int amount) {
 
     private void initializeItem() {
         if (this.item instanceof ItemStackWrapper) {
-            ItemStack copy = new ItemStack(item.getType(), item.getAmount());
+            ItemStack copy = ItemStack.of(item.getType(), item.getAmount());
             if (this.item.hasItemMeta()) {
                 copy.setItemMeta(this.item.getItemMeta());
             }

src/main/java/io/github/thebusybiscuit/slimefun4/core/services/localization/SlimefunLocalization.java

@@ -315,7 +315,7 @@ protected void loadEmbeddedLanguages() {
 
         if (item == null) {
             // Fixes #3088
-            return new ItemStack(Material.AIR);
+            return ItemStack.of(Material.AIR);
         }
 
         Language language = getLanguage(p);

src/main/java/io/github/thebusybiscuit/slimefun4/implementation/guide/SurvivalSlimefunGuide.java

@@ -484,15 +484,15 @@ private <T extends Recipe> void showRecipeChoices(T recipe, ItemStack[] recipeIt
         RecipeChoice[] choices = Slimefun.getMinecraftRecipeService().getRecipeShape(recipe);
 
         if (choices.length == 1 && choices[0] instanceof MaterialChoice materialChoice) {
-            recipeItems[4] = new ItemStack(materialChoice.getChoices().get(0));
+            recipeItems[4] = ItemStack.of(materialChoice.getChoices().get(0));
 
             if (materialChoice.getChoices().size() > 1) {
                 task.add(recipeSlots[4], materialChoice);
             }
         } else {
             for (int i = 0; i < choices.length; i++) {
                 if (choices[i] instanceof MaterialChoice materialChoice) {
-                    recipeItems[i] = new ItemStack(materialChoice.getChoices().get(0));
+                    recipeItems[i] = ItemStack.of(materialChoice.getChoices().get(0));
 
                     if (materialChoice.getChoices().size() > 1) {
                         task.add(recipeSlots[i], materialChoice);

src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/androids/FarmerAndroid.java

@@ -69,13 +69,13 @@ private ItemStack getDropFromCrop(Material crop) {
         Random random = ThreadLocalRandom.current();
 
         return switch (crop) {
-            case WHEAT -> new ItemStack(Material.WHEAT, random.nextInt(2) + 1);
-            case POTATOES -> new ItemStack(Material.POTATO, random.nextInt(3) + 1);
-            case CARROTS -> new ItemStack(Material.CARROT, random.nextInt(3) + 1);
-            case BEETROOTS -> new ItemStack(Material.BEETROOT, random.nextInt(3) + 1);
-            case COCOA -> new ItemStack(Material.COCOA_BEANS, random.nextInt(3) + 1);
-            case NETHER_WART -> new ItemStack(Material.NETHER_WART, random.nextInt(3) + 1);
-            case SWEET_BERRY_BUSH -> new ItemStack(Material.SWEET_BERRIES, random.nextInt(3) + 1);
+            case WHEAT -> ItemStack.of(Material.WHEAT, random.nextInt(2) + 1);
+            case POTATOES -> ItemStack.of(Material.POTATO, random.nextInt(3) + 1);
+            case CARROTS -> ItemStack.of(Material.CARROT, random.nextInt(3) + 1);
+            case BEETROOTS -> ItemStack.of(Material.BEETROOT, random.nextInt(3) + 1);
+            case COCOA -> ItemStack.of(Material.COCOA_BEANS, random.nextInt(3) + 1);
+            case NETHER_WART -> ItemStack.of(Material.NETHER_WART, random.nextInt(3) + 1);
+            case SWEET_BERRY_BUSH -> ItemStack.of(Material.SWEET_BERRIES, random.nextInt(3) + 1);
             default -> null;
         };
     }

src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/androids/FishermanAndroid.java

@@ -28,23 +28,23 @@ public FishermanAndroid(ItemGroup itemGroup, int tier, SlimefunItemStack item, R
 
         // Fish
         for (Material fish : Tag.ITEMS_FISHES.getValues()) {
-            fishingLoot.add(new ItemStack(fish), 25);
+            fishingLoot.add(ItemStack.of(fish), 25);
         }
 
         // Junk
-        fishingLoot.add(new ItemStack(Material.BONE), 10);
-        fishingLoot.add(new ItemStack(Material.STRING), 10);
-        fishingLoot.add(new ItemStack(Material.INK_SAC), 8);
-        fishingLoot.add(new ItemStack(Material.KELP), 6);
-        fishingLoot.add(new ItemStack(Material.STICK), 5);
-        fishingLoot.add(new ItemStack(Material.ROTTEN_FLESH), 3);
-        fishingLoot.add(new ItemStack(Material.LEATHER), 2);
-        fishingLoot.add(new ItemStack(Material.BAMBOO), 3);
+        fishingLoot.add(ItemStack.of(Material.BONE), 10);
+        fishingLoot.add(ItemStack.of(Material.STRING), 10);
+        fishingLoot.add(ItemStack.of(Material.INK_SAC), 8);
+        fishingLoot.add(ItemStack.of(Material.KELP), 6);
+        fishingLoot.add(ItemStack.of(Material.STICK), 5);
+        fishingLoot.add(ItemStack.of(Material.ROTTEN_FLESH), 3);
+        fishingLoot.add(ItemStack.of(Material.LEATHER), 2);
+        fishingLoot.add(ItemStack.of(Material.BAMBOO), 3);
 
         // "loot"
-        fishingLoot.add(new ItemStack(Material.SADDLE), 1);
-        fishingLoot.add(new ItemStack(Material.NAME_TAG), 1);
-        fishingLoot.add(new ItemStack(Material.NAUTILUS_SHELL), 1);
+        fishingLoot.add(ItemStack.of(Material.SADDLE), 1);
+        fishingLoot.add(ItemStack.of(Material.NAME_TAG), 1);
+        fishingLoot.add(ItemStack.of(Material.NAUTILUS_SHELL), 1);
     }
 
     @Override

src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/androids/MinerAndroid.java

@@ -52,7 +52,7 @@
 public class MinerAndroid extends ProgrammableAndroid {
 
     // Determines the drops a miner android will get
-    private final ItemStack effectivePickaxe = new ItemStack(Material.DIAMOND_PICKAXE);
+    private final ItemStack effectivePickaxe = ItemStack.of(Material.DIAMOND_PICKAXE);
 
     private final ItemSetting<Boolean> firesEvent = new ItemSetting<>(this, "trigger-event-for-generators", false);
     private final ItemSetting<Boolean> applyOptimizations = new ItemSetting<>(this, "reduced-block-updates", true);