More work on tab completion, see details.. #77

Completed:
- Jail check
- jail clear
- jail createcell
- jail deletecell
- Jail list
- jail listcells
- jail mute

src/main/java/com/graywolf336/jail/command/subcommands/JailCheckCommand.java

@@ -1,7 +1,12 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
+import org.bukkit.util.StringUtil;
 
 import com.graywolf336.jail.JailManager;
 import com.graywolf336.jail.beans.Prisoner;
@@ -36,4 +41,19 @@ public class JailCheckCommand implements Command{
         return true;
     }
 
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        List<String> results = new ArrayList<String>();
+        
+        for(Prisoner p : jm.getAllPrisoners().values()) {
+            if(args.length == 2 && StringUtil.startsWithIgnoreCase(p.getLastKnownName(), args[1].toLowerCase())) {
+                results.add(p.getLastKnownName());
+            }else {
+                results.add(p.getLastKnownName());
+            }
+        }
+        
+        Collections.sort(results);
+        
+        return results;
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailClearCommand.java

@@ -1,9 +1,15 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
+import org.bukkit.util.StringUtil;
 
 import com.graywolf336.jail.JailManager;
 import com.graywolf336.jail.beans.ConfirmPlayer;
+import com.graywolf336.jail.beans.Jail;
 import com.graywolf336.jail.command.Command;
 import com.graywolf336.jail.command.CommandInfo;
 import com.graywolf336.jail.enums.Confirmation;
@@ -38,4 +44,21 @@ public class JailClearCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        List<String> results = new ArrayList<String>();
+        
+        for(Jail j : jm.getJails()) {
+            if((args.length == 2 && StringUtil.startsWithIgnoreCase(j.getName(), args[1]))
+                    || (args.length == 3 && StringUtil.startsWithIgnoreCase(j.getName(), args[2]))) {
+                results.add(j.getName());
+            }else {
+                results.add(j.getName());
+            }
+        }
+        
+        Collections.sort(results);
+        
+        return results;
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailCommand.java

@@ -1,6 +1,7 @@
 package com.graywolf336.jail.command.subcommands;
 
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.LinkedList;
 import java.util.List;
 
