From 475f3b83c118480c10d9d942eff1e55c856de3fb Mon Sep 17 00:00:00 2001 From: Rsl1122 Date: Fri, 1 Sep 2017 18:15:38 +0300 Subject: [PATCH] PlayersTableCreator bug fixes --- .../utilities/html/tables/PlayersTableCreator.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/Plan/src/main/java/com/djrapitops/plan/utilities/html/tables/PlayersTableCreator.java b/Plan/src/main/java/com/djrapitops/plan/utilities/html/tables/PlayersTableCreator.java index 5861eea26..2f1b5566a 100644 --- a/Plan/src/main/java/com/djrapitops/plan/utilities/html/tables/PlayersTableCreator.java +++ b/Plan/src/main/java/com/djrapitops/plan/utilities/html/tables/PlayersTableCreator.java @@ -27,6 +27,10 @@ public class PlayersTableCreator { } public static String createSortablePlayersTable(List userInfo, JoinInfoPart joinInfoPart, GeolocationPart geolocationPart) { + if (userInfo.isEmpty()) { + return Html.TABLELINE_PLAYERS.parse("No Players", "", "", "", "", "", "", "", "", ""); + } + StringBuilder html = new StringBuilder(); Map> sessions = joinInfoPart.getSessions(); @@ -45,16 +49,17 @@ public class PlayersTableCreator { boolean isBanned = user.isBanned(); List userSessions = sessions.get(uuid); int loginTimes = 0; + long playtime = 0; if (userSessions != null) { loginTimes = userSessions.size(); + playtime = AnalysisUtils.getTotalPlaytime(userSessions); } boolean isUnknown = loginTimes == 1; long registered = user.getRegistered(); - long playtime = AnalysisUtils.getTotalPlaytime(userSessions); boolean isActive = AnalysisUtils.isActive(now, user.getLastSeen(), playtime, loginTimes); - long lastSeen = AnalysisUtils.getLastSeen(userSessions); + long lastSeen = user.getLastSeen(); String activityString = getActivityString(isBanned, isUnknown, isActive); @@ -73,6 +78,7 @@ public class PlayersTableCreator { String.valueOf(geoLocation) )); } catch (NullPointerException ignored) { + ignored.printStackTrace(); // TODO IGNORE AGAIN } i++;