search

SirLalaPyon / MedievalOverhaul

13 stars 15 forks source link

More paranoid null checks on timber gen #49

Closed feldoh closed 3 months ago

feldoh commented 4 months ago

Should fix cases where the harvestable thingDef doesn't have stuffprops etc. Perhaps over-paranoid but it's low cost to save error reports later.

feldoh commented 4 months ago

Example tree causing issues: https://github.com/MrSamuelStreamer/RPGFantasyFlavourMod/blob/main/1.5/Defs/ThingDefs_Plants/MSS_ResourceTrees.xml#L166 which spawns VPE_Eltex on harvest.

ViralReaction commented 3 months ago

Was fixed in https://github.com/SirLalaPyon/MedievalOverhaul/pull/50. Lala, you can close this PR