Merge 8dd052c9ca into ebd1db2d37

fixes #101

CHANGELOG.md

@@ -2,11 +2,13 @@
 
 All notable changes to this project will be documented in this file.
 
-The format is based on [Keep a Changelog],
-and this project adheres to [Semantic Versioning].
-
 ## Unreleased
-- /
+
+- added better logging for cases where items are assigned to multiple unification tags
+- added Turkish translation ([#102](https://github.com/AlmostReliable/almostunified/pull/102))
+- fixed crash when runtime isn't loaded ([#101](https://github.com/AlmostReliable/almostunified/issues/101))
+- fixed newly created custom tags not being considered for unification
+- fixed runtime not being available when items are assigned to multiple unification tags
 
 ## [1.2.0] - 2024-10-06
 
@@ -32,10 +34,6 @@ and this project adheres to [Semantic Versioning].
 
 Initial 1.21.1 port.
 
-<!-- Links -->
-[keep a changelog]: https://keepachangelog.com/en/1.0.0/
-[semantic versioning]: https://semver.org/spec/v2.0.0.html
-
 <!-- Versions -->
 [1.2.0]: https://github.com/AlmostReliable/almostunified/releases/tag/v1.21.1-1.2.0
 [1.1.0]: https://github.com/AlmostReliable/almostunified/releases/tag/v1.21.1-1.1.0

Common/src/main/java/com/almostreliable/unified/AlmostUnifiedCommon.java

@@ -46,12 +46,12 @@ public final class AlmostUnifiedCommon {
     }
 
     public static void onRecipeManagerError(ResourceLocation recipe) {
-        assert RUNTIME != null;
+        if (RUNTIME == null) return;
         RUNTIME.getDebugHandler().collectRecipeError(recipe);
     }
 
     public static void onRecipeManagerEnd() {
-        assert RUNTIME != null;
+        if (RUNTIME == null) return;
         RUNTIME.getDebugHandler().finish();
     }
 }

Common/src/main/java/com/almostreliable/unified/core/AlmostUnifiedRuntimeImpl.java

@@ -76,12 +76,7 @@ public final class AlmostUnifiedRuntimeImpl implements AlmostUnifiedRuntime {
         var tagConfig = Config.load(TagConfig.NAME, TagConfig.SERIALIZER);
         var unificationConfigs = UnificationConfig.safeLoadConfigs();
 
-        var unificationTags = bakeAndValidateTags(
-            unificationConfigs,
-            itemTags,
-            placeholderConfig,
-            debugConfig.shouldLogInvalidTags()
-        );
+        TagReloadHandler.applyCustomTags(tagConfig.getCustomTags(), itemTags);
 
         CustomIngredientUnifierRegistry ingredientUnifierRegistry = new CustomIngredientUnifierRegistryImpl();
         PluginManager.instance().registerCustomIngredientUnifiers(ingredientUnifierRegistry);
@@ -89,7 +84,13 @@ public final class AlmostUnifiedRuntimeImpl implements AlmostUnifiedRuntime {
         PluginManager.instance().registerRecipeUnifiers(recipeUnifierRegistry);
         // TODO: add plugin support for registering config defaults
 
-        TagReloadHandler.applyCustomTags(tagConfig.getCustomTags(), itemTags);
+        var unificationTags = bakeAndValidateTags(
+            unificationConfigs,
+            itemTags,
+            placeholderConfig,
+            debugConfig.shouldLogInvalidTags()
+        );
+
         TagSubstitutionsImpl tagSubstitutions = TagSubstitutionsImpl.create(
             itemTags::has,
             unificationTags::contains,

Common/src/main/java/com/almostreliable/unified/unification/UnificationLookupImpl.java

@@ -7,6 +7,7 @@ import net.minecraft.world.item.Item;
 import net.minecraft.world.item.ItemStack;
 import net.minecraft.world.item.crafting.Ingredient;
 
+import com.almostreliable.unified.AlmostUnifiedCommon;
 import com.almostreliable.unified.api.unification.ModPriorities;
 import com.almostreliable.unified.api.unification.StoneVariants;
 import com.almostreliable.unified.api.unification.TagSubstitutions;
@@ -117,15 +118,22 @@ public final class UnificationLookupImpl implements UnificationLookup {
 
     public static class Builder {
 
-        private final Set<UnificationEntry<Item>> createdEntries = new HashSet<>();
+        private final Map<UnificationEntry<Item>, TagKey<Item>> entriesToTags = new HashMap<>();
         private final Map<TagKey<Item>, Set<UnificationEntry<Item>>> tagsToEntries = new HashMap<>();
 
         private void put(TagKey<Item> tag, UnificationEntry<Item> entry) {
-            if (createdEntries.contains(entry)) {
-                throw new IllegalStateException("entry " + entry + " already created");
+            if (entriesToTags.containsKey(entry)) {
+                var boundTag = entriesToTags.get(entry);
+                AlmostUnifiedCommon.LOGGER.error(
+                    "Unification entry for item '{}' with tag '#{}' is already part of tag '#{}'.",
+                    entry.id(),
+                    tag.location(),
+                    boundTag.location()
+                );
+                return;
             }
 
-            createdEntries.add(entry);
+            entriesToTags.put(entry, tag);
             tagsToEntries.computeIfAbsent(tag, $ -> new HashSet<>()).add(entry);
         }
 

Common/src/main/resources/assets/almostunified/lang/tr_tr.json

@@ -0,0 +1,8 @@
+{
+  "almostunified.description": "Almost Unified tarafından değiştirildi!",
+  "almostunified.warning": "Tarif sorunlarını orijinal sahibine bildirmeyin.",
+  "almostunified.unified": "Birleştirilmiş",
+  "almostunified.duplicate": "Kopyaları Vardı",
+  "almostunified.yes": "Evet",
+  "almostunified.no": "Hayır"
+}