@@ -245,4 +246,9 @@ public class JailCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement (this'll take a while)
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailConfirmCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 
 import com.graywolf336.jail.JailManager;
@@ -88,4 +91,8 @@ public class JailConfirmCommand implements Command{
         return true;
     }
 
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //confirming doesn't require any tab completing
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailCreateCellCommand.java

@@ -1,8 +1,13 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
+import org.bukkit.util.StringUtil;
 
 import com.graywolf336.jail.JailManager;
 import com.graywolf336.jail.beans.Jail;
@@ -63,4 +68,22 @@ public class JailCreateCellCommand implements Command {
         return true;
     }
 
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //We shouldn't provide when they want a cell name
+        if(args.length >= 3) return Collections.emptyList();
+        
+        List<String> results = new ArrayList<String>();
+        
+        for(Jail j : jm.getJails()) {
+            if(args.length == 2 && StringUtil.startsWithIgnoreCase(j.getName(), args[1])) {
+                results.add(j.getName());
+            }else {
+                results.add(j.getName());
+            }
+        }
+        
+        Collections.sort(results);
+        
+        return results;
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailCreateCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
@@ -45,4 +48,9 @@ public class JailCreateCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //Creating a jail shouldn't provide tab completion
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailDeleteCellCommand.java

@@ -1,9 +1,16 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
+import org.bukkit.util.StringUtil;
 
 import com.graywolf336.jail.JailManager;
+import com.graywolf336.jail.beans.Cell;
 import com.graywolf336.jail.beans.ConfirmPlayer;
+import com.graywolf336.jail.beans.Jail;
 import com.graywolf336.jail.command.Command;
 import com.graywolf336.jail.command.CommandInfo;
 import com.graywolf336.jail.enums.Confirmation;
@@ -28,4 +35,32 @@ public class JailDeleteCellCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        List<String> results = new ArrayList<String>();
+        
+        switch(args.length) {
+            case 1:
+                results.addAll(jm.getJailNamesAsList());
+                break;
+            case 2:
+                for(Jail j : jm.getJails())
+                    if(StringUtil.startsWithIgnoreCase(j.getName(), args[1]))
+                        results.add(j.getName());
+                break;
+            case 3:
+                if(jm.isValidJail(args[1])) {
+                    Jail j = jm.getJail(args[1]);
+                    
+                    for(Cell c : j.getCells())
+                        if(StringUtil.startsWithIgnoreCase(c.getName(), args[2]))
+                            results.add(c.getName());
+                }
+                break;
+        }
+        
+        Collections.sort(results);
+        
+        return results;
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailDeleteCellsCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 
 import com.graywolf336.jail.JailManager;
@@ -28,4 +31,9 @@ public class JailDeleteCellsCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailDeleteCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 
 import com.graywolf336.jail.JailManager;
@@ -28,4 +31,9 @@ public class JailDeleteCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailHelpCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
 
@@ -21,4 +24,9 @@ public class JailHelpCommand implements Command {
         sender.sendMessage(ChatColor.GREEN + "https://github.com/graywolf336/Jail/wiki/Commands");
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailListCellsCommand.java

@@ -1,7 +1,12 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
+import org.bukkit.util.StringUtil;
 
 import com.graywolf336.jail.JailManager;
 import com.graywolf336.jail.beans.Cell;
@@ -51,4 +56,20 @@ public class JailListCellsCommand implements Command {
         sender.sendMessage(ChatColor.AQUA + "-------------------------");
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        List<String> results = new ArrayList<String>();
+        
+        for(Jail j : jm.getJails()) {
+            if(args.length == 2 && StringUtil.startsWithIgnoreCase(j.getName(), args[1])) {
+                results.add(j.getName());
+            }else {
+                results.add(j.getName());
+            }
+        }
+        
+        Collections.sort(results);
+        
+        return results;
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailListCommand.java

@@ -1,9 +1,13 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.ArrayList;
 import java.util.Collection;
+import java.util.Collections;
+import java.util.List;
 
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
+import org.bukkit.util.StringUtil;
 
 import com.graywolf336.jail.JailManager;
 import com.graywolf336.jail.beans.Jail;
@@ -61,4 +65,20 @@ public class JailListCommand implements Command {
         sender.sendMessage(ChatColor.AQUA + "-------------------------");
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        List<String> results = new ArrayList<String>();
+        
+        for(Jail j : jm.getJails()) {
+            if(args.length == 2 && StringUtil.startsWithIgnoreCase(j.getName(), args[1])) {
+                results.add(j.getName());
+            }else {
+                results.add(j.getName());
+            }
+        }
+        
+        Collections.sort(results);
+        
+        return results;
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailMuteCommand.java

@@ -1,6 +1,12 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+import org.bukkit.util.StringUtil;
 
 import com.graywolf336.jail.JailManager;
 import com.graywolf336.jail.command.Command;
@@ -35,4 +41,19 @@ public class JailMuteCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        switch(args.length) {
+            case 2:
+                List<String> results = new ArrayList<String>();
+                for(Player p : jm.getPlugin().getServer().getOnlinePlayers())
+                    if(StringUtil.startsWithIgnoreCase(p.getName(), args[1].toLowerCase()))
+                        results.add(p.getName());
+                
+                Collections.sort(results);
+                return results;
+            default:
+                return Collections.emptyList();
+        }
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailPayCommand.java

@@ -1,5 +1,7 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
 import java.util.concurrent.TimeUnit;
 
 import org.bukkit.ChatColor;
@@ -206,4 +208,9 @@ public class JailPayCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailRecordCommand.java

@@ -1,5 +1,6 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
 import java.util.List;
 
 import org.bukkit.command.CommandSender;
@@ -42,4 +43,9 @@ public class JailRecordCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailReloadCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
 
@@ -28,4 +31,9 @@ public class JailReloadCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //Reloading the plugin doesn't require tab completions
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailStatusCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
 
@@ -35,4 +38,8 @@ public class JailStatusCommand implements Command{
         return true;
     }
 
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //Checking your jail status doesn't require tab completion
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailStickCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
 
@@ -33,4 +36,9 @@ public class JailStickCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //Nothing to tab complete on jail stick
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailStopCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
 
@@ -37,4 +40,9 @@ public class JailStopCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //no tab completion required for stop command
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailTeleInCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
 
@@ -51,4 +54,9 @@ public class JailTeleInCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailTeleOutCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
 
@@ -51,4 +54,9 @@ public class JailTeleOutCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailTimeCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 
 import com.graywolf336.jail.JailManager;
@@ -52,4 +55,9 @@ public class JailTimeCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailTransferAllCommand.java

@@ -1,6 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
 import java.util.HashSet;
+import java.util.List;
 
 import org.bukkit.command.CommandSender;
 
@@ -53,4 +55,9 @@ public class JailTransferAllCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailTransferCommand.java

@@ -1,6 +1,7 @@
 package com.graywolf336.jail.command.subcommands;
 
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.LinkedList;
 import java.util.List;
 
@@ -136,4 +137,9 @@ public class JailTransferCommand implements Command {
 
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailVersionCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 
 import com.graywolf336.jail.JailManager;
@@ -23,4 +26,8 @@ public class JailVersionCommand implements Command {
         return true;
     }
 
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //no tab completion required for version command
+        return Collections.emptyList();
+    }
 }

src/main/java/com/graywolf336/jail/command/subcommands/JailVoteCommand.java

@@ -1,5 +1,8 @@
 package com.graywolf336.jail.command.subcommands;
 
+import java.util.Collections;
+import java.util.List;
+
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
 
@@ -97,4 +100,9 @@ public class JailVoteCommand implements Command {
         
         return true;
     }
+
+    public List<String> provideTabCompletions(JailManager jm, CommandSender sender, String... args) throws Exception {
+        //TODO implement
+        return Collections.emptyList();
+    }
 }