feat: Use Enum flag field for editor window

Author: WispyMouse

Assets/Plugins/Source/Editor/EditorWindows/EOSSettingsWindow.cs

@@ -25,6 +25,7 @@
 namespace PlayEveryWare.EpicOnlineServices.Editor.Windows
 {
     using Epic.OnlineServices.UI;
+    using PlayEveryWare.EpicOnlineServices.Extensions;
     using System;
     using System.Collections.Generic;
     using System.IO;
@@ -349,9 +350,11 @@ private void OnDefaultGUI()
                 ref mainEOSConfigFile.alwaysSendInputToOverlay, 190,
                 "If true, the plugin will always send input to the overlay from the C# side to native, and handle showing the overlay. This doesn't always mean input makes it to the EOS SDK.");
 
-            GUIEditorUtility.AssigningFlagTextField("Default Activate Overlay Button",
-                ref mainEOSConfigFile.toggleFriendsButtonCombination, 190,
+            InputStateButtonFlags toggleFriendsButtonCombinationEnum = mainEOSConfigFile.GetToggleFriendsButtonCombinationFlags();
+            GUIEditorUtility.AssigningEnumField<InputStateButtonFlags>("Default Activate Overlay Button",
+                ref toggleFriendsButtonCombinationEnum, 190,
                 "Users can press the button(s) associated with this value to activate the Epic Social Overlay. Not all combinations are valid; the SDK will log an error at the start of runtime if an invalid combination is selected.");
+            mainEOSConfigFile.toggleFriendsButtonCombination = toggleFriendsButtonCombinationEnum.FlagsToStrings();
         }
 
         protected override void RenderWindow()

com.playeveryware.eos/Runtime/Core/EOS_SDK_Additions/Extensions/InputStateButtonFlagsExtensions.cs

@@ -24,7 +24,6 @@
 
 namespace PlayEveryWare.EpicOnlineServices.Extensions
 {
-    using Epic.OnlineServices.Auth;
     using Utility;
     using System;
     using System.Collections.Generic;
@@ -61,11 +60,11 @@ public static class InputStateButtonFlagsExtensions
 
         /// <summary>
         /// Tries to parse the given list of strings representing individual
-        /// AuthScopeFlags enum values, and performing a
+        /// InputStateButtonFlags enum values, and performing a
         /// bitwise OR operation on those values.
         /// </summary>
         /// <param name="stringFlags">
-        /// List of strings representing individual AuthScopeFlags enum values.
+        /// List of strings representing individual InputStateButtonFlags enum values.
         /// </param>
         /// <param name="result">
         /// The result of performing a bitwise OR operation on the values that
@@ -79,6 +78,53 @@ public static bool TryParse(IList<string> stringFlags, out InputStateButtonFlags
         {
             return EnumUtility<InputStateButtonFlags>.TryParse(stringFlags, CustomMappings, out result);
         }
+
+        /// <summary>
+        /// Parses an InputStateButtonFlags value in to its component strings.
+        /// These strings can then be safely stored in a configuration value.
+        /// </summary>
+        /// <param name="flags">Enum to separate into parts.</param>
+        /// <returns>A list of strings representing these flags.</returns>
+        public static List<string> FlagsToStrings(this InputStateButtonFlags flags)
+        {
+            List<string> flagStrings = new List<string>();
+
+            // Iterate over all possible flag values
+            foreach (InputStateButtonFlags flagOption in Enum.GetValues(typeof(InputStateButtonFlags)))
+            {
+                // If the passed in Enum doesn't have this flag, skip it
+                if (!flags.HasFlag(flagOption))
+                {
+                    continue;
+                }
+
+                // Look up this flag's name in the custom mappings dictionary
+                // The dictionary is keyed by string value and valued by enum,
+                // so it must be entirely traversed to find its matching string
+                bool foundMapping = false;
+                foreach (KeyValuePair<string, InputStateButtonFlags> mapping in CustomMappings)
+                {
+                    // Continue if this isn't the flag that matches the current option
+                    if (mapping.Value != flagOption)
+                    {
+                        continue;
+                    }
+
+                    // This is the value needed, so use it
+                    flagStrings.Add(mapping.Key);
+                    foundMapping = true;
+                    continue;
+                }
+
+                // If no custom mapping was found, use ToString instead
+                if (!foundMapping)
+                {
+                    flagStrings.Add(flagOption.ToString());
+                }
+            }
+
+            return flagStrings;
+        }
     }
 }