Merge pull request #32 from sweetrpg/1.18

Release
sweetrpg · Nov 30, 2024 · 3f7a562 · 3f7a562
2 parents ed226a7 + cdabffc
commit 3f7a562
Show file tree

Hide file tree

Showing 16 changed files with 360 additions and 61 deletions.
diff --git a/.release-info/1.18/RELEASE_HASH b/.release-info/1.18/RELEASE_HASH
@@ -1 +1 @@
-b5592d703c7c7533eec9fb420a87bd76b4532e7a
+ed226a7a5730d6a9a3f1050d5c840e23298d1224
diff --git a/.release-info/1.18/VERSION b/.release-info/1.18/VERSION
@@ -1 +1 @@
-0.1.1
+0.1.5
diff --git a/CHANGELOG/1.18/0.1.1.md b/CHANGELOG/1.18/0.1.1.md
@@ -0,0 +1,4 @@
+# 0.1.1
+
+- `[NEW]` The craft queue and shopping list overlays may now be positioned relative to the
+  right-edge or bottom-edge of the screen by specifying negative X or Y positions (respectively)
diff --git a/CHANGELOG/1.18/current.md b/CHANGELOG/1.18/current.md
@@ -1,2 +1,4 @@
-- `[NEW]` The craft queue and shopping list overlays may now be positioned relative to the
-  right-edge or bottom-edge of the screen by specifying negative X or Y positions (respectively)
+- `[NEW]` When an end product's recipe intermediates have multiple variations, the "least expensive"
+  one is chosen
+- `[NEW]` Added hotkey to clear the shopping list
+- `[NEW]` Items in the queue and shopping list are now sorted by name
diff --git a/README.md b/README.md
@@ -25,7 +25,4 @@
 
 ## Contributing
 
-
 Download it from [CurseForge](https://www.curseforge.com/minecraft/mc-mods/craft-tracker) or [Modrinth](https://modrinth.com/mod/craft-tracker).
-
-Want to know more about the mod? Take a peek at the [wiki](https://github.com/sweetrpg/CraftTracker/wiki)!
diff --git a/gradle.properties b/gradle.properties
@@ -23,10 +23,10 @@ forge_version_range=[40.2.17,)
 minecraft_version_range=[1.18, 1.19)
 
 curseforge_project_id=626455
-curseforge_release_type=alpha
+curseforge_release_type=beta
 
 modrinth_project_id=6eqIziNz
-modrinth_release_type=alpha
+modrinth_release_type=beta
 
 # JEI
 jei_version=1.18.2:9.7.2.281

diff --git a/src/main/java/com/sweetrpg/crafttracker/client/event/ClientEventHandler.java b/src/main/java/com/sweetrpg/crafttracker/client/event/ClientEventHandler.java
@@ -90,6 +90,12 @@ else if(KeyUtil.isKeyDown(event.getKey()) &&
 
                 handlePopulateShoppingList();
             }
+            else if(KeyUtil.isKeyDown(event.getKey()) &&
+                    ModKeyBindings.CLEAR_SHOPPING_LIST_MAPPING.matches(event.getKey(), event.getScanCode())) {
+                CraftTracker.LOGGER.debug("#onKeyInput: CLEAR_SHOPPING_LIST_MAPPING");
+
+                handleClearShoppingList();
+            }
 
             return;
         }
@@ -130,6 +136,15 @@ private static void handleToggleCraftList() {
         }
     }
 
