fix: Take appropriate actions when presence enabled state changes

Author: kirre-bylund

Runtime/Client/LootLockerPresenceManager.cs

@@ -497,11 +497,9 @@ public static bool IsEnabled
             set 
             {
                 var instance = Get();
-                if (!value && instance.isEnabled)
-                {
-                    DisconnectAll();
-                }
-                instance.isEnabled = value;
+                if(!instance)
+                    return;
+                instance.SetPresenceEnabled(value);
             }
         }
 
@@ -511,7 +509,7 @@ public static bool IsEnabled
         public static bool AutoConnectEnabled
         {
             get => Get()?.autoConnectEnabled ?? false;
-            set { var instance = Get(); if (instance != null) instance.autoConnectEnabled = value; }
+            set { var instance = Get(); if (instance != null) instance.SetAutoConnectEnabled(value); }
         }
 
         /// <summary>
@@ -546,25 +544,6 @@ public static IEnumerable<string> ActiveClientUlids
 
         #region Public Methods
 
-        /// <summary>
-        /// Connect presence using player data directly (used by event handlers to avoid StateData lookup issues)
-        /// </summary>
-        private static void ConnectPresenceWithPlayerData(LootLockerPlayerData playerData, LootLockerPresenceCallback onComplete = null)
-        {
-            var instance = Get();
-            
-            // Create and initialize the client
-            var client = instance.CreateAndInitializePresenceClient(playerData);
-            if (client == null)
-            {
-                onComplete?.Invoke(false, "Failed to create or initialize presence client");
-                return;
-            }
-
-            // Connect the client
-            instance.ConnectExistingPresenceClient(playerData.ULID, client, onComplete);
-        }
-
         /// <summary>
         /// Connect presence for a specific player session
         /// </summary>
@@ -970,6 +949,38 @@ public static string GetLastSentStatus(string playerUlid = null)
 
         #region Private Helper Methods
 
+        private void SetPresenceEnabled(bool enabled)
+        {
+            bool changingState = isEnabled != enabled;
+            isEnabled = enabled;
+            if(changingState && enabled && autoConnectEnabled)
+            {
+                SubscribeToSessionEvents();
+                StartCoroutine(AutoConnectExistingSessions());
+            } 
+            else if (changingState && !enabled)
+            {
+                UnsubscribeFromSessionEvents();
+                DisconnectAllInternal();
+            }
+        }
+
+        private void SetAutoConnectEnabled(bool enabled)
+        {
+            bool changingState = autoConnectEnabled != enabled;
+            autoConnectEnabled = enabled;
+            if(changingState && enabled)
+            {
+                SubscribeToSessionEvents();
+                StartCoroutine(AutoConnectExistingSessions());
+            } 
+            else if (changingState && !enabled)
+            {
+                UnsubscribeFromSessionEvents();
+                DisconnectAllInternal();
+            }
+        }
+
         /// <summary>
         /// Handle client state changes for automatic cleanup
         /// </summary>