Prevent duplicate name check API call

Smart123s · Smart123s · commit fb8780d00678 · 2021-06-14T19:16:40.000+02:00
Previously: If allowFloodgateNameConflict = false or linked and either autoLoginFloodgate or
autoRegisterFloodgate is set to no-conflict and a player with a non-conflicting name
connects then there will be a double check.
diff --git a/bukkit/src/main/java/com/github/games647/fastlogin/bukkit/task/FloodgateAuthTask.java b/bukkit/src/main/java/com/github/games647/fastlogin/bukkit/task/FloodgateAuthTask.java
@@ -76,7 +76,10 @@ public void run() {
         }
 
         //decide if checks should be made for conflicting Java player names
-        if (!isLinked &&
+        if (!isLinked //linked players have the same name as their Java profile
+                // if allowNameConflict is 'false' or 'linked' and the player had a conflicting
+                // name, than they would have been kicked in FloodgateHook#checkNameConflict
+                && allowNameConflict.equals("true") &&
                 (
                         autoLoginFloodgate.equals("no-conflict")
                         || !isRegistered && autoRegisterFloodgate.equals("no-conflict"))

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,10 @@ public void run() {`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`//decide if checks should be made for conflicting Java player names`
`79`		`- if (!isLinked &&`
	`79`	`+ if (!isLinked //linked players have the same name as their Java profile`
	`80`	`+ // if allowNameConflict is 'false' or 'linked' and the player had a conflicting`
	`81`	`+ // name, than they would have been kicked in FloodgateHook#checkNameConflict`
	`82`	`+ && allowNameConflict.equals("true") &&`
`80`	`83`	`(`
`81`	`84`	`autoLoginFloodgate.equals("no-conflict")`
`82`	`85`	`\|\| !isRegistered && autoRegisterFloodgate.equals("no-conflict"))`