post-kerbin-mining-corporation / CryoTanks

Adds cryogenic fuel storage options and limited fuel switching to Kerbal Space Program
15 stars 30 forks source link

CryoTanks incompatible with some Restock tanks #137

Closed Tahvohck closed 1 year ago

Tahvohck commented 1 year ago

CryoTanks don't correctly apply to Restock tanks. It seems to be a load order issue for the patch: :FOR[zzz_CryoTanks] loads before the :AFTER[zzz_Kerbalism] tag that Restock targets for some of the problematic tanks. However even changing to :LAST[zzz_CryoTanks] doesn't fix the patch (in fact, for some reason it creates double ModularFuelTank modules), so I'm certain that there's more to it than that.

Tanks that I know have the issue:

image image

Tahvohck commented 1 year ago

Nevermind - the duplicate tank issue is something to do with Modular Fuel Tanks adding two MFT modules. The issue with not applying a Cryo Tank to Restock still exists, though with the existing patch set that might actually be MFT failing to support those tanks.

Tahvohck commented 1 year ago

Looks like this is entirely an issue with Modular Fuel Tanks. Adding MFT modules to those tanks properly adds CryoTanks support in as well.