Fix reactor descriptions: fuel requirements

modded-factorio / bobsmods

Factorio mods by Bobingabout

https://mods.factorio.com/user/Bobingabout

Other

65 stars 31 forks source link

Closed dmeltzer closed 9 months ago

dmeltzer commented 10 months ago

Currently, all three tiers of reactor use the same tooltip, which is misleading re: the fuels they accept. This fixes it.

There might be a cleaner lua way to handle the i18n object--I'll not pretend to be a lua expert.

KiwiHawk commented 9 months ago

Good start! But had some issues.

Would crash if bobpower was enabled but bobrevamp was not (referencing non existing setting).
If just bobplates was enabled, reactor description still referenced uranium fuel cells when the reactor could take multiple types.
- If just bobpower was enabled, reactor description was unnecessarily being updated to reference nuclear fuel cells rather than uranium fuel cells. Other fuel cell types will only exist if bobplates is enabled.