fix

Very similar to NetworkList (we might collapse the OnSpawned logic into NetworkVariableBase.

Author: NoelStephensUnity

com.unity.netcode.gameobjects/Runtime/NetworkVariable/Collections/NetworkList.cs

@@ -61,17 +61,17 @@ public NetworkList(IEnumerable<T> values = default,
         internal override void OnSpawned()
         {
             // If the NetworkList is:
+            // - On the spawn authority side.
             // - Dirty
             // - State updates can be sent:
             // -- The instance has write permissions.
             // -- The last sent time plus the max send time period is less than the current time.
             // - User script has modified the list during spawn.
-            // - This instance is on the spawn authority side.
             // When the NetworkObject is finished spawning (on the same frame), go ahead and reset
             // the dirty related properties and last sent time to prevent duplicate entries from
             // being sent (i.e. CreateObjectMessage will contain the changes so we don't need to
             // send a proceeding NetworkVariableDeltaMessage).
-            if (IsDirty() && CanSend() && m_NetworkObject.IsSpawnAuthority)
+            if (m_NetworkObject.IsSpawnAuthority && IsDirty() && CanWrite() && CanSend())
             {
                 UpdateLastSentTime();
                 ResetDirty();

com.unity.netcode.gameobjects/Runtime/NetworkVariable/NetworkVariable.cs

@@ -413,9 +413,18 @@ internal override void WriteFieldSynchronization(FastBufferWriter writer)
         /// </summary>
         internal override void OnSpawned()
         {
-            // Assure any changes made to this NetworkVariable during spawn or post-spawn are
-            // serialized with the CreateObjectMessage.
-            if (IsDirty() && CanSend())
+            // If the NetworkVariable is:
+            // - On the spawn authority side.
+            // - Dirty.
+            // - State updates can be sent:
+            // -- The instance has write permissions.
+            // -- The last sent time plus the max send time period is less than the current time.
+            // - User script has modified the list during spawn.
+            // When the NetworkObject is finished spawning (on the same frame), go ahead and reset
+            // the dirty related properties and last sent time to prevent duplicate updates from
+            // being sent (i.e. CreateObjectMessage will contain the changes so we don't need to
+            // send a proceeding NetworkVariableDeltaMessage).
+            if (m_NetworkObject.IsSpawnAuthority && IsDirty() && CanWrite() && CanSend())
             {
                 UpdateLastSentTime();
                 ResetDirty();