update

Updating the logic behind the list of client identifiers that should receive the ChangeOwnershipMessage when running a distributed authority network topology.

Author: NoelStephensUnity

com.unity.netcode.gameobjects/Runtime/Spawning/NetworkSpawnManager.cs

@@ -588,8 +588,8 @@ internal void ChangeOwnership(NetworkObject networkObject, ulong clientId, bool
                         networkObject.ChildNetworkBehaviours[i].UpdateNetworkProperties();
                     }
 
-                    // DANGO-TODO: CMB Service needs to be updated to check for these two already existing properties.
-                    message.ClientIds = NetworkManager.ConnectedClientsIds.Where((c) => c != NetworkManager.ServerClientId || !IsObjectVisibilityPending(c, ref networkObject)).ToArray();
+                    // Populate valid target client identifiers that should receive this change in ownership message.
+                    message.ClientIds = NetworkManager.ConnectedClientsIds.Where((c) => !IsObjectVisibilityPending(c, ref networkObject) && networkObject.IsNetworkVisibleTo(c)).ToArray();
                     message.ClientIdCount = message.ClientIds.Length;
 
                     size = NetworkManager.ConnectionManager.SendMessage(ref message, NetworkDelivery.ReliableSequenced, NetworkManager.ServerClientId);