Copy and Delete root Node

Author: sarbian

Properties/AssemblyInfo.cs

@@ -17,7 +17,7 @@
 // The form "{Major}.{Minor}.*" will automatically update the build and revision,
 // and "{Major}.{Minor}.{Build}.*" will update just the revision.
 
-[assembly: AssemblyVersion("2.0.7")]
+[assembly: AssemblyVersion("2.0.8")]
 
 // The following attributes are used to specify the signing key for the assembly, 
 // if desired. See the Mono documentation for more information about signing.

moduleManager.cs

@@ -480,7 +480,7 @@ public void Update()
 
         static string status = "Processing Module Manager patch\nPlease Wait...";
         static string errors = "";
-        
+
         public void OnGUI()
         {
             if (HighLogic.LoadedScene != GameScenes.LOADING) 
@@ -508,7 +508,7 @@ public void OnGUI()
         public void ApplyPatch(List<String> excludePaths, string Stage)
         {
             print("[ModuleManager] " + Stage + (Stage == ":FIRST" ? " (default) pass" : " pass"));
-            foreach (UrlDir.UrlConfig mod in GameDatabase.Instance.root.AllConfigs)
+            foreach (UrlDir.UrlConfig mod in GameDatabase.Instance.root.AllConfigs.ToArray())
             {
                 if (mod.type[0] == '@' || (mod.type[0] == '$') || (mod.type[0] == '!'))
                 {
@@ -585,7 +585,7 @@ public void ApplyPatch(List<String> excludePaths, string Stage)
                                 continue;
                             }
 
-                            foreach (UrlDir.UrlConfig url in GameDatabase.Instance.root.AllConfigs)
+                            foreach (UrlDir.UrlConfig url in GameDatabase.Instance.root.AllConfigs.ToArray())
                             {
                                 if (url.type == type
                                     && WildcardMatch(url.name, pattern)
@@ -601,25 +601,22 @@ public void ApplyPatch(List<String> excludePaths, string Stage)
                                     }
                                     else if (mod.type[0] == '$')
                                     {
-                                        // Here we would duplicate an Node if it did not create an exception since we modfiy an enum while it is used                                       
-                                        
-                                        //ConfigNode clone = ConfigManager.ModifyNode(url.config, mod.config);
-                                        //if (url.config.name != mod.name)
-                                        //{
-                                        //    print("[ModuleManager] Copying Node " + url.config.name + " into " + clone.name);
-                                        //    url.parent.configs.Add(new UrlDir.UrlConfig(url.parent, clone));
-                                        //}
-                                        //else
-                                        //{                                            
-                                        //    errorCount++;
-                                        //    print("[ModuleManager] Error while processing " + mod.config.name + " the copy need to have a different name than the parent (use @name = xxx)");
-                                        //}
+                                        ConfigNode clone = ConfigManager.ModifyNode(url.config, mod.config);
+                                        if (url.config.name != mod.name)
+                                        {
+                                            print("[ModuleManager] Copying Node " + url.config.name + " into " + clone.name);
+                                            url.parent.configs.Add(new UrlDir.UrlConfig(url.parent, clone));
+                                        }
+                                        else
+                                        {                                            
+                                            errorCount++;
+                                            print("[ModuleManager] Error while processing " + mod.config.name + " the copy needs to have a different name than the parent (use @name = xxx)");
+                                        }
                                     }
                                     else if (mod.type[0] == '!')
                                     {
-                                        // Same problem
-                                        //print("[ModuleManager] Deleting Node " + url.config.name);
-                                        //url.parent.configs.Remove(url);
+                                        print("[ModuleManager] Deleting Node " + url.config.name);
+                                        url.parent.configs.Remove(url);
                                     }
                                 }
                             }