Skip to content

Commit 32255cd

Browse files
committed
Fix null check for OfflinePlayer in AdvancedCoreUser
- Added null check for OfflinePlayer before accessing last played time. - Included validation to return null if player name is blank.
1 parent 8a47fc1 commit 32255cd

1 file changed

Lines changed: 10 additions & 3 deletions

File tree

AdvancedCore/src/main/java/com/bencodez/advancedcore/api/user/AdvancedCoreUser.java

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -525,10 +525,14 @@ public long getLastOnline() {
525525
time = Long.valueOf(d);
526526
}
527527
if (time == 0 && getPlugin().getOptions().isOnlineMode()) {
528-
time = getOfflinePlayer().getLastPlayed();
529-
if (time > 0) {
530-
setLastOnline(time);
528+
OfflinePlayer player = getOfflinePlayer();
529+
if (player != null) {
530+
time = player.getLastPlayed();
531+
if (time > 0) {
532+
setLastOnline(time);
533+
}
531534
}
535+
532536
}
533537
return time;
534538
}
@@ -557,6 +561,9 @@ public int getNumberOfDaysSinceLogin() {
557561
*/
558562
@SuppressWarnings("deprecation")
559563
public OfflinePlayer getOfflinePlayer() {
564+
if (getPlayerName().isBlank()) {
565+
return null;
566+
}
560567
if (!plugin.getOptions().isOnlineMode()) {
561568
return Bukkit.getOfflinePlayer(getPlayerName());
562569
}

0 commit comments

Comments
 (0)