Added MCPlayer#isOnline

xDec0de · Nov 5, 2024 · 528a3a8 · 528a3a8
1 parent 5c034c0
commit 528a3a8
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 2 deletions.
diff --git a/platforms/spigot/src/main/java/net/codersky/mcutils/spigot/player/SpigotPlayer.java b/platforms/spigot/src/main/java/net/codersky/mcutils/spigot/player/SpigotPlayer.java
@@ -44,6 +44,11 @@ public String getName() {
 		return handle.getName();
 	}
 
+	@Override
+	public boolean isOnline() {
+		return handle.isOnline();
+	}
+
 	// Messages //
 
 	@Override

diff --git a/platforms/velocity/src/main/java/net/codersky/mcutils/velocity/player/VelocityPlayer.java b/platforms/velocity/src/main/java/net/codersky/mcutils/velocity/player/VelocityPlayer.java
@@ -40,6 +40,11 @@ public String getName() {
 		return handle.getUsername();
 	}
 
+	@Override
+	public boolean isOnline() {
+		return handle.isActive();
+	}
+
 	// Messages //
 
 	@Override

diff --git a/shared/src/main/java/net/codersky/mcutils/crossplatform/player/MCPlayer.java b/shared/src/main/java/net/codersky/mcutils/crossplatform/player/MCPlayer.java
@@ -67,6 +67,8 @@ public interface MCPlayer extends MessageReceiver, Replacement {
 	@NotNull
 	String getName();
 
+	boolean isOnline();
+
 	/*
 	 * Legacy actionbar messages (String)
 	 */

diff --git a/shared/src/main/java/net/codersky/mcutils/crossplatform/player/PlayerProvider.java b/shared/src/main/java/net/codersky/mcutils/crossplatform/player/PlayerProvider.java
@@ -19,8 +19,11 @@ public abstract class PlayerProvider<T> {
 	@Nullable
 	public MCPlayer getPlayer(@NotNull UUID uuid) {
 		MCPlayer player = playerCache.get(uuid);
-		if (player != null)
-			return player;
+		if (player != null) {
+			if (player.isOnline())
+				return player;
+			playerCache.remove(uuid);
+		}
 		player = fetchPlayer(uuid);
 		if (player != null)
 			playerCache.put(uuid, player);