Fix loading detail from presets and make backwards compatible (#6)

Author: Halbann

Source/TexturesUnlimited/Addon/TexturesUnlimitedLoader.cs

@@ -797,7 +797,7 @@ public RecoloringDataPreset(ConfigNode node)
 
         public RecoloringData getRecoloringData()
         {
-            return new RecoloringData(color, specular, metallic, 1);
+            return new RecoloringData(color, specular, metallic, detail);
         }
     }
 

Source/TexturesUnlimited/GUI/CraftRecolorGUI.cs

@@ -453,7 +453,7 @@ private void drawPresetColorArea()
                     bStr = (editingColor.color.b * 255f).ToString("F0");
                     aStr = (editingColor.specular * 255f).ToString("F0");
                     mStr = (editingColor.metallic * 255f).ToString("F0");
-                    //dStr = (editingColor.detail * 100f).ToString("F0");//leave detail mult as pre-specified value (user/config); it does not pull from preset colors at all
+                    dStr = (editingColor.detail * 100f).ToString("F0");
                     update = true;
                 }
                 GUI.color = old;

Source/TexturesUnlimited/Util/Utils.cs

@@ -486,6 +486,13 @@ public static Color GetColor(this ConfigNode node, string name)
         public static float GetColorChannelValue(this ConfigNode node, string name)
         {
             string value = node.GetStringValue(name).Trim();
+
+            // Detail was not included in presets originally, so on old presets GetStringValue will return "", which will default to 0.
+            // Default to 1 instead if trying to access detail, which results in a default detail of 100, as before.
+
+            if (value == "" && name == "detail")
+                return 1f;
+
             float floatValue = safeParseFloat(value);
             if (value.Contains(".")) { return floatValue; }
             if (name == "detail") { return floatValue / 100; }