Solved Banning a Player

For some reason, when I try and ban a player for reach (I'm making an anti-reach and soon anti-aura plugin), however ironically upon banning and kicking the player, it crashes the server. I get no errors, it just crashes the server. Completely stops it. I get so much console spam from Paper I can't really send it. Basically this is what I do:

Code:java

 
if (bantwo.equalsIgnoreCase("ban")) {
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been banned for reach! Amount of checks done: " + checks);
                        Bukkit.getBanList(Type.NAME).addBan(player.getName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been banned for reach! Amount of checks done: " + checks, null, null);
                    } else if (bantwo.equalsIgnoreCase("banip")) {
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + "You have been IP banned for reach! Amount of checks done: " + checks);
                        Bukkit.getBanList(Type.IP).addBan(player.getAddress().getHostName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been IP banned for reach! Amount of checks done: " + checks, null, null);
                    } else if (bantwo.equalsIgnoreCase("tempban") || ban.equalsIgnoreCase("temp-ban") || ban.equalsIgnoreCase("temp ban")) {
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been temp-banned for reach! Amount of checks done: " + checks + ". You have been banned for " + date + ".");
                        Bukkit.getBanList(Type.NAME).addBan(player.getName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been temp-banned for reach! Amount of checks done: " + checks + ". You have been banned for " + datetwo + ".", datetwo, null);
                    }
 

As you can see, I kick the player first the add them to the ban list. Obviously, that doesn't make sense, but I have also tried not kicking the player, adding the player to the ban list first, kicking them after, etc. I assume there is a better way to ban the player, but I can't find anything on the internet. Everyone says to add the player to the ban list.

Code:

Code:java

 
package endran.voide.reach;
 
import java.text.SimpleDateFormat;
import java.util.Date;
import java.util.Random;
 
import org.bukkit.BanList.Type;
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.util.Vector;
 
public class PlayerListeners implements Listener {
 
    // Get's the core variable.
    private Core core;
 
    // Constructor
    public PlayerListeners(Core core) {
        this.core = core;
    }
 
    // Get's the date + time to then write into core.otherLog()
    Date now = new Date();
    SimpleDateFormat format = new SimpleDateFormat("dd-MM-yyyy HH:mm:ss");
    // Hypixel Hit Reg
    @EventHandler
    public void onDamage(EntityDamageByEntityEvent event) {
        if (core.getConfig().getString("enabled").equalsIgnoreCase("true")) {
        /*
        if (event.getCause() != DamageCause.ENTITY_ATTACK || event.getCause() != DamageCause.CONTACT) {
            return;
        }
        */
 
        // Get the damager
        Player player = (Player) event.getDamager();
 
        Player entity = (Player) event.getEntity();
 
        // If the damager damages the damaged...   
        if (((event.getDamager() instanceof Player)) && ((event.getEntity() instanceof Player))) {
            int upper = 5;
            // Random number
            Random random = new Random();
            // If num is greater than or equal to 5...
            int rNum = random.nextInt(5+(upper));
            if (rNum >= 5) {
                event.setCancelled(true);
            } else {
                event.setCancelled(false);
            }
 
            // Test for reach
 
            // Get the distance between the player's location and the victim's location
            /*
            Vector from = new Vector(player.getLocation().getX(), 1, player.getLocation().getZ());
            Vector to  = new Vector(entity.getLocation().getX(), 1, entity.getLocation().getZ());
            */
            Vector from = player.getLocation().toVector().setY(1);
            Vector to = entity.getLocation().toVector().setY(1);
 
            //Vector vector = to.subtract(from);
 
            double distance = from.distance(to) - 0.4;
 
            /*
            if (distance >= 3.5) {
                event.setCancelled(true);
            }
            */
            if (distance > 3) { // Since MC's default reach is 3, anything above that is considered reach
                int checks = core.getConfig().getInt(player.getUniqueId().toString()); // Get the amount of checks a player has in the config file
                int stageOneChecks = core.getConfig().getInt("stage-one-checks"); // The amount of checks done in stage one
                int stageTwoChecks = core.getConfig().getInt("stage-two-checks"); // The amount of checks done in stage two
                int stageThreeChecks = core.getConfig().getInt("stage-three-checks"); // The amount of checks done in stage three
                int stageTwoC = stageOneChecks + stageTwoChecks;
                int stageThree = stageOneChecks + stageTwoChecks + stageThreeChecks; // Same with stageTwo var
                String ban = core.getConfig().getString("ban-stage-one"); // What type of ban to do in stage one
                String bantwo = core.getConfig().getString("ban-stage-two"); // Same with ban var
                String banthree = core.getConfig().getString("ban-stage-three"); // Same with ban var
                //String bumper = StringUtils.repeat("\n", 35); // To create custom ban screen
                int min = core.getConfig().getInt("minutes-stage-one"); // Amount of minutes to temp-ban a player
                int mintwo = core.getConfig().getInt("minutes-stage-two"); // Same with min var
                int minthree = core.getConfig().getInt("minutes-stage-three"); // Same with min var
                Date date = new Date(System.currentTimeMillis()+min*60*1000); // Calculate the amount of time to ban a player for
                Date datetwo = new Date(System.currentTimeMillis()+mintwo*60*1000); // Same with date var
                Date datethree = new Date(System.currentTimeMillis()+minthree*60*1000); // Same with date var
                /*
                core.getConfig().set(player.getUniqueId().toString(), checks + 1);
                player.sendMessage(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You seem to have reach. Amount of checks so far: " + checks);
                */
                if (!core.getConfig().contains(player.getUniqueId().toString())) { // If the player isn't in the config file...
                    core.getConfig().set(player.getUniqueId().toString(), 1); // Add the player to the config file, then set the amount of checks to 1.
                    core.saveConfig(); // Save the config file
                } else if (checks == stageOneChecks) { // If the checks are not equal to null and the amount of checks equals stage one checks...
                    // Send a message to the victim
                    entity.sendMessage(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.GREEN + " " + player + " had reach and was banned. Player is at stage one.");
                    // Check on what type of ban to execute
                    if (ban.equalsIgnoreCase("ban")) { // If the stage-one ban is equal to ban...
                        // Ban the player and send a custom screen message. Type.NAME is a regular ban, Type.IP is an IP ban.
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been banned for reach! Amount of checks done: " + checks);
                        Bukkit.getBanList(Type.NAME).addBan(player.getName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been banned for reach! Amount of checks done: " + checks, null, null);
                    } else if (ban.equalsIgnoreCase("banip")) { // Same as above
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + "You have been IP banned for reach! Amount of checks done: " + checks);
                        Bukkit.getBanList(Type.IP).addBan(player.getAddress().getHostName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been IP banned for reach! Amount of checks done: " + checks, null, null);
                    } else if (ban.equalsIgnoreCase("tempban") || ban.equalsIgnoreCase("temp-ban") || ban.equalsIgnoreCase("temp ban")) {
                        // To temp ban a player, execute a regular ban but for a certain amount of time (specified by the date var)
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been temp-banned for reach! Amount of checks done: " + checks + ". You have been banned for " + date + ".");
                        Bukkit.getBanList(Type.NAME).addBan(player.getName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been temp-banned for reach! Amount of checks done: " + checks + ". You have been banned for " + date + ".", date, null);
                    }
                } else if (checks == stageTwoC) {
                    // Same as above except fill in all the stage-two values.
                    entity.sendMessage(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.GREEN + " " + player + " had reach and was banned.");
                    if (bantwo.equalsIgnoreCase("ban")) {
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been banned for reach! Amount of checks done: " + checks);
                        Bukkit.getBanList(Type.NAME).addBan(player.getName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been banned for reach! Amount of checks done: " + checks, null, null);
                    } else if (bantwo.equalsIgnoreCase("banip")) {
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + "You have been IP banned for reach! Amount of checks done: " + checks);
                        Bukkit.getBanList(Type.IP).addBan(player.getAddress().getHostName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been IP banned for reach! Amount of checks done: " + checks, null, null);
                    } else if (bantwo.equalsIgnoreCase("tempban") || ban.equalsIgnoreCase("temp-ban") || ban.equalsIgnoreCase("temp ban")) {
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been temp-banned for reach! Amount of checks done: " + checks + ". You have been banned for " + date + ".");
                        Bukkit.getBanList(Type.NAME).addBan(player.getName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been temp-banned for reach! Amount of checks done: " + checks + ". You have been banned for " + datetwo + ".", datetwo, null);
                    }
                } else if (checks == stageThree) {
                    // Same as above except fill in all the stage-three values.
                    entity.sendMessage(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.GREEN + " " + player + " had reach and was banned.");
                    if (banthree.equalsIgnoreCase("ban")) {
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been banned for reach! Amount of checks done: " + checks);
                        Bukkit.getBanList(Type.NAME).addBan(player.getName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been banned for reach! Amount of checks done: " + checks, null, null);
                    } else if (banthree.equalsIgnoreCase("banip")) {
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + "You have been IP banned for reach! Amount of checks done: " + checks);
                        Bukkit.getBanList(Type.IP).addBan(player.getAddress().getHostName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been IP banned for reach! Amount of checks done: " + checks, null, null);
                    } else if (banthree.equalsIgnoreCase("tempban") || ban.equalsIgnoreCase("temp-ban") || ban.equalsIgnoreCase("temp ban")) {
                        player.kickPlayer(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been temp-banned for reach! Amount of checks done: " + checks + ". You have been banned for " + date + ".");
                        Bukkit.getBanList(Type.NAME).addBan(player.getName(), ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You have been temp-banned for reach! Amount of checks done: " + checks + ". You have been banned for " + datethree + ".", datethree, null);
                    }
                } else {
                    // If the player is in the config file but doesn't have the same amount of checks as required, add one check to the player.
                    core.getConfig().set(player.getUniqueId().toString(), checks + 1);
                    player.sendMessage(ChatColor.WHITE + "[" + ChatColor.GOLD + "Voide Reach" + ChatColor.WHITE + "]" + ChatColor.RED + " You seem to have reach. Amount of checks so far: " + checks);
                }
            }
        }
    }
  }
}
 

 

There is nothing in your code that should cause that. There are plenty of issues, but nothing so drastic as to cause the whole server to go down in flames as you describe. What are your config values, just out of curiosity?

 

That might be it:

Code:yml

 
enabled: true
stage-one-checks: 3
stage-two-checks: 3
stage-three-checks: 3
ban-stage-one: temp-ban
minutes-stage-one: 1
ban-stage-two: temp-ban
minutes-stage-two: 1
ban-stage-three: ban
minutes-stage-three: 1
 

And yes, I made the whole "stage" thing within like 30 minutes. Rushed it, didn't take the time to think through it :/

EDIT: alright, I recoded the whole entire class. Made the banning a config thing. Marking as solved. Sorry for wasting everyone's time.