charleslemaux/Mirror
1
0
Fork 0
mirror of https://github.com/MirrorNetworking/Mirror.git synced 2024-11-18 02:50:32 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix(NetworkManager): Move networkAddress check up in StartClient

Browse Source 
- If we're going to abort for this, do it before setting anything up.
- If/when the networkAddress is later set, we won't have issue with double initilization when StartClient is called again.
This commit is contained in:
MrGadget1024 2023-11-24 18:03:05 -05:00
parent dfec9a713e
commit 11dc4a9f60
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
Assets/Mirror/Core/NetworkManager.cs
View File

@ -404,6 +404,12 @@ public void StartClient()
return;
}
if (string.IsNullOrWhiteSpace(networkAddress))
{
Debug.LogError("Must set the Network Address field in the manager");
return;
}
mode = NetworkManagerMode.ClientOnly;
SetupClient();
@ -413,13 +419,6 @@ public void StartClient()
RegisterClientMessages();
if (string.IsNullOrWhiteSpace(networkAddress))
{
Debug.LogError("Must set the Network Address field in the manager");
return;
}
// Debug.Log($"NetworkManager StartClient address:{networkAddress}");
NetworkClient.Connect(networkAddress);
OnStartClient();