Railcraft

Railcraft

34M Downloads

[Request] Move recipes to Init instead of PreInit

BluSunrize opened this issue ยท 5 comments

commented

Basically, people keep filing error reports that Immersive Engineering's creosote bottles+buckets do not work to create wooden ties. The reason for that is that IE registers it's FluidContainers in PreInit, but not before Railcraft has already registered it's recipes for wooden ties. @malte0811 on my issue tracker has already traced this:

BluSunrize/ImmersiveEngineering#101

The obvious solution would be to move the registration of those recipes to the init phase, assuming that this does mess up other parts of the code (Haven't checked that yet ^^"). A different, probably less breakage inducing way (but not as pretty) would be to not use defineRecipes() in ItemTie and instead register the recipes manually in the init phase.

commented

Hmm...yes fair point.

commented

yes please +1 :)

commented

Recipes should not be done in preInit. Recipes should be done in init or postInit so that all mods have a chance to register their items before recipes are set.

commented

Thank you. Now if only Covert or one of the Railcraft maintainers (provided there are some) would aid with this >_>

commented

related to my issue regrading CraftGuide? #679
Anyway, i support based what you says about code practices.