Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Tags do not work correctly in repairing recipes #2546

Open
6 tasks done
Partonetrain opened this issue Feb 7, 2025 · 4 comments
Open
6 tasks done

Bug: Tags do not work correctly in repairing recipes #2546

Partonetrain opened this issue Feb 7, 2025 · 4 comments
Assignees
Labels
status/blocked This issue depends on another issue being resolved first. status/dependent This issue requires changes in external API. type/enhancement This issue is associated with improvements to an existing feature.

Comments

@Partonetrain
Copy link

Partonetrain commented Feb 7, 2025

What Feature Types Apply to This Bug?

Other (Please Describe)

Other Type

Crafting

What Type of Bug Is This?

Design

Mod Loader

NeoForge

Mod Loader Version

21.1.115

The Aether Version

1.21.1-1.5.3

Is This Bug a Conflict With Another Mod?

no

Client Log

n/a

Crash Report (if applicable)

n/a

Steps to Reproduce

  1. Create a datapack with this recipe:
{
  "type": "aether:repairing",
  "ingredient": {
    "tag": "minecraft:enchantable/durability"
  },
  "repairTime": 1500
}

(any valid tag will demonstrate this)
2. Attempt to perform recipe in-game:
Image
3. observe the incorrect output:
Image
(it seems to be leather boots because that happens to be the 0th item in the #minecraft:enchantable/durability tag)

What You Expect To Happen

The item that was input it output, but repaired

What Actually Happened

leather boots were output

Additional Details

Unrelated, but CraftTweaker does not consider aether:repairing a recipetype. However, <recipetype:aether:enchanting>.removeByRegex("aether:.*_repairing"); seemed to work for removing all other repairing recipes. (This is just a note, having this script or not does not change the outcome of the custom recipe)

Please Read and Confirm The Following

  • I have confirmed this bug can be replicated without the use of Optifine.
  • I have confirmed this bug is on the most recently supported version of Minecraft.
  • I have confirmed the details provided in this report are concise as possible and does not contain vague information (ie. Versions are properly recorded, answers to questions are clear).
  • I have confirmed this bug is unique and has not been reported already.
  • If playing on a modpack, I have reported this bug to their issue tracker already.
  • I have confirmed that I'm reporting a bug in The Aether I, not The Aether II.
@bconlon1
Copy link
Contributor

bconlon1 commented Feb 7, 2025

Repairing isn't a separate recipe type, it's just a separate recipe serializer type, so that would be why CraftTweaker wouldn't distinguish it. As for the tag support, I'll see if I can get that working; it previously was considered intended that it didn't work with tags due to a limitation with recipe results but some changes in 1.21.1 might make it doable now.

@bconlon1
Copy link
Contributor

bconlon1 commented Feb 7, 2025

Actually nevermind, immediately noticing that it still isn't doable since there's not input slot context for the recipe book's display of results; might have to save this for looking into in 1.21.4 where there were changes to the recipe book code.

@bconlon1 bconlon1 added status/blocked This issue depends on another issue being resolved first. status/dependent This issue requires changes in external API. labels Feb 7, 2025
@bconlon1 bconlon1 self-assigned this Feb 7, 2025
@Partonetrain
Copy link
Author

If it helps anyone out there, I made a quick Python script that generates repair recipes for datapack developers (or modders for that matter):
https://github.com/Partonetrain/AetherRepairRecipeGen

@bconlon1
Copy link
Contributor

Nice! I'd been wanting to make something like this for awhile since I know the repair recipes specifically can be tedious to write but I never got around to it. Would you be willing to share it in the Aether Discord as well?

@bconlon1 bconlon1 added type/enhancement This issue is associated with improvements to an existing feature. and removed type/bug labels Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/blocked This issue depends on another issue being resolved first. status/dependent This issue requires changes in external API. type/enhancement This issue is associated with improvements to an existing feature.
Projects
Status: Back Log
Development

No branches or pull requests

2 participants