Merge branch 'develop' into singleton-removal-network-object

Author: TwoTenPvP

.yamato/_triggers.yml

@@ -1,6 +1,5 @@
 {% metadata_file .yamato/project.metafile %}
 ---
-
 develop_nightly_trigger:
   name: Develop Branch Nightly Trigger
   triggers:
@@ -20,6 +19,7 @@ develop_nightly_trigger:
 pull_request_trigger:
   name: Pull Request Trigger on {{ test_editors.first }} (master, develop, & release branches)
   dependencies:
+    - .yamato/project-standards.yml#standards_{{ projects.first.name }}
 {% for project in projects -%}
 {% for package in project.packages -%}
     - .yamato/project-tests.yml#validate_{{ package.name }}_{{ test_platforms.first.name }}_{{ test_editors.first }}

.yamato/project-standards.yml

@@ -0,0 +1,16 @@
+{% metadata_file .yamato/project.metafile %}
+---
+standards_{{ projects.first.name }}:
+  name: Standards Check {{ projects.first.name }}
+  agent:
+    type: Unity::VM
+    image: desktop/logging-testing-linux:stable
+    flavor: b1.small
+  commands:
+    - dotnet --version
+    - $HOME/.dotnet/tools/dotnet-format --version
+    - python3 --version
+    - pip install unity-downloader-cli --upgrade --index-url https://artifactory.prd.it.unity3d.com/artifactory/api/pypi/pypi/simple
+    - unity-downloader-cli -u {{ test_editors.first }} -c editor --wait --fast
+    - .Editor/Unity -batchmode -nographics -logFile - -executeMethod Packages.Rider.Editor.RiderScriptEditor.SyncSolution -projectPath {{ projects.first.path }} -quit
+    - ./standards.py --tool-path $HOME/.dotnet/tools/dotnet-format --project-path {{ projects.first.path }} --yamato

com.unity.multiplayer.mlapi/Editor/NetworkManagerEditor.cs

@@ -36,7 +36,7 @@ public class NetworkManagerEditor : UnityEditor.Editor
         private SerializedProperty m_TimeResyncIntervalProperty;
         private SerializedProperty m_EnableNetworkVariableProperty;
         private SerializedProperty m_EnsureNetworkVariableLengthSafetyProperty;
-        private SerializedProperty m_ForceSamePrefabsProperty;        
+        private SerializedProperty m_ForceSamePrefabsProperty;
         private SerializedProperty m_EnableSceneManagementProperty;
         private SerializedProperty m_RecycleNetworkIdsProperty;
         private SerializedProperty m_NetworkIdRecycleDelayProperty;
@@ -112,7 +112,7 @@ private void Initialize()
             m_EnableTimeResyncProperty = m_NetworkConfigProperty.FindPropertyRelative("EnableTimeResync");
             m_TimeResyncIntervalProperty = m_NetworkConfigProperty.FindPropertyRelative("TimeResyncInterval");
             m_EnableNetworkVariableProperty = m_NetworkConfigProperty.FindPropertyRelative("EnableNetworkVariable");
-            m_EnsureNetworkVariableLengthSafetyProperty = m_NetworkConfigProperty.FindPropertyRelative("EnsureNetworkVariableLengthSafety");            
+            m_EnsureNetworkVariableLengthSafetyProperty = m_NetworkConfigProperty.FindPropertyRelative("EnsureNetworkVariableLengthSafety");
             m_ForceSamePrefabsProperty = m_NetworkConfigProperty.FindPropertyRelative("ForceSamePrefabs");
             m_EnableSceneManagementProperty = m_NetworkConfigProperty.FindPropertyRelative("EnableSceneManagement");
             m_RecycleNetworkIdsProperty = m_NetworkConfigProperty.FindPropertyRelative("RecycleNetworkIds");
@@ -133,7 +133,7 @@ private void CheckNullProperties()
             m_RunInBackgroundProperty = serializedObject.FindProperty(nameof(NetworkManager.RunInBackground));
             m_LogLevelProperty = serializedObject.FindProperty(nameof(NetworkManager.LogLevel));
             m_NetworkConfigProperty = serializedObject.FindProperty(nameof(NetworkManager.NetworkConfig));
-            
+
             // NetworkConfig properties
             m_PlayerPrefabProperty = m_NetworkConfigProperty.FindPropertyRelative(nameof(NetworkConfig.PlayerPrefab));
             m_ProtocolVersionProperty = m_NetworkConfigProperty.FindPropertyRelative("ProtocolVersion");

com.unity.multiplayer.mlapi/Runtime/Core/NetworkBehaviour.cs

@@ -385,8 +385,6 @@ public virtual void OnGainedOwnership() { }
         /// </summary>
         public virtual void OnLostOwnership() { }
 
