Some config keys break tags when disabled
kwpugh opened this issue ยท 22 comments
Hello,
I'm the author of the Gobber mod and we both have mods in the Medieval Minecraft modpack. When your mod is enabled, the ores in my mod cannot be broken with drops with vanilla or modded pickaxes. Everything is fine if your mod is disabled.
I spent some time looking through your source code on GitHub and I do not see anything in particular that. might cause this.
Any ideas?
Regards,
Kevin
I noticed this in the log after running Medieval Minecraft pack:
[11:52:53] [Worker-Main-4/ERROR]: Couldn't load tag fabric:pickaxes as it is missing following references: additionaladditions:rose_gold_pickaxe (from Additional Additions)
[11:52:53] [Worker-Main-4/ERROR]: Couldn't load tag fabric:axes as it is missing following references: additionaladditions:rose_gold_axe (from Additional Additions)
[11:52:53] [Worker-Main-4/ERROR]: Couldn't load tag fabric:hoes as it is missing following references: additionaladditions:rose_gold_hoe (from Additional Additions)
[11:52:53] [Worker-Main-4/ERROR]: Couldn't load tag fabric:shovels as it is missing following references: additionaladditions:rose_gold_shovel (from Additional Additions)
I think when yours fail, my fail as well.
Hello, Kevin!
Thanks for reporting this issue to me. Is the issue that your blocks don't drop when broken with a pickaxe with Additional Additions enabled? I am unable to reproduce that issue, with both Additional Additions and Gobber2 enabled on their latest versions, as posted on CurseForge. I used vanilla, Additional Additions and Gobber2 pickaxes, and upon breaking the Gobber Ore (gobber2:gobber2_ore), it dropped an item as it should have. I didn't use the Medieval Minecraft modpack for this, instead I used this list of Fabric mods:
- Additional Additions (2.1.0)
- Gobber2 Fabric (2.4.37)
- Lithium (0.7.3)
- Sodium (0.3.0)
The issue may be on the Medieval Minecraft side, or maybe it is intended.
The errors in the log are probably from RoseGold
feature disabled in Additional Additions. Can you please send me your Additional Additions config? Maybe it is because of modified config file.
Oh, I see. You need to wrap each entry, like this: https://github.com/kwpugh/gobber_fabric-1.17/blob/1.17.1/src/main/resources/data/fabric/tags/items/pickaxes.json
I think the issue is that neither of us is the required=false tag for the pickaxe.json fabric tags. If an item is disabled via config AND it is not tagged with required=false, the fabric tag reading will abort reading ANY tags from the pickaxe.json file.
I got bitten by this before with another mod.
Medieval Minecraft disables RoseGold in the config and it busts the process of adding additional pickaxes to the fabric pickaxe.json tag. I will adjust mine and you need to adjust yours, that should fix things.
that way, when the tags are read, if any individual item is not actually registered, it won't stop the process of reading others. Annoying bug in Fabric.
can you send me a test build with the new tags when you finish, so I can test with my mod to confirm the fix?
yes. I disabled rose gold with the latest of your mod and mine and block breaking was normal.
Gobber2-Fabric-1.17.1-2.4.38.jar.zip
here is my latest if you want to try it yourself.
Closed by 771ab78
any tool you add to any of the fabric tool tags (pickaxe.json, axe.json, etc.) that can be disabled via config, will need the wrapper tag around it.
dumb question, how do I get a jar out of that?
https://github.com/Dqu1J/additionaladditions/actions/runs/1133175382