+    private static void handleClearShoppingList() {
+        CraftTracker.LOGGER.debug("#handleClearShoppingList");
+
+        var player = Minecraft.getInstance().player;
+        var sMgr = ShoppingListManager.INSTANCE;
+
+        sMgr.clearItems(player);
+    }
+
     private static void handlePopulateShoppingList() {
         CraftTracker.LOGGER.debug("#handlePopulateShoppingList");
 

diff --git a/src/main/java/com/sweetrpg/crafttracker/client/overlay/CraftQueueOverlay.java b/src/main/java/com/sweetrpg/crafttracker/client/overlay/CraftQueueOverlay.java
@@ -35,7 +35,11 @@ public class CraftQueueOverlay {
         CraftTracker.LOGGER.trace("CRAFT_QUEUE");
 
         var mgr = CraftingQueueManager.INSTANCE;
-        var products = mgr.getEndProducts();
+        var products = mgr.getEndProducts().stream().sorted((i1, i2) -> {
+            var item1 = ForgeRegistries.ITEMS.getValue(i1.getItemId());
+            var item2 = ForgeRegistries.ITEMS.getValue(i2.getItemId());
+            return item1.getDescription().getString().compareTo(item2.getDescription().getString());
+        }).toList();
 
         switch(CTRuntime.INSTANCE.queueOverlayRequestedState) {
             case SHOW:
@@ -129,8 +133,13 @@ public class CraftQueueOverlay {
             CraftTracker.LOGGER.trace("yPos (after intermediates title): {}", yPos);
 
             // items
-            for(int i = 0; i < mgr.getIntermediates().size(); i++) {
-                var inter = mgr.getIntermediates().get(i);
+            var sortedIntermediates = mgr.getIntermediates().stream().sorted((i1, i2) -> {
+                var item1 = ForgeRegistries.ITEMS.getValue(i1.getItemId());
+                var item2 = ForgeRegistries.ITEMS.getValue(i2.getItemId());
+                return item1.getDescription().getString().compareTo(item2.getDescription().getString());
+            }).toList();
+            for(int i = 0; i < sortedIntermediates.size(); i++) {
+                var inter = sortedIntermediates.get(i);
 
                 var item = ForgeRegistries.ITEMS.getValue(inter.getItemId());
                 var stack = item.getDefaultInstance();
@@ -179,8 +188,13 @@ public class CraftQueueOverlay {
             CraftTracker.LOGGER.trace("yPos (after materials title): {}", yPos);
 
             // items
-            for(int i = 0; i < mgr.getRawMaterials().size(); i++) {
-                var m = mgr.getRawMaterials().get(i);
+            var sortedMaterials = mgr.getRawMaterials().stream().sorted((i1, i2) -> {
+                var item1 = ForgeRegistries.ITEMS.getValue(i1.getItemId());
+                var item2 = ForgeRegistries.ITEMS.getValue(i2.getItemId());
+                return item1.getDescription().getString().compareTo(item2.getDescription().getString());
+            }).toList();
+            for(int i = 0; i < sortedMaterials.size(); i++) {
+                var m = sortedMaterials.get(i);
 
                 var item = ForgeRegistries.ITEMS.getValue(m.getItemId());
                 var stack = item.getDefaultInstance();
@@ -229,8 +243,13 @@ public class CraftQueueOverlay {
             CraftTracker.LOGGER.trace("yPos: {}", yPos);
 
             // items
-            for(int i = 0; i < mgr.getFuel().size(); i++) {
-                var f = mgr.getFuel().get(i);
+            var sortedFuels = mgr.getFuel().stream().sorted((i1, i2) -> {
+                var item1 = ForgeRegistries.ITEMS.getValue(i1.getItemId());
+                var item2 = ForgeRegistries.ITEMS.getValue(i2.getItemId());
+                return item1.getDescription().getString().compareTo(item2.getDescription().getString());
+            }).toList();
+            for(int i = 0; i < sortedFuels.size(); i++) {
+                var f = sortedFuels.get(i);
 
                 var item = ForgeRegistries.ITEMS.getValue(f.getItemId());
                 var stack = item.getDefaultInstance();

diff --git a/src/main/java/com/sweetrpg/crafttracker/client/overlay/ShoppingListOverlay.java b/src/main/java/com/sweetrpg/crafttracker/client/overlay/ShoppingListOverlay.java
@@ -92,8 +92,13 @@ public class ShoppingListOverlay {
         var inventory = Minecraft.getInstance().player.getInventory();
 
         // items
-        for(int i = 0; i < mgr.getItems().size(); i++) {
-            var m = mgr.getItems().get(i);
+        var sortedItems = mgr.getItems().stream().sorted((i1, i2) -> {
+            var item1 = ForgeRegistries.ITEMS.getValue(i1.getItemId());
+            var item2 = ForgeRegistries.ITEMS.getValue(i2.getItemId());
+            return item1.getDescription().getString().compareTo(item2.getDescription().getString());
+        }).toList();
+        for(int i = 0; i < sortedItems.size(); i++) {
+            var m = sortedItems.get(i);
 
             var item = ForgeRegistries.ITEMS.getValue(m.getItemId());
             var stack = item.getDefaultInstance();

diff --git a/src/main/java/com/sweetrpg/crafttracker/common/manager/CraftingQueueManager.java b/src/main/java/com/sweetrpg/crafttracker/common/manager/CraftingQueueManager.java
@@ -133,11 +133,7 @@ public void addProduct(Player player, ResourceLocation itemId, int quantity) {
             endProducts.compute(itemId, (rl, p) -> p == null ? product :
                     new CraftingQueueProduct(p.getItemId(), p.getRecipes(), p.getQuantity() + quantity));
 
-//            CraftingQueueStorage.get(level).putData(itemId, quantity);
-
             computeAll();
-
-//            PacketHandler.sendToPlayer(this.player, new UpdateCraftQueueData(this.getEndProducts()));
         }
         else {
             CraftTracker.LOGGER.info("Not adding {} to queue, since there are no recipes for it.", itemId);
@@ -167,6 +163,10 @@ public void removeProduct(Player player, ResourceLocation itemId) {
         CraftTracker.LOGGER.debug("CraftingQueueManager#removeProduct: {}", itemId);
 
         this.endProducts.remove(itemId);
+
+        computeAll();
+
+        this.save(player);
     }
 
     public void removeProduct(Player player, ResourceLocation itemId, int quantity) {
@@ -307,7 +307,8 @@ public void computeRecipe(Recipe recipe, int recipeQuantity) {
                                             });
                                         }
 
-                                        this.computeRecipe(subRecipes.get(0), recipeQuantity);
+                                        int subIndex = RecipeUtil.chooseLeastExpensiveOf(subRecipes);
+                                        this.computeRecipe(subRecipes.get(subIndex), recipeQuantity);
                                     }
                                 });
                     }

diff --git a/src/main/java/com/sweetrpg/crafttracker/common/util/RecipeUtil.java b/src/main/java/com/sweetrpg/crafttracker/common/util/RecipeUtil.java
@@ -1,26 +1,67 @@
 package com.sweetrpg.crafttracker.common.util;
 
 import com.sweetrpg.crafttracker.CraftTracker;
+import com.sweetrpg.crafttracker.common.lib.Constants;
 import net.minecraft.client.Minecraft;
 import net.minecraft.core.NonNullList;
 import net.minecraft.resources.ResourceLocation;
+import net.minecraft.tags.TagKey;
+import net.minecraft.util.Tuple;
+import net.minecraft.world.item.Item;
+import net.minecraft.world.item.ItemStack;
 import net.minecraft.world.item.crafting.Ingredient;
 import net.minecraft.world.item.crafting.Recipe;
 
-import java.util.Arrays;
-import java.util.List;
-import java.util.Set;
+import java.io.IOException;
+import java.util.*;
 import java.util.stream.Collectors;
 
 public class RecipeUtil {
 
+    private static Map<ResourceLocation, Integer> ingredientCostsByTag = new HashMap<>();
+    private static Map<ResourceLocation, Integer> ingredientCostOverrides = new HashMap<>();
+
+    static {
+        var mgr = Minecraft.getInstance().getResourceManager();
+
+        var ingCostsResource = new ResourceLocation(Constants.MOD_ID, "ingredient-costs.properties");
+        try {
+            var costs = mgr.getResource(ingCostsResource);
+            var props = new Properties();
+            props.load(costs.getInputStream());
+            props.entrySet().forEach(entry -> {
+                var key = new ResourceLocation((String) entry.getKey());
+                var value = Integer.parseInt((String) entry.getValue());
+                ingredientCostsByTag.put(key, value);
+            });
+        }
+        catch (IOException e) {
+            CraftTracker.LOGGER.error("I/O exception when trying to load ingredient costs file", e);
+        }
+
+        var ingOverridesResource = new ResourceLocation(Constants.MOD_ID, "ingredient-overrides.properties");
+        try {
+            var costs = mgr.getResource(ingOverridesResource);
+            var props = new Properties();
+            props.load(costs.getInputStream());
+            props.entrySet().forEach(entry -> {
+                var key = new ResourceLocation((String) entry.getKey());
+                var value = Integer.parseInt((String) entry.getValue());
+                ingredientCostOverrides.put(key, value);
+            });
+        }
+        catch (IOException e) {
+            CraftTracker.LOGGER.error("I/O exception when trying to load ingredient overrides file", e);
+        }
+    }
+
     public static List<? extends Recipe<?>> getRecipesFor(ResourceLocation itemId) {
         CraftTracker.LOGGER.debug("RecipeUtil#getRecipesFor: {}", itemId);
 
         var mgr = Minecraft.getInstance().level.getRecipeManager();
         var recipes = mgr.getRecipes().stream()
                 .filter(r -> r.getResultItem().getItem().getRegistryName().equals(itemId))
-                .collect(Collectors.toUnmodifiableList());
+                .toList();
 
         CraftTracker.LOGGER.debug("RecipeUtil#getRecipesFor: recipes {}", recipes);
         return recipes;
@@ -30,12 +71,71 @@ public static boolean areIngredientsSame(NonNullList<Ingredient> ingredients) {
         CraftTracker.LOGGER.debug("RecipeUtil#areIngredientsSame: {}", ingredients);
 
         Set<String> ing = ingredients.stream()
-                .map((i) -> Arrays.asList(i.getItems()))
-                .filter((l) -> !l.isEmpty())
-                .map((l) -> l.get(0))
-                .map((i) -> i.getItem().getRegistryName().toString())
+                .map(i -> Arrays.asList(i.getItems()))
+                .filter(l -> !l.isEmpty())
+                .map(l -> l.get(0))
+                .map(i -> i.getItem().getRegistryName().toString())
                 .collect(Collectors.toSet());
 
         return ing.size() == 1;
     }
+
+    public static int calculateRecipeCost(Recipe<?> recipe) {
+        CraftTracker.LOGGER.debug("RecipeUtil#calculateRecipeCost: {}", recipe);
+
+        return recipe.getIngredients().stream()
+                .map(RecipeUtil::getIngredientCost)
+                .reduce(0, Integer::sum);
+    }
+
+    public static int getIngredientCost(Ingredient ingredient) {
+        CraftTracker.LOGGER.debug("RecipeUtil#getIngredientCost: {}", ingredient);
+
+        for(ItemStack stack : ingredient.getItems()) {
+            // is the item in the override list?
+            var itemId = stack.getItem().getRegistryName();
+            if(ingredientCostOverrides.containsKey(itemId)) {
+                CraftTracker.LOGGER.debug("found item {} in override list", itemId);
+                return ingredientCostOverrides.get(itemId);
+            }
+
+            // it's not, so check its tags
+            if(stack.hasTag()) {
+                for(TagKey<Item> tag : stack.getTags().toList()) {
+                    var tagId = tag.location();
+                    if(ingredientCostsByTag.containsKey(tagId)) {
+                        CraftTracker.LOGGER.debug("found item {} in tag list", tagId);
+                        return ingredientCostsByTag.get(tagId);
+                    }
+                }
+            }
+        }
+
+        return 1;
+    }
+
+    public static int chooseLeastExpensiveOf(List<? extends Recipe<?>> recipes) {
+        CraftTracker.LOGGER.debug("RecipeUtil#chooseLeastExpensiveOf: {}", recipes);
+
+        List<Tuple<ResourceLocation, Integer>> recipeCosts = new ArrayList<>();
+
+        for(Recipe<?> recipe : recipes) {
+            var cost = RecipeUtil.calculateRecipeCost(recipe);
+            var tuple = new Tuple<>(recipe.getId(), cost);
+
+            recipeCosts.add(tuple);
+        }
+
+        int lowestCostIndex = 0;
+        int lowestCost = Integer.MAX_VALUE;
+        for(int i = 0; i < recipeCosts.size(); i++) {
+            var cost = recipeCosts.get(i).getB();
+            if(cost < lowestCost) {
+                lowestCostIndex = i;
+                lowestCost = cost;
+            }
+        }
+
+        return lowestCostIndex;
+    }
 }
diff --git a/src/main/java/com/sweetrpg/crafttracker/common/util/TextUtils.java b/src/main/java/com/sweetrpg/crafttracker/common/util/TextUtils.java
@@ -9,7 +9,7 @@ public class TextUtils {
     private static final MutableComponent NO_EFFECTS = (new TranslatableComponent("effect.none")).withStyle(ChatFormatting.GRAY);
 
     /**
-     * Syntactic sugar for custom translation keys. Always prefixed with the mod's ID in lang files (e.g. farmersdelight.your.key.here).
+     * Syntactic sugar for custom translation keys. Always prefixed with the mod's ID in lang files (e.g. crafttracker.your.key.here).
      */
     public static MutableComponent getTranslation(String type, String key, Object... args) {
         return new TranslatableComponent(type + "." + Constants.MOD_ID + "." + key, args);
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		b5592d703c7c7533eec9fb420a87bd76b4532e7a
		ed226a7a5730d6a9a3f1050d5c840e23298d1224
Original file line number	Diff line number	Diff line change
Expand Up		@@ -25,7 +25,4 @@

		## Contributing


		Download it from [CurseForge](https://www.curseforge.com/minecraft/mc-mods/craft-tracker) or [Modrinth](https://modrinth.com/mod/craft-tracker).

		Want to know more about the mod? Take a peek at the [wiki](https://github.com/sweetrpg/CraftTracker/wiki)!