From bdbdf8f82ed78a746e4e00ac8b33c6d09805a8ca Mon Sep 17 00:00:00 2001 From: Luke Stampfli Date: Fri, 5 Mar 2021 19:50:41 +0000 Subject: [PATCH] refactor: Improve error messages for NetworkPrefab list. --- .../Runtime/Core/NetworkManager.cs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/com.unity.multiplayer.mlapi/Runtime/Core/NetworkManager.cs b/com.unity.multiplayer.mlapi/Runtime/Core/NetworkManager.cs index 10ebde73b9..85c368fb2f 100644 --- a/com.unity.multiplayer.mlapi/Runtime/Core/NetworkManager.cs +++ b/com.unity.multiplayer.mlapi/Runtime/Core/NetworkManager.cs @@ -204,7 +204,7 @@ private void OnValidate() { if (NetworkLog.CurrentLogLevel <= LogLevel.Normal) { - NetworkLog.LogWarning($"{nameof(NetworkManager)} cannot be a {nameof(NetworkObject)}. This will lead to weird side effects."); + NetworkLog.LogWarning($"{nameof(NetworkManager)} cannot be a {nameof(NetworkObject)}."); } } @@ -360,14 +360,14 @@ private void Init(bool server) { if (NetworkLog.CurrentLogLevel <= LogLevel.Error) { - NetworkLog.LogError($"{nameof(NetworkPrefab)} cannot be null"); + NetworkLog.LogError($"{nameof(NetworkPrefab)} cannot be null ({nameof(NetworkPrefab)} at index: {i})"); } } else if (ReferenceEquals(NetworkConfig.NetworkPrefabs[i].Prefab.GetComponent(), null)) { if (NetworkLog.CurrentLogLevel <= LogLevel.Error) { - NetworkLog.LogError($"{nameof(NetworkPrefab)} is missing a {nameof(NetworkObject)} component"); + NetworkLog.LogError($"{nameof(NetworkPrefab)} (\"{NetworkConfig.NetworkPrefabs[i].Prefab.name}\") is missing a {nameof(NetworkObject)} component"); } } else @@ -932,7 +932,7 @@ internal void HandleIncomingData(ulong clientId, NetworkChannel networkChannel, { if (NetworkLog.CurrentLogLevel <= LogLevel.Error) { - NetworkLog.LogError("Message unwrap could not be completed. Was the header corrupt? Crypto error?"); + NetworkLog.LogError("Message unwrap could not be completed. Was the header corrupt?"); } return;