From e3183e66df6abfe5cc043fbd03c8f5640b1697cc Mon Sep 17 00:00:00 2001 From: GunfighterJ Date: Mon, 1 Apr 2013 15:49:03 -0500 Subject: Show Geolocation in /seen Refactor argument for clarity --- .../earth2me/essentials/commands/Commandseen.java | 40 ++++++++++++++-------- 1 file changed, 26 insertions(+), 14 deletions(-) diff --git a/Essentials/src/com/earth2me/essentials/commands/Commandseen.java b/Essentials/src/com/earth2me/essentials/commands/Commandseen.java index 0b6cd6e46..b6640a0ca 100644 --- a/Essentials/src/com/earth2me/essentials/commands/Commandseen.java +++ b/Essentials/src/com/earth2me/essentials/commands/Commandseen.java @@ -9,6 +9,7 @@ import java.util.List; import org.bukkit.Location; import org.bukkit.Server; import org.bukkit.command.CommandSender; +import org.bukkit.entity.Player; public class Commandseen extends EssentialsCommand @@ -72,43 +73,54 @@ public class Commandseen extends EssentialsCommand if (user.isJailed()) { sender.sendMessage(_("whoisJail", (user.getJailTimeout() > 0 - ? Util.formatDateDiff(user.getJailTimeout()) - : _("true")))); + ? Util.formatDateDiff(user.getJailTimeout()) + : _("true")))); } if (user.isMuted()) { sender.sendMessage(_("whoisMuted", (user.getMuteTimeout() > 0 - ? Util.formatDateDiff(user.getMuteTimeout()) - : _("true")))); + ? Util.formatDateDiff(user.getMuteTimeout()) + : _("true")))); } if (extra) { sender.sendMessage(_("whoisIPAddress", user.getAddress().getAddress().toString())); + final String location = user.getGeoLocation(); + if (location != null && (!(sender instanceof Player) || ess.getUser(sender).isAuthorized("essentials.geoip.show"))) + { + sender.sendMessage(_("whoisGeoLocation", location)); + } + } } - private void seenOffline(final Server server, final CommandSender sender, User player, final boolean showBan, final boolean extra) throws Exception + private void seenOffline(final Server server, final CommandSender sender, User user, final boolean showBan, final boolean extra) throws Exception { - player.setDisplayNick(); - if (player.getLastLogout() > 0) + user.setDisplayNick(); + if (user.getLastLogout() > 0) { - sender.sendMessage(_("seenOffline", player.getName(), Util.formatDateDiff(player.getLastLogout()))); + sender.sendMessage(_("seenOffline", user.getName(), Util.formatDateDiff(user.getLastLogout()))); } else { - sender.sendMessage(_("userUnknown", player.getName())); + sender.sendMessage(_("userUnknown", user.getName())); } - if (player.isBanned()) + if (user.isBanned()) { - sender.sendMessage(_("whoisBanned", showBan ? player.getBanReason() : _("true"))); + sender.sendMessage(_("whoisBanned", showBan ? user.getBanReason() : _("true"))); } if (extra) { - if (!player.getLastLoginAddress().isEmpty()) + if (!user.getLastLoginAddress().isEmpty()) + { + sender.sendMessage(_("whoisIPAddress", user.getLastLoginAddress())); + } + final String location = user.getGeoLocation(); + if (location != null && (!(sender instanceof Player) || ess.getUser(sender).isAuthorized("essentials.geoip.show"))) { - sender.sendMessage(_("whoisIPAddress", player.getLastLoginAddress())); + sender.sendMessage(_("whoisGeoLocation", location)); } - final Location loc = player.getLogoutLocation(); + final Location loc = user.getLogoutLocation(); if (loc != null) { sender.sendMessage(_("whoisLocation", loc.getWorld().getName(), loc.getBlockX(), loc.getBlockY(), loc.getBlockZ())); -- cgit v1.2.3