Exclude parts with realchute modules from setting craft bounds on rec…

…overy. This is a hack, but should fix complaints that recovered planes don't fit hangar dimensions. By definition the plane met the requirements to be integrated and launched, so it's safe to do this on recovery (and any editing will go back through the normal path). Close #2024
KSP-RO · Jul 2, 2023 · 3c01249 · 3c01249
1 parent e1296c3
commit 3c01249
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/Source/KerbalConstructionTime/BuildItems/BuildListVessel.cs b/Source/KerbalConstructionTime/BuildItems/BuildListVessel.cs
@@ -184,7 +184,7 @@ public BuildListVessel(ShipConstruct s, string ls, string flagURL, bool storeCon
 
             StorePartNames(s.parts);
             // Override KSP sizing of the ship construct
-            ShipSize = Utilities.GetShipSize(s, true);
+            ShipSize = Utilities.GetShipSize(s, true, false);
 
             if (storeConstruct)
                 StoreShipConstruct(s);
@@ -361,7 +361,7 @@ private ConfigNode FromInFlightVessel(Vessel VesselToSave, ListType listType)
             ConfigNode cn = ConstructToSave.SaveShip();
             SanitizeShipNode(cn);
             // override KSP sizing of the ship construct
-            cn.SetValue("size", KSPUtil.WriteVector(Utilities.GetShipSize(ConstructToSave, true)));
+            cn.SetValue("size", KSPUtil.WriteVector(Utilities.GetShipSize(ConstructToSave, true, true)));
             // These are actually needed, do not comment them out
             VesselToSave.SetRotation(OriginalRotation);
             VesselToSave.SetPosition(OriginalPosition);

diff --git a/Source/KerbalConstructionTime/Utilities/Utilities.cs b/Source/KerbalConstructionTime/Utilities/Utilities.cs
@@ -224,7 +224,7 @@ public static float GetShipMass(this ShipConstruct sc, bool excludeClamps, out f
         }
 
         // Reimplemented from stock so we ignore tags.
-        public static Vector3 GetShipSize(ShipConstruct ship, bool excludeClamps)
+        public static Vector3 GetShipSize(ShipConstruct ship, bool excludeClamps, bool excludeChutes)
         {
             if (ship.parts.Count == 0)
                 return Vector3.zero;
@@ -248,6 +248,11 @@ public static Vector3 GetShipSize(ShipConstruct ship, bool excludeClamps)
                     if (p.IsClampOrChild())
                         continue;
                 }
+                if (excludeChutes)
+                {
+                    if (p.Modules["RealChuteModule"] != null)
+                        continue;
+                }
 
                 Bounds[] bounds = GetPartRendererBounds(p);
                 Bounds b;