Fixed a potentially fatal bug; Build 4

- Added "breakApart" variable to the illegal characters cleaning for
configure command. Done to prevent a potentially fatal bug when adding
more than one role to the configuration if tagging the role.
- No longer using Instant to grab tiem, but using system time in ms
precision.

Author: AlphaSerpentis

Project-Lynx/pom.xml

@@ -2,7 +2,7 @@
   <modelVersion>4.0.0</modelVersion>
   <groupId>Project-Lynx</groupId>
   <artifactId>Project-Lynx</artifactId>
-  <version>0.2a</version>
+  <version>Pre-Release 0.2.0a Build 4</version>
   <name>Project Lynx</name>
   <description>JDA-based moderation bot</description>
   <properties>

Project-Lynx/src/main/java/commands/utilities/Configure.java

@@ -178,17 +178,19 @@ public boolean action(MessageChannel chn, String msg, Object misc) {
 									brokenString = breakApart;
 									breakApart = "";
 								}
-
-								// Check if they're tagged Roles or straight up long IDs
-								System.out.println("DEBUG - BREAK APART [Configure.java] " + breakApart);
-								System.out.println("DEBUG - BROKEN STRING [Configure.java] " + brokenString);
 
+								// Clean the strings
 								String[] illegal_chars = new String[] {"<", ">", "@", "&", "!", " "};
 
 								for(String ill_char: illegal_chars) {
 									brokenString = brokenString.replaceAll(ill_char, "");
+									breakApart = breakApart.replaceAll(ill_char, "");
 								}
 
+								// Check if they're tagged Roles or straight up long IDs
+								System.out.println("DEBUG - BREAK APART [Configure.java] " + breakApart);
+								System.out.println("DEBUG - BROKEN STRING [Configure.java] " + brokenString);
+
 								try {
 									decipheredRole = new Long(brokenString);
 								} catch (NumberFormatException e) {

Project-Lynx/src/main/java/handlers/MessageHandler.java

@@ -1,7 +1,6 @@
 package handlers;
 
 import java.io.File;
-import java.time.Instant;
 import java.util.HashMap;
 import java.util.function.Consumer;
 
@@ -36,7 +35,7 @@ public void onEvent(GenericEvent event) {
 
 			User author = ((MessageReceivedEvent) event).getAuthor();
 			Guild g = null;
-			int delayTimer = 1;
+			long delayTimer = 150; // 50 ms
 			String prefix; //Server's prefix... if it's even a server.
 
 			try {
@@ -80,9 +79,9 @@ public void onEvent(GenericEvent event) {
 
 				// Check if user is trying to spam the command.
 				if(!(limitCheck.get(((MessageReceivedEvent) event).getAuthor()) == null)) {
-					long currentInstant = Instant.now().getEpochSecond();
+					long currentMs = System.currentTimeMillis();
 
-					if(currentInstant-delayTimer <= limitCheck.get(((MessageReceivedEvent) event).getAuthor())) {
+					if(currentMs-delayTimer <= limitCheck.get(((MessageReceivedEvent) event).getAuthor())) {
 						System.out.println("WARNING [MessageHandler.java] " + author + " has breached the delay timer!");
 						return;
 					}
@@ -95,13 +94,19 @@ public void onEvent(GenericEvent event) {
 
 				//System.out.println("DEBUG [MessageHandler.java]: " + cmd.getName());
 				//System.out.println("DEBUG [MessageHandler.java]: (ChannelType) " + c);
-
-				if(c.equals(ChannelType.PRIVATE))
-					cmd.action(((MessageReceivedEvent) event).getAuthor().openPrivateChannel().complete(), fullMsg, event); //Just pass the entire thing to prevent NullPointers, each command will handle them appropriately
-				else if(c.equals(ChannelType.TEXT))
-					cmd.action(((MessageReceivedEvent) event).getChannel(), fullMsg, event);
 
-				limitCheck.put(((MessageReceivedEvent) event).getAuthor(), Instant.now().getEpochSecond());
+				new Thread() {
+					public void run() {
+						
+						if(c.equals(ChannelType.PRIVATE))
+							cmd.action(((MessageReceivedEvent) event).getAuthor().openPrivateChannel().complete(), fullMsg, event); //Just pass the entire thing to prevent NullPointers, each command will handle them appropriately
+						else if(c.equals(ChannelType.TEXT))
+							cmd.action(((MessageReceivedEvent) event).getChannel(), fullMsg, event);
+						
+					}
+				}.run();
+				
+				limitCheck.put(((MessageReceivedEvent) event).getAuthor(), System.currentTimeMillis());
 
 			}
 

Project-Lynx/src/main/java/init/InitData.java

@@ -78,6 +78,6 @@ public class InitData {
 	/*
 	 * Use this to define the version
 	 */
-	public static String version = "Pre-Release v0.2.0a Build 3 - The Rewrite Update";
+	public static String version = "Pre-Release v0.2.0a Build 4 - The Rewrite Update";
 
 }