Solved Can't get arraylist to work the way I want

So I've been messing with this quite a bit, but in the end I can't get it to work.

Unfortunately, there are no errors popping up I can show.
What I wanted was, when the player name gets stored in the arraylist, I want the player to be able to move items around in the inventory.

When I type /invedit, it all works fine, and it knows that I am stored in the array. But when I try to move my items, it's like it doesn't check if I'm in the array.
The command works fine. It knows when the player is stored and when the player isn't stored. The same can't be said for the event though.

Here is the class.

Code:java

public class AdminEditTools implements Listener, CommandExecutor{
 
    private ArrayList<String> admin = new ArrayList<String>();
 
    public boolean onCommand(CommandSender sender, Command command, String alias, String[] args) {
        Player p = (Player)sender;
        if (alias.equalsIgnoreCase("invedit")) {
            if (p.hasPermission("YangFu.invedit")) {
                if (!admin.contains(p.getName())) {
                    p.sendMessage("§aInventory edit has been enabled");
                    admin.add(p.getName());
                    return true;
                }
                else if (admin.contains(p.getName()));
                    p.sendMessage("§cInventory edit has been disabled");
                    admin.remove(p.getName());
                    return true;
            }
        }
        return true;
    }
 
    @EventHandler
    public void onItemClick(InventoryClickEvent e) {
        Player p = (Player)e.getWhoClicked();
        if (!admin.contains(p.getName())) {
            e.setCancelled(true);
            return;
        }
    }
}

Thanks in advance. Dearly, TheLovelySloth.

Edit:
Okay I tweaked a few things, and did it the way Zombie_Striker told me. It got fixed! Woohoo! :3 You learn something new everyday

 

@TheLovelySlothMan
Most likely, you are registering two instance of the class. Do tell me, do you have code that looks like this?:

Code:

getCommand("...").setExecutor(New AdminEdit...);
...
Bukkit.getPluginManager.registerEvents(New AdminEdit...., this);

Notice how you have to create two instance of AdminEdit'. What that means is the command will not interact with the events, because they are two separate objects.

To fix this, make sure they are the same instance by using the following:

Code:

AdminEdit adminEdit = new AdminEdit;

getCommand("...").setExecutor(adminEdit);
...
Bukkit.getPluginManager.registerEvents(adminEdit, this);