Allow creation of more than one ring (#374)

Because extinction

NewHorizons/Builder/Body/ProxyBuilder.cs

@@ -120,10 +120,13 @@ namespace NewHorizons.Builder.Body
                     }
                 }
 
-                if (body.Config.Ring != null)
+                if (body.Config.Rings != null)
                 {
-                    RingBuilder.MakeRingGraphics(proxy, null, body.Config.Ring, body.Mod);
+                    foreach (var ring in body.Config.Rings)
-                    if (realSize < body.Config.Ring.outerRadius) realSize = body.Config.Ring.outerRadius;
+                    {
+                        RingBuilder.MakeRingGraphics(proxy, null, ring, body.Mod);
+                        if (realSize < ring.outerRadius) realSize = ring.outerRadius;
+                    }
                 }
 
                 Renderer starAtmosphere = null;

NewHorizons/Builder/Body/RingBuilder.cs

@@ -73,7 +73,7 @@ namespace NewHorizons.Builder.Body
                 return null;
             }
 
-            var ringGO = new GameObject("Ring");
+            var ringGO = new GameObject(!string.IsNullOrEmpty(ring.rename) ? ring.rename : "Ring");
             ringGO.transform.parent = sector?.transform ?? rootObject.transform;
             ringGO.transform.position = rootObject.transform.position;
             ringGO.transform.rotation = rootObject.transform.rotation;

NewHorizons/External/Configs/PlanetConfig.cs

@@ -67,6 +67,10 @@ namespace NewHorizons.External.Configs
 
         [Obsolete("Signal is deprecated, please use Props->signals")]
         public SignalModule Signal;
+
+        [Obsolete("Ring is deprecated, please use Rings")]
+        public RingModule Ring;
+
         #endregion Obsolete
 
         /// <summary>
@@ -135,9 +139,9 @@ namespace NewHorizons.External.Configs
         public string[] removeChildren;
 
         /// <summary>
-        /// Creates a ring around the planet
+        /// Create rings around the planet
         /// </summary>
-        public RingModule Ring;
+        public RingModule[] Rings;
 
         /// <summary>
         /// Add sand to this planet
@@ -401,10 +405,20 @@ namespace NewHorizons.External.Configs
                 if (!string.IsNullOrEmpty(Cloak.audioFilePath)) Cloak.audio = Cloak.audioFilePath;
             }
 
-            // Rings are no longer variable size module
+            // Ring is now a list so you can have many per planet
             if (Ring != null)
             {
-                if (Ring.curve != null) Ring.scaleCurve = Ring.curve;
+                if (Rings == null) Rings = new RingModule[0];
+                Rings = Rings.Append(Ring).ToArray();
+            }
+
+            // Rings are no longer variable size module
+            if (Rings != null)
+            {
+                foreach (var ring in Rings)
+                {
+                    if (ring.curve != null) ring.scaleCurve = ring.curve;
+                }
             }
         }
     }

NewHorizons/External/Modules/RingModule.cs

@@ -62,5 +62,10 @@ namespace NewHorizons.External.Modules
         /// Fade rings in/out over time. Optional. Value between 0-1, time is in minutes.
         /// </summary>
         public TimeValuePair[] opacityCurve;
+
+        /// <summary>
+        /// An optional rename of this object
+        /// </summary>
+        public string rename;
     }
 }

NewHorizons/Handlers/PlanetCreationHandler.cs

@@ -547,9 +547,12 @@ namespace NewHorizons.Handlers
                 }
             }
 
-            if (body.Config.Ring != null)
+            if (body.Config.Rings != null)
             {
-                RingBuilder.Make(go, sector, body.Config.Ring, body.Mod);
+                foreach (var ring in body.Config.Rings)
+                {
+                    RingBuilder.Make(go, sector, ring, body.Mod);
+                }
             }
 
             if (body.Config.AsteroidBelt != null)

NewHorizons/Handlers/TitleSceneHandler.cs

@@ -107,13 +107,16 @@ namespace NewHorizons.Handlers
             }
             pivot.name = "Pivot";
 
-            if (body.Config.Ring != null)
+            if (body.Config.Rings != null && body.Config.Rings.Length > 0)
             {
-                RingModule newRing = new RingModule();
+                foreach (var ring in body.Config.Rings)
-                newRing.innerRadius = size * 1.2f;
+                {
-                newRing.outerRadius = size * 2f;
+                    RingModule newRing = new RingModule();
-                newRing.texture = body.Config.Ring.texture;
+                    newRing.innerRadius = size * 1.2f;
-                var ring = RingBuilder.Make(titleScreenGO, null, newRing, body.Mod);
+                    newRing.outerRadius = size * 2f;
+                    newRing.texture = ring.texture;
+                    RingBuilder.Make(titleScreenGO, null, newRing, body.Mod);
+                }
                 titleScreenGO.transform.localScale = Vector3.one * 0.8f;
             }
 

NewHorizons/Schemas/body_schema.json

@@ -100,9 +100,12 @@
         "type": "string"
       }
     },
-    "Ring": {
+    "Rings": {
-      "description": "Creates a ring around the planet",
+      "type": "array",
-      "$ref": "#/definitions/RingModule"
+      "description": "Create rings around the planet",
+      "items": {
+        "$ref": "#/definitions/RingModule"
+      }
     },
     "Sand": {
       "description": "Add sand to this planet",
@@ -2062,6 +2065,10 @@
           "items": {
             "$ref": "#/definitions/TimeValuePair"
           }
+        },
+        "rename": {
+          "type": "string",
+          "description": "An optional rename of this object"
         }
       }
     },