only show online people in /vanish list

Dumbo52 · Sep 25, 2011 · ad6f460 · ad6f460
1 parent 334cd00
commit ad6f460
Showing 1 changed file with 14 additions and 8 deletions.
diff --git a/src/cc/co/traviswatkins/ModMode.java b/src/cc/co/traviswatkins/ModMode.java
@@ -149,7 +149,7 @@ public boolean onCommand(CommandSender sender, Command command, String commandLa
     public boolean disableModMode(Player player)
     {
         if (!Permissions.hasPermission(player, Permissions.MODMODE_TOGGLE))
-            return false;
+            return true;
 
         if (!modmode.contains(player.getDisplayName()))
         {
@@ -167,7 +167,7 @@ public boolean disableModMode(Player player)
     public boolean enableModMode(Player player)
     {
         if (!Permissions.hasPermission(player, Permissions.MODMODE_TOGGLE))
-            return false;
+            return true;
 
         if (modmode.contains(player.getDisplayName()))
         {
@@ -210,7 +210,7 @@ public boolean enableModMode(Player player)
     public boolean disableVanish(Player player)
     {
         if (!Permissions.hasPermission(player, Permissions.UNVANISH))
-            return false;
+            return true;
 
         if (!isPlayerInvisible(player.getName()))
         {
@@ -244,7 +244,7 @@ public boolean disableVanish(Player player)
     public boolean enableVanish(Player player)
     {
         if (!Permissions.hasPermission(player, Permissions.VANISH))
-            return false;
+            return true;
 
         if (!isPlayerInvisible(player.getName()))
             invisible.add(player.getName());
@@ -273,21 +273,27 @@ public boolean enableVanish(Player player)
     private boolean vanishList(Player player)
     {
         if (!Permissions.hasPermission(player, Permissions.VANISH_LIST))
-            return false;
+            return true;
+
+        Set<String> online = new HashSet<String>();
+        Player[] onlinePlayers = this.getServer().getOnlinePlayers();
+        for (Player p : onlinePlayers)
+            if (invisible.contains(player.getName()))
+                online.add(player.getDisplayName());
 
-        if (invisible.isEmpty())
+        if (online.isEmpty())
         {
             player.sendMessage(ChatColor.RED + "Everyone is visible");
             return true;
         }
 
         String message = "Invisible Players: ";
         int i = 0;
-        for (String name : invisible)
+        for (String name : online)
         {
             message += name;
             i++;
-            if (i != invisible.size())
+            if (i != online.size())
             {
                 message += " ";
             }