Remove duplicated disconnect case

-This removed disconnect case is covered on Line 2216.
Also removed case for resetOnLogon. Since we're checking for isResetSent(), the check on resetOnLogon should be redundant.
-Added return statement after disconnect(). This is behaviour is a bit inconsistent in the Session class, however it prevents duplicated state reset updates that are covered in disconnect().

Author: Ramonster1

quickfixj-core/src/main/java/quickfix/Session.java

@@ -2277,11 +2277,8 @@ private void nextLogon(Message logon) throws FieldNotFound, RejectLogon, Incorre
 
         // Check for proper sequence reset response
         if (state.isResetSent() && !state.isResetReceived()) {
-            if (resetOnLogon) {
-                disconnect("Expected Logon response to have reset sequence numbers in response to ResetSeqNumFlag", true);
-            } else {
-                disconnect("Received Logon response before sending request", true);
-            }
+            disconnect("Expected Logon response to have reset sequence numbers in response to ResetSeqNumFlag", true);
+            return;
         }
 
         state.setResetSent(false);