feat: adding event to NetworkPlayer when Identity is changed

MirageNet · Mar 31, 2022 · 9e22ff4 · 9e22ff4
1 parent de874ab
commit 9e22ff4
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/Assets/Mirage/Runtime/INetworkPlayer.cs b/Assets/Mirage/Runtime/INetworkPlayer.cs
@@ -49,6 +49,7 @@ public interface IVisibilityTracker
     /// </summary>
     public interface IObjectOwner
     {
+        event Action<NetworkIdentity> OnIdentityChanged;
         NetworkIdentity Identity { get; set; }
         bool HasCharacter { get; }
         void RemoveOwnedObject(NetworkIdentity networkIdentity);

diff --git a/Assets/Mirage/Runtime/NetworkPlayer.cs b/Assets/Mirage/Runtime/NetworkPlayer.cs
@@ -39,6 +39,11 @@ public sealed class NetworkPlayer : INetworkPlayer, IMessageSender
         /// </summary>
         bool isDisconnected = false;
 
+        /// <summary>
+        /// Backing field for <see cref="Identity"/>
+        /// </summary>
+        private NetworkIdentity _identity;
+
         /// <summary>
         /// Marks if this player has been accepted by a <see cref="NetworkAuthenticator"/>
         /// </summary>
@@ -93,10 +98,26 @@ public void MarkAsDisconnected()
             isDisconnected = true;
         }
 
+        /// <summary>
+        /// Event called when <see cref="Identity"/> property is changed
+        /// </summary>
+        public event Action<NetworkIdentity> OnIdentityChanged;
+
         /// <summary>
         /// The NetworkIdentity for this connection.
         /// </summary>
-        public NetworkIdentity Identity { get; set; }
+        public NetworkIdentity Identity
+        {
+            get => _identity;
+            set
+            {
+                if (_identity == value)
+                    return;
+
+                _identity = value;
+                OnIdentityChanged?.Invoke(_identity);
+            }
+        }
 
         /// <summary>
         /// A list of the NetworkIdentity objects owned by this connection. This list is read-only.