fix: NetworkConfig exclude registered scenes list from hash if scene managment is disabled

Luke Stampfli · Luke Stampfli · commit 3e7f96b9f2a5 · 2020-09-25T09:59:25.000+01:00
Currently if 'Enable Scene Management' is set to false MLAPI prints a NetworkConfiguration
mismatch warning on connection if the server and client have a different registered scenes list.
diff --git a/MLAPI/Configuration/NetworkConfig.cs b/MLAPI/Configuration/NetworkConfig.cs
@@ -344,7 +344,7 @@ public ulong GetConfig(bool cache = true)
                     writer.WriteUInt16Packed(ProtocolVersion);
                     writer.WriteString(MLAPIConstants.MLAPI_PROTOCOL_VERSION);
 
-                    if (!AllowRuntimeSceneChanges)
+                    if (EnableSceneManagement && !AllowRuntimeSceneChanges)
                     {
                         for (int i = 0; i < RegisteredScenes.Count; i++)
                         {

Original file line number	Diff line number	Diff line change
`@@ -344,7 +344,7 @@ public ulong GetConfig(bool cache = true)`
`344`	`344`	`writer.WriteUInt16Packed(ProtocolVersion);`
`345`	`345`	`writer.WriteString(MLAPIConstants.MLAPI_PROTOCOL_VERSION);`
`346`	`346`
`347`		`- if (!AllowRuntimeSceneChanges)`
	`347`	`+ if (EnableSceneManagement && !AllowRuntimeSceneChanges)`
`348`	`348`	`{`
`349`	`349`	`for (int i = 0; i < RegisteredScenes.Count; i++)`
`350`	`350`	`{`