Longs not working correctly

For some reason the variable check outputs two numbers and so it checks the second one and it automaticly cancels the event

Code:java

package net.othercraft.steelsecurity.listeners;
 
import java.util.HashMap;
import java.util.List;
import java.util.Map;
 
import net.othercraft.steelsecurity.Main;
import net.othercraft.steelsecurity.utils.SSCmdExe;
 
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerChatEvent;
 
public class ChatFilter extends SSCmdExe implements Listener {
 
	public Main plugin;
 
	Map<String, Long> chattimes = new HashMap<String, Long>();// for tracking the speed of chat
 
	public ChatFilter(String name, Main instance) {
		super(name, true);//true only if its a listener, false if it isnt
		this.plugin = instance;
	}
 
	@EventHandler
	public void onPlayerChat(PlayerChatEvent event){
		try {
			Boolean spam = true;//if this is true at the end cancel the event
			System.out.println(spam);//debug
			String name = event.getPlayer().getName();//name of the player for use in the hashmap
			Long time = System.currentTimeMillis();//current time
			Long lasttime = chattimes.get(name);//last time the player has chatted
			if(lasttime == null)lasttime = Long.valueOf(time - 1000);// default if the player hasnt chatted yet
			long check = (time - lasttime);//used to compare to the configured speed
			System.out.println(check);//debug
			chattimes.put(name, time);//overwrites the old time with the new one
			if (check > plugin.getConfig().getInt("AntiSpam.AntiFlood.Speed") || plugin.getConfig().getBoolean("AntiSpam.AntiFlood.Enabled") == false) {//checks if the speed of chat is faster than what is configured
				spam = false;//sets spam to false
				System.out.println(spam);//debug
			}
			System.out.println(spam);//debug
			String message = event.getMessage();//prepairs message for editing
			if ((spam == false)) {//only bothers with the anticaps and the censoring if it isnt spam.
				System.out.println(spam);//debug
				if (plugin.getConfig().getBoolean("AntiSpam.Censoring.Enabled") && event.getPlayer().hasPermission("steelsecurity.antispam.bypasscensor") == false) {//checks if it so scan for configured word
					@SuppressWarnings("unchecked")
					List<String> list =  (List<String>) plugin.getConfig().getList("AntiSpam.Censoring.Block_Words");//retreives the list of blocked words
					int wordcount = list.size();//the length of the list of blocked words
					int wordcounter = 0;//i like using this instead of for loops becasue these are less restricting
					while (wordcounter<wordcount) {
						String newword;
						String badword = list.get(wordcounter);
						int lettercount = list.get(wordcounter).toCharArray().length;//word length
						int lettercounter = 0;
						newword = "";
						while (lettercounter < lettercount) {//used to generate a new word considting of *s
							newword = (newword + "*");
							++lettercounter;
						}
						message = message.replaceAll("(?i)" + badword, newword);
						++wordcounter;
					}
				}
 
				if (event.getMessage().length()>plugin.getConfig().getInt("AntiSpam.AntiCaps.Minimum_Length")){//
					if (plugin.getConfig().getBoolean("AntiSpam.AntiCaps.Enabled") && event.getPlayer().hasPermission("steelsecurity.antispam.bypassanticaps") == false) {//checks for if it should do the anticaps check
						double percent = plugin.getConfig().getInt("AntiSpam.AntiCaps.Percent");//gets the configured percent
						int capcount = message.length();
						int capcounter = 0;
						Double uppercase = 0.0;
						Double lowercase = 0.0;
						while (capcounter<capcount) {
							if (message.toCharArray()[capcounter] == message.toLowerCase().toCharArray()[capcounter]) {//counts both upper case and lower case letters
								++lowercase;
							}
							else {
								++uppercase;
							}
							++capcounter;
						}
						double total = uppercase + lowercase;
						double result = uppercase/total;
						percent = percent/100;
						if (percent<result){//converts to lowercase if needed
							message = message.toLowerCase();
						}
					}
				}
				event.setMessage(message);//changes message to newly generated one
				System.out.println(spam);//debug
			}
			System.out.println(spam);//debug
			event.setCancelled(spam);//cancel if it spam from the beginning
		}
		catch (Exception e){
			catchListenerException(e, event.getEventName());
		}
	}
 
}

does anyone know how to fix this?

 

I dont get what your problem is

 

system.out.println(check); returns two numbers

 

mayby you registered the event 2 times?

 

i just checked but no it is only registered once.

no wait never mind i did register it twice ty

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