Merge pull request #163 from FFXIV-CombatReborn/fix-aoe-toggle

Improve error handling and value parsing in RSCommands_OtherCommand.cs

.gitignore

@@ -1,4 +1,5 @@
 .vs/
 bin/
 obj/
-lib/
+lib/
+.idea/

RotationSolver/Commands/RSCommands_OtherCommand.cs

@@ -42,7 +42,6 @@ private static void DoSettingCommand(string str)
         var strs = str.Split(' ');
         var settingName = strs[0];
         var value = strs.LastOrDefault();
-
         foreach (var property in typeof(Configs).GetRuntimeProperties().Where(p => p.GetMethod?.IsPublic ?? false))
         {
             if (!settingName.Equals(property.Name, StringComparison.OrdinalIgnoreCase))
@@ -53,29 +52,45 @@ private static void DoSettingCommand(string str)
                 type = typeof(bool);
 
             object convertedValue = null;
-            try
+            bool valueParsedSuccessfully = true;
+
+            if (type.IsEnum)
             {
-                if (type.IsEnum)
+                valueParsedSuccessfully = Enum.TryParse(type, value, ignoreCase: true, out var parsedEnum);
+                if (valueParsedSuccessfully)
                 {
-                    convertedValue = Enum.Parse(type, value, ignoreCase: true);
+                    convertedValue = parsedEnum;
                 }
-                else
+            }
+            else
+            {
+                try
                 {
                     convertedValue = Convert.ChangeType(value, type);
                 }
+                catch
+                {
+                    valueParsedSuccessfully = false;
+                }
             }
-            catch
+
+            if (!valueParsedSuccessfully)
             {
                 if (type == typeof(bool))
                 {
-                    // Toggle the boolean value if no value is specified
                     var config = property.GetValue(Service.Config) as ConditionBoolean;
                     if (config != null)
                     {
                         config.Value = !config.Value;
                         convertedValue = config.Value;
                     }
                 }
+                else if (type.IsEnum)
+                {
+                    // If invalid enum value provided - increment to the next enum value
+                    var currentEnumValue = (Enum)property.GetValue(Service.Config);
+                    convertedValue = GetNextEnumValue(currentEnumValue);
+                }
             }
 
             if (convertedValue == null)
@@ -86,7 +101,6 @@ private static void DoSettingCommand(string str)
                 continue;
             }
 
-            // If it's a ConditionBoolean, handle it specifically
             if (property.PropertyType == typeof(ConditionBoolean))
             {
                 var relay = (ConditionBoolean)property.GetValue(Service.Config)!;
@@ -97,14 +111,22 @@ private static void DoSettingCommand(string str)
             property.SetValue(Service.Config, convertedValue);
             value = convertedValue.ToString();
 
-            // Notify the user of the change
             Svc.Chat.Print(string.Format(UiString.CommandsChangeSettingsValue.Local(), property.Name, value));
+
             return;
         }
 
         Svc.Chat.PrintError(UiString.CommandsCannotFindConfig.Local());
     }
 
+    private static Enum GetNextEnumValue(Enum currentEnumValue)
+    {
+        var enumValues = Enum.GetValues(currentEnumValue.GetType()).Cast<Enum>().ToArray();
+        var nextIndex = Array.IndexOf(enumValues, currentEnumValue) + 1;
+
+        return enumValues.Length == nextIndex ? enumValues[0] : enumValues[nextIndex];
+    }
+
     private static void ToggleActionCommand(string str)
     {
         foreach (var act in RotationUpdater.RightRotationActions)
@@ -172,4 +194,4 @@ private static void DoRotationCommand(ICustomRotation customCombo, string str)
 
         Svc.Chat.PrintError(UiString.CommandsInsertActionFailure.Local());
     }
-}
+}