SpigotMC/Saber-Factions
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Double-space (or more) after command alias is no longer accepted, i.e. "f--home" (replace "-" with " " since GitHub is combining multiple spaces). Otherwise, command prevention routines (in Factions itself and in other plugins) can be bypassed. For example, preventing the "f-home" command wouldn't prevent anyone from using "f--home" or "f-----home" instead.

Browse Source
This commit is contained in:
Brettflan 2012-03-19 08:24:01 -05:00
parent d31bac8da8
commit 0eaffd7aef
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/com/massivecraft/factions/zcore/MPlugin.java
View File

@ -199,6 +199,9 @@ public abstract class MPlugin extends JavaPlugin
for (String alias : command.aliases) for (String alias : command.aliases)
{ {
// disallow double-space after alias, so specific commands can be prevented (preventing "f home" won't prevent "f home")
if (commandString.startsWith(alias+" ")) return false;
if (commandString.startsWith(alias+" ") || commandString.equals(alias)) if (commandString.startsWith(alias+" ") || commandString.equals(alias))
{ {
List<String> args = new ArrayList<String>(Arrays.asList(commandString.split("\\s+"))); List<String> args = new ArrayList<String>(Arrays.asList(commandString.split("\\s+")));