Command (SafePVP)

package xyz.fergy.safepvp.commands;

import net.md_5.bungee.api.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import xyz.fergy.safepvp.SafePvP;

import java.util.ArrayList;

public class SafePvPMode implements CommandExecutor{

    SafePvP plugin;

    public SafePvPMode(SafePvP instance){
        plugin = instance;
    }

    public static ArrayList<Player> toggledPlayers = new ArrayList<Player>();

    String noPermission = ChatColor.translateAlternateColorCodes('&', plugin.getConfig().getString("permissionmessage"));
    String toggleOn = ChatColor.translateAlternateColorCodes('&', plugin.getConfig().getString("toggleonmessage"));
    String toggleOff = ChatColor.translateAlternateColorCodes('&', plugin.getConfig().getString("toggleoffmessage"));


    public boolean onCommand(CommandSender sender, Command cmd, String label, String[]args){
        if(!sender.hasPermission("safepvp.safepvpmode")){
            sender.sendMessage(noPermission);
            return true;
        }

        if(!(sender instanceof Player)){
            sender.sendMessage(ChatColor.RED + "The Console may not execute this command");
            return true;
        }

        Player player = (Player) sender;

        if(!toggledPlayers.contains(player)){
            toggledPlayers.add(player);
            sender.sendMessage(toggleOn);
        }else {
            toggledPlayers.remove(player);
            sender.sendMessage(toggleOff);
        }


        return true;
    }
}