GenDeathrow / MoreChickens

5 stars 14 forks source link

[1.12.2] problem registering some chickens in log file output... #40

Open PhoenixRiver opened 6 years ago

PhoenixRiver commented 6 years ago

Version: morechickens-1.12.2-3.0.0.jar Forge Version: forge1.12.2-14.23.1.2587

These errors turned up in the log file (possibly because I don't have a mod including these items?? I'm pretty sure I've one for bronze, zinc and tin (plus malachite is one of the default minecraft rocks isn't it?) either way I thought you might like to know!):

[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'invarchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'bronzechicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'zincchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'tinchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'platinumchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'brasschicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'cupronickelchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'siliconchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'amberchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'amethystchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'malachitechicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'peridotchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'rubychicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'tanzanitechicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'topazchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'garnetchicken' It's LayItem was null
[15:21:31] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'rubberchicken' It's LayItem was null

Mod list: AIImprovements-1.12-0.0.1b3.jar animania-1.12.2-1.4.3.jar AppleCore-mc1.12.2-3.1.0.jar CarryOn MC1.12 v1.7.jar Chameleon-1.12-4.1.3.jar chameleoncreepers-1.4.1-1.12.jar chickens-6.0.3.jar CookingForBlockheads_1.12.1-6.3.17.jar CraftStudio-1.0.0.93-mc1.12-alpha.jar DynamicSurroundings-1.12.2-3.4.7.2.jar DynamicTrees-1.12.2-0.7.2c.jar Erosion-2.2.1.jar GiacomosFishingNet-1.12.2-1.6.1.jar hatchery-1.12.2-2.1.2.jar Hwyla-1.8.23-B38_1.12.jar iceandfire-1.3.1-1.12.2.jar ImmersiveEngineering-0.12-76-core.jar ImmersiveEngineering-0.12-76.jar japaricraftmod-1.12-3.6.0.jar jehc-1.12-1.3.1.jar jei_1.12.2-4.8.5.132.jar jeiintegration_1.12.2-1.5.1.36.jar journeymap-1.12.2-5.5.2.jar llibrary-1.7.9-1.12.jar Mantle-1.12-1.3.1.21.jar morechickens-1.12.2-3.0.0.jar natura-1.12.2-4.3.2.42.jar OptiFine_1.12.2_HD_U_C8.jar Owls-1.12.2-1.0.1.jar Pam's HarvestCraft 1.12.2f.jar simplytea-1.4.jar SpiceOfLife-mc1.12-1.3.11.jar StorageDrawers-1.12.1-5.3.3.jar Waddles-1.12.2-0.5.6.jar WailaHarvestability-mc1.12-1.1.12.jar

Hope this helps :)

GenDeathrow commented 6 years ago

Yeah those are pretty standard in the log file atm. Its because it cant locate the items, when it try's to register the chicken. malachitechicken isnt the same as you think for that particular chicken. I want to reduce those checks, but its just a limitation to the way chickens get registered with the base mod. you could always delete the morechickns file and let it regenerate. see if it changes. but those errors are normal atm.

PhoenixRiver commented 6 years ago

No problem, hope you don't get too many of these from it especially since it's something outwith your control :/

GenDeathrow commented 6 years ago

I don't, most ppl don't really see that. But I think I may be able to reduce them some.. now that I think about it more.

PhoenixRiver commented 6 years ago

Thank you, it's nice to have clean logs rather than the more normal ones with huge numbers of errors and warnings. Though I have a lot of respect for some of the big mods and their large followings, they also tend to rather blasé about cleaning their stuff up and even some of the better ones spam a lot of exceptions and warnings into the logs :(

I look forward to seeing your work in action :) - still at the wooden tools stage at present - been looking into too many bugs! ;)

PhoenixRiver commented 6 years ago

P.S. I forgot to say that, in case you're interested, I first heard and became interested in you mod due to Geoff and his chicken farm from Achievements Hunters' Let's Play of Sky Factory (I think they're playing v3) :)

GenDeathrow commented 6 years ago

I love hearing how ppl get interested. And I appreciate that you like the mod. I work on Chickens also now, so I have been slowly fixing up the core code to make it easier to handle stuff in more chickens.

PhoenixRiver commented 6 years ago

Version: morechickens-1.12.2-3.1.0

I'm just going to add these lists from Base Metals (which sees to have improved a bit! :) and TechReborn here as well in case you find a way to avoid this issue or something improves in Chickens and you need to know what needs fixed :)

[23:06:00] [main/INFO] [More Chickens]: Loading Base Metals addon...
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'invarchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'bronzechicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'zincchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'tinchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'platinumchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'brasschicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'cupronickelchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'siliconchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'garnetchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (basemetals) Chicken: 'rubberchicken' It's LayItem was null
[23:06:00] [main/INFO] [More Chickens]: Loading Tech Reborn addon...
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (techreborn) Chicken: 'yellowgarnetchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (techreborn) Chicken: 'chromechicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (techreborn) Chicken: 'titaniumchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (techreborn) Chicken: 'tungstenchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (techreborn) Chicken: 'tungstensteelchicken' It's LayItem was null
[23:06:00] [main/ERROR] [More Chickens]: Error Registering (techreborn) Chicken: 'refinedironchicken' It's LayItem was null
tyler489 commented 6 years ago

Is it because forge registers entitys before the items are loaded?

GenDeathrow commented 6 years ago

no because the chicken registry doesn't have a lot to with the entity registry. but it does deal a lot mod load order and items. Its on the todo list to change. but things have been way to busy for me to do that yet.