Cooldowns.... Not working.

Okay, so i dont know what ive done wrong ive spent hours and hours googled and looking on bukkit for fixes... Nothing.. So what happens is when you hit a player with the stick it all works.. then if you try to hit him or another player it tells you that you are still on a cooldown. which is good. but you never get off the cooldown?

Code:java

 
@EventHandler
public void onDowny(EntityDamageByEntityEvent e) {
if ((e.getDamager() instanceof Player && (e.getEntity() instanceof Player))) {
    Player hit = (Player)e.getEntity();
    final Player hitter = (Player)e.getDamager();
   if(hitter.getItemInHand().getType() == Material.STICK){
   if (Main.cooldown.contains(hitter.getName())) {
   hitter.sendMessage(ChatColor.RED + "You are still on a cooldown!");
   return;
   }
   if (!Main.cooldown.contains(hitter.getName())) {
   if (Main.downy.contains(hitter.getName())) {
   hit.addPotionEffect(new PotionEffect(PotionEffectType.CONFUSION, 120, 0));
   hit.addPotionEffect(new PotionEffect(PotionEffectType.BLINDNESS, 120, 0));
   hit.addPotionEffect(new PotionEffect(PotionEffectType.WEAKNESS, 120, 0));
   hit.addPotionEffect(new PotionEffect(PotionEffectType.POISON, 120, 0));
   hitter.sendMessage(ChatColor.GRAY + "You gave " + ChatColor.AQUA + hit.getName() + ChatColor.GRAY + " down syndrome!" );
   hit.sendMessage(ChatColor.GRAY + "You got down syndrome from " + ChatColor.AQUA + hitter.getName() + ChatColor.GRAY + "!");
   Main.cooldown.add(hitter.getName());
   Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable() {
                         public void run() {
                                 Main.cooldown.remove(hitter.getName());
                         }
                 }, 300L);
   }
 
   }
 
   }
 
}
 
}
 

Anyone help me please.. I would really need it and appreciate it! Please <3

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

SenseTheGod
Is this in a separate class?

 

Yes it is. Gater12
In case you wanna see the main class..

Code:java

 
package me.sensethegod.main;
 
import java.util.ArrayList;
import java.util.List;
 
import me.sensethegod.kits.downy;
import me.sensethegod.kits.switcher;
import me.sensethegod.kits.fisherman;
import me.sensethegod.kits.archer;
import me.sensethegod.kits.pvp;
 
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener{
 
public void onEnable() {
    PluginManager pm = getServer().getPluginManager();
getServer().getPluginManager().registerEvents((Listener) this, this);
pm.registerEvents((Listener) new pvp(), this);
pm.registerEvents((Listener) new archer(), this);
pm.registerEvents((Listener) new fisherman(), this);
pm.registerEvents((Listener) new switcher(), this);
pm.registerEvents((Listener) new downy(), this);
getLogger().info("dKits Enabled!");
getCommands();
}
 
public void onDisable() {
getLogger().info("dKits Disabled! Please fix!");
}
 
 private void getCommands() {
   getCommand("pvp").setExecutor(new pvp());
   getCommand("archer").setExecutor(new archer());
   getCommand("fisherman").setExecutor(new fisherman());
   getCommand("switcher").setExecutor(new switcher());
   getCommand("downy").setExecutor(new downy());
 }
 
public static List<String> used = new ArrayList<String>();
public static List<String> cooldown = new ArrayList<String>();
public static List<String> downy = new ArrayList<String>();
@EventHandler
public void playerDeath(PlayerDeathEvent e) {
Player p = e.getEntity().getPlayer();
used.remove(p.getName());
downy.remove(p.getName());
cooldown.remove(p.getName());
}
}
 

 

SenseTheGod The scheduler first parameter takes in the the main class instance. So replace this with Main.

 

What do you mean? I don't follow, also do you have skype?

Gater12 Can you help me please? Anyone please help me?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

The bukkit schedule method takes plugin as its first parameter, so you can't use the "this" keyword if you're not in your main class.
Put "public static <Main class name> plugin" in your main class; then, to initialize the variable, put "plugin = this" in your onEnable() method. This isn't the ideal way to accomplish this, but it works.
Why do you make a joke out of down syndrome?