-        #region NetworkVariable
-
         private bool m_VarInit = false;
 
         private readonly List<HashSet<int>> m_ChannelMappedNetworkVariableIndexes = new List<HashSet<int>>();
@@ -999,8 +997,6 @@ internal static void SetNetworkVariableData(List<INetworkVariable> networkVariab
             }
         }
 
-        #endregion
-
         /// <summary>
         /// Gets the local instance of a object with a given NetworkId
         /// </summary>

com.unity.multiplayer.mlapi/Runtime/Core/NetworkManager.cs

@@ -1154,8 +1154,6 @@ internal void HandleIncomingData(ulong clientId, NetworkChannel networkChannel,
                     return;
                 }
 
-                #region INTERNAL MESSAGE
-
                 switch (messageType)
                 {
                     case NetworkConstants.CONNECTION_REQUEST:
@@ -1312,8 +1310,6 @@ internal void HandleIncomingData(ulong clientId, NetworkChannel networkChannel,
                         break;
                 }
 
-                #endregion
-
 #if !UNITY_2020_2_OR_NEWER
                 NetworkProfiler.EndEvent();
 #endif

com.unity.multiplayer.mlapi/Runtime/Messaging/CustomMessageManager.cs

@@ -23,8 +23,6 @@ internal CustomMessagingManager(NetworkManager networkManager)
             m_NetworkManager = networkManager;
         }
 
-        #region Unnamed
-
         /// <summary>
         /// Delegate used for incoming unnamed messages
         /// </summary>
@@ -73,10 +71,6 @@ public void SendUnnamedMessage(ulong clientId, NetworkBuffer buffer, NetworkChan
             PerformanceDataManager.Increment(ProfilerConstants.UnnamedMessageSent);
         }
 
-        #endregion
-
-        #region Named
-
         /// <summary>
         /// Delegate used to handle named messages
         /// </summary>
@@ -215,7 +209,5 @@ public void SendNamedMessage(string name, List<ulong> clientIds, Stream stream,
                 PerformanceDataManager.Increment(ProfilerConstants.NamedMessageSent);
             }
         }
-
-        #endregion
     }
 }

com.unity.multiplayer.mlapi/Runtime/Spawning/NetworkSpawnManager.cs

@@ -196,7 +196,7 @@ internal NetworkObject CreateLocalNetworkObject(bool softCreate, uint prefabHash
                 {
                     // See if there is a valid registered NetworkPrefabOverrideLink associated with the provided prefabHash
                     GameObject networkPrefabReference = null;
-                    if(NetworkManager.NetworkConfig.NetworkPrefabOverrideLinks.ContainsKey(prefabHash))
+                    if (NetworkManager.NetworkConfig.NetworkPrefabOverrideLinks.ContainsKey(prefabHash))
                     {
                         switch (NetworkManager.NetworkConfig.NetworkPrefabOverrideLinks[prefabHash].Override)
                         {

com.unity.multiplayer.mlapi/Tests/Editor/NetworkManagerMessageSenderTests.cs

@@ -21,7 +21,7 @@ public void MessageSenderAssigned()
             networkManager.NetworkConfig = new NetworkConfig()
             {
                 // Set the current scene to prevent unexpected log messages which would trigger a failure
-                RegisteredScenes = new List<string>() {SceneManager.GetActiveScene().name}
+                RegisteredScenes = new List<string>() { SceneManager.GetActiveScene().name }
             };
 
             // Set dummy transport that does nothing

com.unity.multiplayer.mlapi/Tests/Editor/NetworkManagerSceneManagerTests.cs

@@ -21,7 +21,7 @@ public void SceneManagerAssigned()
             networkManager.NetworkConfig = new NetworkConfig()
             {
                 // Set the current scene to prevent unexpected log messages which would trigger a failure
-                RegisteredScenes = new List<string>() {SceneManager.GetActiveScene().name}
+                RegisteredScenes = new List<string>() { SceneManager.GetActiveScene().name }
             };
 
             // Set dummy transport that does nothing

com.unity.multiplayer.mlapi/Tests/Runtime/NetworkManagerHelper.cs

@@ -75,7 +75,7 @@ public static bool StartNetworkManager(out NetworkManager networkManager, Networ
                 NetworkManagerObject.NetworkConfig = new Configuration.NetworkConfig
                 {
                     EnableSceneManagement = false,
-                    RegisteredScenes = new List<string>(){SceneManager.GetActiveScene().name}
+                    RegisteredScenes = new List<string>() { SceneManager.GetActiveScene().name }
                 };
                 unetTransport.ConnectAddress = "127.0.0.1";
                 unetTransport.ConnectPort = 7777;