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 Report crash when making a world #818

Closed
1 task done
VaporeonScripts opened this issue Sep 27, 2024 · 10 comments · Fixed by #820
Closed
1 task done

Bug Report crash when making a world #818

VaporeonScripts opened this issue Sep 27, 2024 · 10 comments · Fixed by #820
Labels
P-0-High High priority, crash/breaking. Status-Triage A Triage Member should take a look at this Type-Bug There is a problem.

Comments

@VaporeonScripts
Copy link

Is there an existing issue for this?

  • I did not find any existing issues.

Current Behavior

https://mclo.gs/rp0BoCU

Expected Behavior

crash when making a world

How to Test

add the mod

Local Environment

- **Operating System**: arch
- **Java version**: 21
- **Minecraft Version**: 1.21.1
- **Version**: latest
- **Other Mods**: too many

Any Additional Information?

No response

@VaporeonScripts VaporeonScripts added Status-Triage A Triage Member should take a look at this Type-Bug There is a problem. labels Sep 27, 2024
@dphaldes
Copy link
Contributor

java.lang.IllegalArgumentException: Multiple entries with same key: enderio:smelting/bread_from_smelting=enderio:smelting/bread_from_smelting and enderio:smelting/bread_from_smelting=enderio:smelting/bread_from_smelting

I wonder if its #812. @Rover656 @rlnt

@dphaldes dphaldes added the P-0-High High priority, crash/breaking. label Sep 27, 2024
@rlnt
Copy link
Contributor

rlnt commented Sep 27, 2024

Interesting, the PR converts all smelting recipes to Alloy Smelter recipes. I changed the namespace to EnderIO and even prefixed the recipe ID path with smelting/ and it still conflicts. 🤔

@Rover656
Copy link
Member

I think we need to also include the namespace of the inherited recipe, so enderio:/minecraft/smelting/...

@rlnt
Copy link
Contributor

rlnt commented Sep 27, 2024

@VaporeonScripts This is an easy fix if we make the recipe ID unique enough but I want to know something just for clarity. Do you happen to have a custom data pack or KubeJS script that adds recipes that may have this exact ID? It should not be possible to have a duplicate recipe ID within the mod itself considering the duplicate ID even uses the EnderIO namespace.

@VaporeonScripts
Copy link
Author

@VaporeonScripts This is an easy fix if we make the recipe ID unique enough but I want to know something just for clarity. Do you happen to have a custom data pack or KubeJS script that adds recipes that may have this exact ID? It should not be possible to have a duplicate recipe ID within the mod itself considering the duplicate ID even uses the EnderIO namespace.

Idrk,i might check later if I got some kube js script that does that

@rlnt
Copy link
Contributor

rlnt commented Sep 27, 2024

All good, nevermind. It was a conflict with Farmers Delight. Good that you brought it up. Didn't have this edge case in mind. Thank you.

@VaporeonScripts
Copy link
Author

alright! no problem

@VaporeonScripts
Copy link
Author

i get a new issue now
https://mclo.gs/TY4xgH1

@VaporeonScripts
Copy link
Author

VaporeonScripts commented Sep 27, 2024

is it related somehow to ender io
image

@VaporeonScripts
Copy link
Author

VaporeonScripts commented Sep 27, 2024

yup,using 7.0.2 works fine, its a 7.0.3 and 7.0.4 error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-0-High High priority, crash/breaking. Status-Triage A Triage Member should take a look at this Type-Bug There is a problem.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants