feat!: SpawnVisibleObjects now calls RemoveAllVisibleObjects

calling RemoveAllVisibleObjects means that Mirage will send spawn message for objects destroyed in scene change.

BREAKING CHANGE: SpawnVisibleObjects now calls RemoveAllVisibleObjects before spawning objects

Author: James-Frowen

Assets/Mirage/Runtime/ServerObjectManager.cs

@@ -739,7 +739,8 @@ public void SpawnVisibleObjects(INetworkPlayer player, bool ignoreHasCharacter,
         /// <param name="skip">NetworkIdentity to skip when spawning. Can be null</param>
         public void SpawnVisibleObjects(INetworkPlayer player, bool ignoreHasCharacter, HashSet<NetworkIdentity> skip)
         {
-            // todo Call player.RemoveAllVisibleObjects() first so that it will send spawn message for objects destroyed in scene change
+            // remove all, so that it will send spawn message for objects destroyed in scene change
+            player.RemoveAllVisibleObjects();
 
             if (!ignoreHasCharacter && !player.HasCharacter)
             {
@@ -768,12 +769,12 @@ public void SpawnVisibleObjects(INetworkPlayer player, bool ignoreHasCharacter,
                 if (skip != null && skip.Contains(identity))
                     continue;
 
-                    if (logger.LogEnabled()) logger.Log($"Checking Observers on server objects name='{identity.name}' netId={identity.NetId} sceneId={identity.SceneId:X}");
+                if (logger.LogEnabled()) logger.Log($"Checking Observers on server objects name='{identity.name}' netId={identity.NetId} sceneId={identity.SceneId:X}");
 
-                    var visible = identity.OnCheckObserver(player);
-                    if (visible)
-                    {
-                        identity.AddObserver(player);
+                var visible = identity.OnCheckObserver(player);
+                if (visible)
+                {
+                    identity.AddObserver(player);
                 }
             }