Merge pull request #1097 from PlayEveryWare/fix/auto-select-options-for-new-platforms

fix: Auto select deployment and client credentials for newly introduced platforms

Author: paulhazen

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

@@ -235,12 +235,7 @@ protected override void RenderWindow()
 
             if (_platformTabs != null && _platformConfigEditors.Count != 0)
             {
-                var newlySelectedTabIndex = GUILayout.Toolbar(_selectedTab, _platformTabs, TAB_STYLE);
-
-                if (newlySelectedTabIndex != _selectedTab)
-                {
-                    _selectedTab = newlySelectedTabIndex;
-                }
+                _selectedTab = GUILayout.Toolbar(_selectedTab, _platformTabs, TAB_STYLE);
 
                 GUILayout.Space(30);
 

com.playeveryware.eos/Runtime/Core/Config/PlatformConfig.cs

@@ -270,6 +270,44 @@ protected override void OnReadCompleted()
         {
             base.OnReadCompleted();
 
+            // If the deployment and client credentials are complete, there is
+            // nothing to do.
+            if (deployment.IsComplete && clientCredentials is { IsComplete: true })
+            {
+                return;
+            }
+
+            ProductConfig productConfig = Get<ProductConfig>();
+            bool valuesImported = false;
+
+            if (!deployment.IsComplete && 
+                productConfig.Environments.TryGetFirstDefinedNamedDeployment(out Named<Deployment> namedDeployment))
+            {
+                deployment = namedDeployment.Value;
+                Debug.Log($"Platform {Platform} has no defined deployment, " +
+                          $"so one was selected: {namedDeployment}.");
+                valuesImported = true;
+            }
+
+            if (clientCredentials is not { IsComplete: true } &&
+                productConfig.TryGetFirstCompleteNamedClientCredentials(
+                    out Named<EOSClientCredentials> namedCredentials))
+            {
+                clientCredentials = namedCredentials.Value;
+                Debug.Log($"Platform {Platform} has no defined client " +
+                          $"credentials, so one was selected: " +
+                          $"{namedCredentials}.");
+                valuesImported = true;
+            }
+
+            // This compile conditional is here because writing configs to disk
+            // is only allowed within the context of the unity editor.
+#if UNITY_EDITOR
+            if (valuesImported)
+            {
+                Write();
+            }
+#endif
             // If thread affinity is null then instantiate it.
             threadAffinity ??= new();
         }