Fixed bloated connection errors in logs;

Fixed that data update tasks runs after disconnect;

Author: JayDi85

Mage.Client/src/main/java/mage/client/MageFrame.java

@@ -1416,11 +1416,11 @@ public void disconnected(final boolean askToReconnect) {
             // USER mode, e.g. user plays and got disconnect
             LOGGER.info("Disconnected from user mode");
             SwingUtilities.invokeLater(() -> {
+                        SessionHandler.disconnect(false); // user already disconnected, can't do any online actions like quite chat
                         setConnectButtonText(NOT_CONNECTED_TEXT);
                         disableButtons();
                         hideGames();
                         hideTables();
-                        SessionHandler.disconnect(false);
                         if (askToReconnect) {
                             UserRequestMessage message = new UserRequestMessage("Connection lost", "The connection to server was lost. Reconnect to " + MagePreferences.getLastServerAddress() + "?");
                             message.setButton1("No", null);

Mage.Client/src/main/java/mage/client/table/TablesPanel.java

@@ -513,6 +513,7 @@ public void mouseClicked(MouseEvent e) {
     public void cleanUp() {
         saveGuiSettings();
         chatPanelMain.cleanUp();
+        stopTasks();
     }
 
     public void changeGUISize() {

Mage.Common/src/main/java/mage/remote/SessionImpl.java

@@ -1583,12 +1583,24 @@ public boolean lockUser(String userName, long durationMinute) {
     }
 
     private void handleThrowable(Throwable t) {
-        logger.fatal("Communication error", t);
 
+        // ignore interrupted exceptions -- it's connection problem or user's close
+        if (t instanceof InterruptedException) {
+            //logger.error("Connection error: was interrupted", t);
+            Thread.currentThread().interrupt();
+            return;
+        }
+
+        if (t instanceof RuntimeException) {
+            RuntimeException re = (RuntimeException) t;
+            if (t.getCause() instanceof InterruptedException) {
+                //logger.error("Connection error: was interrupted by runtime exception", t.getCause());
+                Thread.currentThread().interrupt();
+                return;
+            }
+        }
 
-        // Probably this can cause hanging the client under certain circumstances as the disconnect method is synchronized
-        // so check if it's needed
-        // disconnect(true);
+        logger.fatal("Connection error: other", t);
     }
 
     private void handleMageException(MageException ex) {