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: NetworkServer.OnDisconnect now calls DestroyPlayerForConnection if NetworkManager doesn't handle it.

Browse Source
This commit is contained in:
vis2k 2021-05-10 21:04:03 +08:00
parent eadda02701
commit 939f2820ea
1 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
Assets/Mirror/Runtime/NetworkServer.cs
View File

@ -424,18 +424,25 @@ internal static void OnTransportDisconnected(int connectionId)
RemoveConnection(connectionId);
// Debug.Log("Server lost client:" + connectionId);
// call OnDisconnected event.
// NetworkManager hooks into it with a virtual function to make
// DestroyPlayerForConnection(conn) optional, e.g. for PvP MMOs
// where players shouldn't be able to escape combat instantly.
// call OnDisconnected below. it's used in multiple places.
OnDisconnected(conn);
}
}
static void OnDisconnected(NetworkConnection conn)
{
OnDisconnectedEvent?.Invoke(conn);
//Debug.Log("Server lost client:" + conn);
// NetworkManager hooks into OnDisconnectedEvent to make
// DestroyPlayerForConnection(conn) optional, e.g. for PvP MMOs
// where players shouldn't be able to escape combat instantly.
if (OnDisconnectedEvent != null)
{
OnDisconnectedEvent.Invoke(conn);
}
// if nobody hooked into it, then simply call DestroyPlayerForConnection
else
{
DestroyPlayerForConnection(conn);
}
}
static void OnError(int connectionId, Exception exception)