Merge 9efd8a9473 into 451c297a43

Hopefully adds additional support in preventing confusion.

Assets/Mirror/Components/NetworkTransform/NetworkTransformHybrid.cs

@@ -367,7 +367,7 @@ protected virtual void OnClientToServerDeltaSync(byte baselineTick, Vector3 posi
                 bufferSizeLimit,
                 new TransformSnapshot(
                 timestamp,         // arrival remote timestamp. NOT remote time.
-                Time.timeAsDouble,
+                NetworkTime.localTime, // Unity 2019 doesn't have Time.timeAsDouble yet
                 position,
                 rotation,
                 scale
@@ -711,7 +711,7 @@ protected virtual void OnServerToClientDeltaSync(byte baselineTick, Vector3 posi
                 bufferSizeLimit,
                 new TransformSnapshot(
                 timestamp,             // arrival remote timestamp. NOT remote time.
-                Time.timeAsDouble,
+                NetworkTime.localTime, // Unity 2019 doesn't have Time.timeAsDouble yet
                 position,
                 rotation,
                 scale

Assets/Mirror/Components/NetworkTransform/NetworkTransformReliable.cs

@@ -61,7 +61,7 @@ void LateUpdate()
             // instead.
             if (isServer || (IsClientWithAuthority && NetworkClient.ready))
             {
-                if (sendIntervalCounter == sendIntervalMultiplier && (!onlySyncOnChange || Changed(Construct())))
+                if (sendIntervalCounter >= sendIntervalMultiplier && (!onlySyncOnChange || Changed(Construct())))
                     SetDirty();
 
                 CheckLastSendTime();
@@ -129,7 +129,7 @@ protected virtual void CheckLastSendTime()
             // timeAsDouble not available in older Unity versions.
             if (AccurateInterval.Elapsed(NetworkTime.localTime, NetworkServer.sendInterval, ref lastSendIntervalTime))
             {
-                if (sendIntervalCounter == sendIntervalMultiplier)
+                if (sendIntervalCounter >= sendIntervalMultiplier)
                     sendIntervalCounter = 0;
                 sendIntervalCounter++;
             }

Assets/Mirror/Core/Attributes.cs

@@ -48,28 +48,28 @@ public class TargetRpcAttribute : Attribute
     }
 
     /// <summary>
-    /// Prevents clients from running this method.
-    /// <para>Prints a warning if a client tries to execute this method.</para>
+    /// Only an active server will run this method.
+    /// <para>Prints a warning if a client or in-active server tries to execute this method.</para>
     /// </summary>
     [AttributeUsage(AttributeTargets.Method)]
     public class ServerAttribute : Attribute {}
 
     /// <summary>
-    /// Prevents clients from running this method.
+    /// Only an active server will run this method.
     /// <para>No warning is thrown.</para>
     /// </summary>
     [AttributeUsage(AttributeTargets.Method)]
     public class ServerCallbackAttribute : Attribute {}
 
     /// <summary>
-    /// Prevents the server from running this method.
-    /// <para>Prints a warning if the server tries to execute this method.</para>
+    /// Only an active client will run this method.
+    /// <para>Prints a warning if the server or in-active client tries to execute this method.</para>
     /// </summary>
     [AttributeUsage(AttributeTargets.Method)]
     public class ClientAttribute : Attribute {}
 
     /// <summary>
-    /// Prevents the server from running this method.
+    /// Only an active client will run this method.
     /// <para>No warning is printed.</para>
     /// </summary>
     [AttributeUsage(AttributeTargets.Method)]