refactor: using interface instead of network client (#721)

BREAKING CHANGE: fields and parameters using NetworkClient are now using INetworkClient Instead

Assets/Mirage/Runtime/INetworkClient.cs

@@ -36,6 +36,8 @@ public interface INetworkClient : IMessageSender
         /// </summary>
         bool IsLocalClient { get; }
 
+        NetworkTime Time { get; }
+
         void Disconnect();
     }
 }

Assets/Mirage/Runtime/INetworkServer.cs

@@ -45,7 +45,7 @@ public interface INetworkServer
         /// <summary>
         /// The host client for this server 
         /// </summary> 
-        NetworkClient LocalClient { get; }
+        INetworkClient LocalClient { get; }
 
         /// <summary>
         /// True if there is a local client connected to this server (host mode)

Assets/Mirage/Runtime/IServerObjectManager.cs

@@ -20,9 +20,9 @@ public interface IServerObjectManager
 
         bool AddCharacter(INetworkPlayer player, GameObject character, Guid assetId);
 
-        bool ReplaceCharacter(INetworkPlayer player, NetworkClient client, GameObject character, bool keepAuthority = false);
+        bool ReplaceCharacter(INetworkPlayer player, INetworkClient client, GameObject character, bool keepAuthority = false);
 
-        bool ReplaceCharacter(INetworkPlayer player, NetworkClient client, GameObject character, Guid assetId, bool keepAuthority = false);
+        bool ReplaceCharacter(INetworkPlayer player, INetworkClient client, GameObject character, Guid assetId, bool keepAuthority = false);
 
         void Spawn(GameObject obj, GameObject owner);
 

Assets/Mirage/Runtime/NetworkBehaviour.cs

@@ -102,7 +102,7 @@ public abstract class NetworkBehaviour : MonoBehaviour
         /// <summary>
         /// The <see cref="NetworkClient">NetworkClient</see> associated to this object.
         /// </summary>
-        public NetworkClient Client => NetIdentity.Client;
+        public INetworkClient Client => NetIdentity.Client;
 
         /// <summary>
         /// Quick Reference to the NetworkIdentities ClientObjectManager. Present only for instances instances.

Assets/Mirage/Runtime/NetworkIdentity.cs

@@ -175,7 +175,7 @@ public sealed class NetworkIdentity : MonoBehaviour
         /// <summary>
         /// The NetworkClient associated with this NetworkIdentity.
         /// </summary>
-        public NetworkClient Client { get; internal set; }
+        public INetworkClient Client { get; internal set; }
 
         /// <summary>
         /// The ClientObjectManager is present only for client instances.

Assets/Mirage/Runtime/NetworkServer.cs

@@ -100,7 +100,7 @@ public class NetworkServer : MonoBehaviour, INetworkServer
         /// <summary>
         /// The host client for this server 
         /// </summary>
-        public NetworkClient LocalClient { get; private set; }
+        public INetworkClient LocalClient { get; private set; }
 
         /// <summary>
         /// True if there is a local client connected to this server (host mode)
@@ -339,7 +339,7 @@ public void RemoveConnection(INetworkPlayer player)
         /// </summary>
         /// <param name="client">The local client</param>
         /// <param name="tconn">The connection to the client</param>
-        internal void SetLocalConnection(NetworkClient client, IConnection tconn)
+        internal void SetLocalConnection(INetworkClient client, IConnection tconn)
         {
             if (LocalPlayer != null)
             {

Assets/Mirage/Runtime/NetworkTime.cs

@@ -53,7 +53,7 @@ public void Reset()
             offsetMax = double.MaxValue;
         }
 
-        internal void UpdateClient(NetworkClient client)
+        internal void UpdateClient(INetworkClient client)
         {
             if (UnityEngine.Time.time - lastPingTime >= PingFrequency)
             {

Assets/Mirage/Runtime/ServerObjectManager.cs

@@ -198,7 +198,7 @@ void StartedHost()
         /// <param name="assetId"></param>
         /// <param name="keepAuthority">Does the previous player remain attached to this connection?</param>
         /// <returns></returns>
-        public bool ReplaceCharacter(INetworkPlayer player, NetworkClient client, GameObject character, Guid assetId, bool keepAuthority = false)
+        public bool ReplaceCharacter(INetworkPlayer player, INetworkClient client, GameObject character, Guid assetId, bool keepAuthority = false)
         {
             NetworkIdentity identity = character.GetNetworkIdentity();
             identity.AssetId = assetId;
@@ -214,7 +214,7 @@ public bool ReplaceCharacter(INetworkPlayer player, NetworkClient client, GameOb
         /// <param name="character">Player object spawned for the player.</param>
         /// <param name="keepAuthority">Does the previous player remain attached to this connection?</param>
         /// <returns></returns>
-        public bool ReplaceCharacter(INetworkPlayer player, NetworkClient client, GameObject character, bool keepAuthority = false)
+        public bool ReplaceCharacter(INetworkPlayer player, INetworkClient client, GameObject character, bool keepAuthority = false)
         {
             return InternalReplacePlayerForConnection(player, client, character, keepAuthority);
         }
@@ -329,7 +329,7 @@ void Respawn(NetworkIdentity identity)
             }
         }
 
-        internal bool InternalReplacePlayerForConnection(INetworkPlayer player, NetworkClient client, GameObject character, bool keepAuthority)
+        internal bool InternalReplacePlayerForConnection(INetworkPlayer player, INetworkClient client, GameObject character, bool keepAuthority)
         {
             NetworkIdentity identity = character.GetComponent<NetworkIdentity>();
             if (identity is null)