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

Raw copper has no output if EFR copper is off #177

Closed
1 of 4 tasks
Roadhog360 opened this issue Sep 9, 2022 · 8 comments
Closed
1 of 4 tasks

Raw copper has no output if EFR copper is off #177

Roadhog360 opened this issue Sep 9, 2022 · 8 comments
Labels
Priority: High For serious issues

Comments

@Roadhog360
Copy link
Owner

Roadhog360 commented Sep 9, 2022

Please check any boxes that apply to you and your issue

  • I use a translator application to post this issue.

  • This is a crash. Please upload, Pastebin, Gist or copypaste the whole crash report along with this issue.

  • This is a mod incompatibility. If I do this in vanilla Forge with only Et Futurum Requiem installed, it works normally.

  • This is a code problem. Please link the probably problematic lines of code in the issue and/or explain in detail what is wrong.

Version number of Et-Futurum-Requiem (IMPORTANT)

Specify the actual version, please don't just say "latest", tell us the version number of the mod in-game, or the name the jar had when downloading it.

2.4.1 CI f0a1db5

Describe the issue (IMPORTANT)

A list of steps to reproduce the issue may be helpful too. Screenshots could also prove beneficial for visual issues.

If raw ores are enabled but the EFR copper is off, there's no way to smelt it to another mod's copper ingot. Expected result should be it just smelts to a different copper ingot.
Line 721 in ModRecipes is disabled when copper is off, but then there is no alternative present for raw copper only.

@github-actions github-actions bot added the Invalid Not a proper bug report label Sep 9, 2022
@github-actions
Copy link

github-actions bot commented Sep 9, 2022

@Roadhog360: hello! 👋

This issue is being automatically closed because it does not follow the issue template.

@github-actions github-actions bot closed this as completed Sep 9, 2022
@Roadhog360
Copy link
Owner Author

Go fuck yourself

@Roadhog360 Roadhog360 reopened this Sep 9, 2022
@Roadhog360 Roadhog360 added Priority: Low When an issue is minor and doesn't need attention for a while Priority: High For serious issues and removed Invalid Not a proper bug report Priority: Low When an issue is minor and doesn't need attention for a while labels Sep 9, 2022
@GregoriusT
Copy link
Collaborator

GregoriusT commented Sep 9, 2022

Lol telling GitHub actions to screw with Issue Reports might not be the smartest thing to do, it might turn the slightest typo into an invalid template message. XD

@Roadhog360
Copy link
Owner Author

It is in my best interest to make sure people provide all necessary information.

And besides, I looked at the action file and there was nothing actually checking if the issue used the template, so it must have been unconditionally closing the tickets.

@MikhailTapio
Copy link

Could that be done by simply getting 'OreDictionary.getOres("ingotCopper").get(0)'(if it exists) and adding it to the recipe result?

@Roadhog360
Copy link
Owner Author

That was the exact plan, yes.

@GregoriusT
Copy link
Collaborator

GregoriusT commented Sep 11, 2022

Yeah should work fine, I haven't seen any Mod that adds Copper Ore but doesn't also have Copper ingots,
so the Raw Copper Ore Item should be perfectly unobtainable if no Copper Ingots exist,
meaning this is a safe assumption to make as long as Mods have already registered their Ingots to the OreDict at the time you check the List of Copper Ingots,
and because I have never seen a Mod registering specifically Ingots in PostInit or later (apart from MineTweaker, but that doesn't count), it is fine to do in PostInit.

Also make sure the List is not empty when you try to get(0), because it might end up with an Index Out of Bounds if no Copper exists whatsoever.

@Roadhog360
Copy link
Owner Author

Yes, that too.

The raw ores are intended to be compatible with modded copper ores in which the user may prefer another mod's ingot so it needs to be obtainable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: High For serious issues
Projects
None yet
Development

No branches or pull requests

3 participants