Added command to toggler, unknown command

So I decided to add a toggler to my antifenceglitch plugin, so admins can toggle the notifications they get when people are trying to fence glitch, on and off. but when i did /af it says unknown command, could someone tell me what I am doing wrong?

AntiFence.java

Code:

package me.jorrik98;
 
import org.bukkit.event.Listener;
import org.bukkit.plugin.java.JavaPlugin;
 
public class AntiFence extends JavaPlugin implements Listener{
   
    @Override
    public void onEnable() {
        getLogger().info("PollosAntiFence by jorrik98 has been enabled!");
        new PlayerListener(this);
    }
   
    @Override
    public void onDisable() {
        getLogger().info("PollosAntiFence by jorrik98 has been disabled!");
    }
}

PlayerListener.java

Code:

package me.jorrik98;
 
import java.util.ArrayList;
import java.util.List;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.EventHandler;
 
import me.jorrik98.AntiFence;
 
public class PlayerListener implements Listener {
List<String> toggled = new ArrayList<String>();
   
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
        if (label.equalsIgnoreCase("af")) {
            if(!(sender instanceof Player)) {
                sender.sendMessage(ChatColor.RED + "You must be a player to use this command");
                return false;
            }
            Player player = (Player) sender;
            if(toggled.contains(player.getName())) {
                player.sendMessage(ChatColor.RED +  "No longer receing fence glitch notifications");
                toggled.remove(player.getName());
                return true;
            }
           
            player.sendMessage(ChatColor.RED +  "Now receing fence glitch notifications");
            toggled.add(player.getName());
            return true;
        }
        return false;
    }
   
    public PlayerListener(AntiFence plugin)  {
        plugin.getServer().getPluginManager().registerEvents(this, plugin);
    }
   
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
        Player p = e.getPlayer();
        if (e.getAction() == Action.RIGHT_CLICK_BLOCK) {
            if(e.getClickedBlock().getType().equals(Material.FENCE) || e.getClickedBlock().getType().equals(Material.NETHER_FENCE)) {
                e.setCancelled(true);
                p.sendMessage(ChatColor.RED + "You actually thought fence glitchin still works? - jorrik98");
                for(String player : toggled){
                      Bukkit.getPlayer(player).sendMessage(ChatColor.DARK_PURPLE +  e.getPlayer().getName() + ChatColor.AQUA + " just failed to fence glitch.");
                    }
            }   
        }
    }
}

Also, I just started coding, so don't rage at me for not knowing something basic, thanks.

 

jorrik98
http://wiki.bukkit.org/Plugin_Tutorial#Using_a_separate_CommandExecutor_class

 

Listeners don't listen for Commands but for events, such as PlayerMoveEvent

You should let your class implements CommandExecutor instead

 

AronTheGamer
He has an event in the class he needs to listen, so instead let the class implement both CommandExecutor and Listener.

 

Oh overlooked that.

Code:java

 
package me.jorrik98;
 
import java.util.ArrayList;
import java.util.List;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.command.CommandExecutor;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.EventHandler;
 
import me.jorrik98.AntiFence;
 
public class PlayerListener implements Listener, CommandExecutor {
List<String> toggled = new ArrayList<String>();
 
public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
if (label.equalsIgnoreCase("af")) {
if(!(sender instanceof Player)) {
sender.sendMessage(ChatColor.RED + "You must be a player to use this command");
return false;
}
Player player = (Player) sender;
if(toggled.contains(player.getName())) {
player.sendMessage(ChatColor.RED + "No longer receing fence glitch notifications");
toggled.remove(player.getName());
return true;
}
 
player.sendMessage(ChatColor.RED + "Now receing fence glitch notifications");
toggled.add(player.getName());
return true;
}
return false;
}
 
public PlayerListener(AntiFence plugin) {
plugin.getServer().getPluginManager().registerEvents(this, plugin);
plugin.getServer().getCommand("af").setExecutor(this);
}
 
@EventHandler
public void onPlayerInteract(PlayerInteractEvent e) {
Player p = e.getPlayer();
if (e.getAction() == Action.RIGHT_CLICK_BLOCK) {
if(e.getClickedBlock().getType().equals(Material.FENCE) || e.getClickedBlock().getType().equals(Material.NETHER_FENCE)) {
e.setCancelled(true);
p.sendMessage(ChatColor.RED + "You actually thought fence glitchin still works? - jorrik98");
for(String player : toggled){
Bukkit.getPlayer(player).sendMessage(ChatColor.DARK_PURPLE + e.getPlayer().getName() + ChatColor.AQUA + " just failed to fence glitch.");
}
}
}
}
}