PlayerCommandPreprocessEvent?

Hello,
Wasn't really sure what to call this so I just called it one of the events in the plugin related to my question...
Anyways, I'm trying to create a plugin with a configuration file, which allows you to customize banned commands, the message received when attempting to use one and a possible punishment.
The command "bannedadd" isn't working for some reason, I'd appreciate any help. Thanks. Sorry if the indentation broke while copying and pasting...

Code:java

package me.Caprei.Restriction;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerCommandPreprocessEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Restriction extends JavaPlugin implements Listener{
public void onEnable(){
getServer().getLogger().info("[Restriction] Restriction has been enabled. Made by Caprei.");
getServer().getPluginManager().registerEvents(this, this);
getConfig().options().copyDefaults(true);
saveConfig();
}
public void onDisable(){
getServer().getLogger().info("[Restriction] Restriction has been disabled. Made by Caprei.");
 
}
@EventHandler(priority = EventPriority.HIGHEST)
public void onPlayerCommand(PlayerCommandPreprocessEvent e){
Player c = e.getPlayer();
String msg = e.getMessage().toLowerCase();
for(String banned:getConfig().getStringList("Banned Commands")){
if(msg.startsWith(banned)){
if(c.hasPermission("restriction.freechat")){
return;
}
c.sendMessage(getConfig().getString("Restricted Word Message"));
e.setCancelled(true);
if(getConfig().getBoolean("Punish Player")){
String p = getConfig().getString("Punishment").toLowerCase();
if(p == "kill"){
c.setHealth(0.0d);
return;
}
if(p == "starve"){
c.setFoodLevel(0);
return;
}
if(p == "burn"){
c.setFireTicks(1000);
return;
}
if(p == "fall")
{c.setFallDistance(50.0f); return;}
if(p == "explode"){
c.getWorld().createExplosion(c.getLocation(), 4.0f);
c.setHealth(0);
}
 
return;
}
}
}
}
public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String [] args){
if(cmd.getName().equalsIgnoreCase("bannedadd")){
if(sender.hasPermission("restriction.bannedadd")){
if(args.length == 0){
sender.sendMessage("" + ChatColor.RED + ChatColor.ITALIC + "Please specify a command to add to the list of banned commands.");
return true;
}
for(String banned:args){
getConfig().getStringList("Banned Commands").add(banned);
saveConfig();
}
sender.sendMessage("" + ChatColor.GREEN + ChatColor.ITALIC + "The commands you specified were added to the banned list!");
return true;
}sender.sendMessage("" + ChatColor.RED + ChatColor.ITALIC + "You do not have permission to add to the list of banned commands.");
return true;
}
if(cmd.getName().equalsIgnoreCase("banned"))
if(sender.hasPermission("restriction.see")){
sender.sendMessage("" + ChatColor.RED + ChatColor.BOLD + "These are the banned comands: " + getConfig().getStringList("Banned Commands"));
return true;
}sender.sendMessage("" + ChatColor.RED + ChatColor.ITALIC + "You do not have permission to see the banned commands.");
return true;
 
}
}
 

config.yml:

Code:

Banned Commands:
  - /op
  - /help
  - /pl
  - /give
  - /gamemode
 
Restricted Word Message: Please do not use that command. It is restricted for you.
 
Punish Player: false
Punishment: 

It's probably some really stupid mistake...
Thanks!

 

Why not put the "bannedadd" Command directly into PlayerCommandPrepocessEvent ?

 

Flybelette Well, is there any reason to do that? Will it fix the problem?

 

Maybe, Why not try ?

 

Flybelette Because I'm pretty sure that won't do anything other then calling the onCommand method every time a player enters a command.

 

Woot ? Everytime a player run a command it will throw PlayerCommandPreprocessEvent without need to add the Command in plugin.yml ...

 

Flybelette Maybe you should re-read my post, I wasn't saying anything about plugin.yml. I was simply saying, if I put the onCommand method in the onPlayerCommand method, everytime a player runs any command, it will call the onCommand method twice.