Skip to content

Commit dce0a80

Browse files
committed
fix(GetPlayer): Return value (#141)
1 parent 82d91eb commit dce0a80

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

managed/src/SwiftlyS2.Core/Modules/Players/PlayerManagerService.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,9 @@ public void ClearAllBlockedTransmitEntities()
1818
NativePlayerManager.ClearAllBlockedTransmitEntity();
1919
}
2020

21-
public IPlayer GetPlayer( int playerid )
21+
public IPlayer? GetPlayer( int playerid )
2222
{
23+
if (!IsPlayerOnline(playerid)) return null;
2324
return new Player(playerid);
2425
}
2526

managed/src/SwiftlyS2.Shared/Modules/Players/IPlayerManager.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ public interface IPlayerManagerService
8484
/// <param name="playerid">The unique identifier of the player to retrieve. Must be a valid player ID.</param>
8585
/// <returns>An <see cref="IPlayer"/> instance representing the player with the specified ID, or <c>null</c> if no such
8686
/// player exists.</returns>
87-
public IPlayer GetPlayer( int playerid );
87+
public IPlayer? GetPlayer( int playerid );
8888

8989
/// <summary>
9090
/// Retrieves all players currently online.

0 commit comments

Comments
 (0)