blowfishpro / B9PartSwitch

A Kerbal Space Program plugin designed to implement switching of part meshes, resources, and nodes
GNU Lesser General Public License v3.0
50 stars 33 forks source link

Log warning with only one subtype #114

Closed jsolson closed 5 years ago

jsolson commented 5 years ago

Should I be concerned about this?

[WRN 19:53:49.699] [WARNING] [Part bluedog.Apollo.AARDV.Cargo] [ModuleB9PartSwitch 'cargoSwitch'] Only one subtype found, this may lead to unexpected behavior

We use Ore as a placeholder to define the volume on cargo parts. What gets added to the SUBTYPE's depends on what life support mods are installed, if any are.

blowfishpro commented 5 years ago

Yeah, it occurred to me last night that this might be easier than trying to selectively add ModuleB9PartSwitch based on one of several mods being present. I think it works fine so I guess it makes sense for it to be a supported use case and just disable the UI when it happens?

jsolson commented 5 years ago

You could leave the UI. It advertises to the user the part could do more than be an Ore container if they wanted. I had envisioned someone using a personal patch to make cargo subtypes for whatever their need is.