Added out of bounds checks to GetBehaviourAtOrderIndex

TwoTenPvP · TwoTenPvP · commit 003377ba215d · 2018-10-10T14:14:02.000+02:00
diff --git a/MLAPI/MonoBehaviours/Core/NetworkedObject.cs b/MLAPI/MonoBehaviours/Core/NetworkedObject.cs
@@ -247,7 +247,11 @@ internal ushort GetOrderIndex(NetworkedBehaviour instance)
 
         internal NetworkedBehaviour GetBehaviourAtOrderIndex(ushort index)
         {
-            //TODO index out of bounds
+            if (index >= childNetworkedBehaviours.Count)
+            {
+                if (LogHelper.CurrentLogLevel <= LogLevel.Error) LogHelper.LogError("Behaviour index was out of bounds. Did you mess up the order of your NetworkedBehaviours?");
+                return null;
+            }
             return childNetworkedBehaviours[index];
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -247,7 +247,11 @@ internal ushort GetOrderIndex(NetworkedBehaviour instance)`
`247`	`247`
`248`	`248`	`internal NetworkedBehaviour GetBehaviourAtOrderIndex(ushort index)`
`249`	`249`	`{`
`250`		`- //TODO index out of bounds`
	`250`	`+ if (index >= childNetworkedBehaviours.Count)`
	`251`	`+ {`
	`252`	`+ if (LogHelper.CurrentLogLevel <= LogLevel.Error) LogHelper.LogError("Behaviour index was out of bounds. Did you mess up the order of your NetworkedBehaviours?");`
	`253`	`+ return null;`
	`254`	`+ }`
`251`	`255`	`return childNetworkedBehaviours[index];`
`252`	`256`	`}`
`253`	`257`	`}`