Fix progress query management and thread safety

Flow.Launcher/MainWindow.xaml.cs

@@ -26,7 +26,7 @@
 using Flow.Launcher.Plugin.SharedModels;
 using Flow.Launcher.ViewModel;
 using iNKORE.UI.WPF.Modern;
 using iNKORE.UI.WPF.Modern.Controls;
 using DataObject = System.Windows.DataObject;
 using Key = System.Windows.Input.Key;
 using MouseButtons = System.Windows.Forms.MouseButtons;
@@ -76,6 +76,9 @@
         private const double DefaultRightMargin = 66; //* this value from base.xaml
         private bool _isClockPanelAnimating = false;
 
+        // Search Delay
+        private bool _ignoreTextChange = false;
+
         // IDisposable
         private bool _disposed = false;
 
@@ -116,7 +119,7 @@
         {
             var handle = Win32Helper.GetWindowHandle(this, true);
             _hwndSource = HwndSource.FromHwnd(handle);
             _hwndSource.AddHook(WndProc);
             Win32Helper.HideFromAltTab(this);
             Win32Helper.DisableControlBox(this);
         }
@@ -374,7 +377,7 @@
         {
             try
             {
                 _hwndSource.RemoveHook(WndProc);
             }
             catch (Exception)
             {
@@ -587,9 +590,9 @@
 
         #endregion
 
         #region Window WndProc
 
         private IntPtr WndProc(IntPtr hwnd, int msg, IntPtr wParam, IntPtr lParam, ref bool handled) 
         {
             switch (msg)
             {
@@ -813,17 +816,17 @@
                 Header = Localize.iconTrayOpen() + " (" + _settings.Hotkey + ")",
                 Icon = openIcon
             };
             var gamemodeIcon = new FontIcon { Glyph = "\ue7fc" };
             var gamemode = new MenuItem
             {
                 Header = Localize.GameMode(),
                 Icon = gamemodeIcon
             };
             var positionresetIcon = new FontIcon { Glyph = "\ue73f" };
             var positionreset = new MenuItem
             {
                 Header = Localize.PositionReset(),
                 Icon = positionresetIcon
             };
             var settingsIcon = new FontIcon { Glyph = "\ue713" };
             var settings = new MenuItem
@@ -1421,11 +1424,19 @@
 
         private void QueryTextBox_TextChanged1(object sender, TextChangedEventArgs e)
         {
+            if (_ignoreTextChange) return;
             var textBox = (TextBox)sender;
             _viewModel.QueryText = textBox.Text;
             _viewModel.Query(_settings.SearchQueryResultsWithDelay);
         }
 
+        public void SetQueryTextBoxText(string text)
+        {
+            _ignoreTextChange = true;
+            QueryTextBox.Text = text;
+            _ignoreTextChange = false;
+        }
+
         #endregion
 
         #region Dialog Jump

Flow.Launcher/ViewModel/MainViewModel.cs

@@ -759,7 +759,14 @@ public void ChangeQueryText(string queryText, bool isReQuery = false)
             if (QueryText != queryText)
             {
                 // Change query text first
-                QueryText = queryText;
+                // We use private field and manually set QueryTextBox instead of QueryText setter
+                // so that QueryTextBox_TextChanged1 will not be invoked to avoid duplicated Query calls
+                _queryText = queryText;
+                if (Application.Current?.MainWindow is MainWindow mainWindow)
+                {
+                    mainWindow.SetQueryTextBoxText(queryText);
+                }
+
                 // When we are changing query from codes, we should not delay the query
                 Query(false, isReQuery: false);
 
@@ -791,7 +798,14 @@ private async Task ChangeQueryTextAsync(string queryText, bool isReQuery = false
             if (QueryText != queryText)
             {
                 // Change query text first
-                QueryText = queryText;
+                // We use private field and manually set QueryTextBox instead of QueryText setter
+                // so that QueryTextBox_TextChanged1 will not be invoked to avoid duplicated Query calls
+                _queryText = queryText;
+                if (Application.Current?.MainWindow is MainWindow mainWindow)
+                {
+                    mainWindow.SetQueryTextBoxText(queryText);
+                }
+
                 // When we are changing query from codes, we should not delay the query
                 await QueryAsync(false, isReQuery: false);
 
@@ -870,11 +884,18 @@ private ResultsViewModel SelectedResults
                     }
                     _queryTextBeforeLeaveResults = QueryText;
 
+                    // We use private field and manually set QueryTextBox instead of QueryText setter
+                    // so that QueryTextBox_TextChanged1 will not be invoked to avoid duplicated Query calls
+                    _queryText = string.Empty;
+                    if (Application.Current?.MainWindow is MainWindow mainWindow)
+                    {
+                        mainWindow.SetQueryTextBoxText(string.Empty);
+                    }
+
                     // Because of Fody's optimization
                     // setter won't be called when property value is not changed.
                     // so we need manually call Query()
                     // http://stackoverflow.com/posts/25895769/revisions
-                    QueryText = string.Empty;
                     // When we are changing query because selected results are changed to history or context menu,
                     // we should not delay the query
                     Query(false);