Fix: Don't allow reconnect while connecting

Author: BornToBeRoot

Source/NETworkManager/Controls/DragablzTabHostWindow.xaml.cs

@@ -1,4 +1,12 @@
-using System;
+using Dragablz;
+using MahApps.Metro.Controls.Dialogs;
+using NETworkManager.Localization;
+using NETworkManager.Localization.Resources;
+using NETworkManager.Models;
+using NETworkManager.Models.RemoteDesktop;
+using NETworkManager.Settings;
+using NETworkManager.Utilities;
+using System;
 using System.Collections.Generic;
 using System.ComponentModel;
 using System.Diagnostics;
@@ -9,15 +17,6 @@
 using System.Windows.Forms;
 using System.Windows.Input;
 using System.Windows.Interop;
-using Dragablz;
-using MahApps.Metro.Controls.Dialogs;
-using NETworkManager.Localization;
-using NETworkManager.Localization.Resources;
-using NETworkManager.Models;
-using NETworkManager.Models.RemoteDesktop;
-using NETworkManager.Settings;
-using NETworkManager.Utilities;
-using Application = System.Windows.Application;
 
 namespace NETworkManager.Controls;
 
@@ -71,7 +70,7 @@ private async void FocusEmbeddedWindow()
 
             // Focus embedded window in the selected tab
             (((DragablzTabItem)tabablzControl.SelectedItem)?.View as IEmbeddedWindow)?.FocusEmbeddedWindow();
-            
+
             break;
         }
     }
@@ -169,7 +168,7 @@ private bool RemoteDesktop_IsConnected_CanExecute(object view)
     private bool RemoteDesktop_IsDisconnected_CanExecute(object view)
     {
         if (view is RemoteDesktopControl control)
-            return !control.IsConnected;
+            return !control.IsConnected && !control.IsConnecting;
 
         return false;
     }
@@ -376,8 +375,8 @@ private void DragablzTabHostWindow_OnClosing(object sender, CancelEventArgs e)
     {
         // Find all TabablzControl in the active window
         foreach (var tabablzControl in VisualTreeHelper.FindVisualChildren<TabablzControl>(this))
-        foreach (var tabItem in tabablzControl.Items.OfType<DragablzTabItem>())
-            ((IDragablzTabItem)tabItem.View).CloseTab();
+            foreach (var tabItem in tabablzControl.Items.OfType<DragablzTabItem>())
+                ((IDragablzTabItem)tabItem.View).CloseTab();
 
         // Reset the dragging state
         switch (ApplicationName)
@@ -481,14 +480,14 @@ private IntPtr HwndHook(IntPtr hwnd, int msg, IntPtr wParam, IntPtr lParam, ref
     }
 
     private void UpdateOnWindowResize()
-    { 
+    {
         // Find all TabablzControl
         foreach (var tabablzControl in VisualTreeHelper.FindVisualChildren<TabablzControl>(this))
         {
             // Skip if no items
             if (tabablzControl.Items.Count == 0)
                 continue;
-            
+
             foreach (var item in tabablzControl.Items.OfType<DragablzTabItem>())
             {
                 if (item.View is RemoteDesktopControl control)

Source/NETworkManager/Controls/RemoteDesktopControl.xaml.cs

@@ -8,6 +8,7 @@
 using NETworkManager.Settings;
 using NETworkManager.Utilities;
 using System;
+using System.Diagnostics;
 using System.Threading.Tasks;
 using System.Windows;
 using System.Windows.Input;
@@ -65,6 +66,8 @@ public bool IsConnected
             if (value == _isConnected)
                 return;
 
+            Debug.WriteLine("IsConnected: " + value);
+
             _isConnected = value;
             OnPropertyChanged();
         }
@@ -80,6 +83,8 @@ public bool IsConnecting
             if (value == _isConnecting)
                 return;
 
+            Debug.WriteLine("IsConnecting: " + value);
+
             _isConnecting = value;
             OnPropertyChanged();
         }
@@ -327,6 +332,9 @@ private void Connect()
 
     private void Reconnect()
     {
+        if (IsConnecting)
+            return;
+
         if (IsConnected)
             return;
 

Source/NETworkManager/ViewModels/RemoteDesktopHostViewModel.cs

@@ -245,7 +245,7 @@ private bool IsConnected_CanExecute(object view)
     private bool IsDisconnected_CanExecute(object view)
     {
         if (view is RemoteDesktopControl control)
-            return !control.IsConnected;
+            return !control.IsConnected && !control.IsConnecting;
 
         return false;
     }
@@ -281,7 +281,7 @@ private void FullscreenAction(object view)
     private void AdjustScreenAction(object view)
     {
         if (view is RemoteDesktopControl control)
-            control.AdjustScreen(force:true);
+            control.AdjustScreen(force: true);
     }
 
     public ICommand SendCtrlAltDelCommand => new RelayCommand(SendCtrlAltDelAction, IsConnected_CanExecute);