Fix issue that ActionKeywordRegistered will not exclude old action key

Author: Jack251970

Flow.Launcher.Core/Plugin/PluginManager.cs

@@ -340,18 +340,24 @@ public static List<Result> GetContextMenusForPlugin(Result result)
             return results;
         }
 
-        public static bool ActionKeywordRegistered(IReadOnlyList<string> actionKeywords)
+        public static bool ActionKeywordRegistered(IReadOnlyList<string> newActionKeywords, IReadOnlyList<string> oldActionKeywords)
         {
-            foreach (var actionKeyword in actionKeywords)
+            foreach (var actionKeyword in newActionKeywords)
             {
-                if (ActionKeywordRegistered(actionKeyword))
+                if (ActionKeywordRegistered(actionKeyword, oldActionKeywords))
                 {
                     return true;
                 }
             }
             return false;
         }
 
+        private static bool ActionKeywordRegistered(string actionKeyword, IReadOnlyList<string> oldActionKeywords)
+        {
+            if (oldActionKeywords.Contains(actionKeyword)) return false;
+            return ActionKeywordRegistered(actionKeyword);
+        }
+
         public static bool ActionKeywordRegistered(string actionKeyword)
         {
             // this method is only checking for action keywords (defined as not '*') registration

Flow.Launcher/ActionKeywords.xaml.cs

@@ -4,6 +4,7 @@
 using Flow.Launcher.ViewModel;
 using Flow.Launcher.Core;
 using System.Linq;
+using Flow.Launcher.Core.Plugin;
 
 namespace Flow.Launcher
 {
@@ -41,7 +42,7 @@ private void btnDone_OnClick(object sender, RoutedEventArgs _)
 
             newActionKeywords = newActionKeywords.Count > 0 ? newActionKeywords : new() { Query.GlobalPluginWildcardSign };
 
-            if (!PluginViewModel.IsActionKeywordRegistered(newActionKeywords))
+            if (!PluginManager.ActionKeywordRegistered(newActionKeywords, oldActionKeywords))
             {
                 pluginViewModel.ChangeActionKeyword(newActionKeywords, oldActionKeywords);
                 Close();

Flow.Launcher/ViewModel/PluginViewModel.cs

@@ -151,8 +151,6 @@ private void OpenDeletePluginWindow()
             PluginManager.API.ShowMainWindow();
         }
 
-        public static bool IsActionKeywordRegistered(IReadOnlyList<string> newActionKeywords) => PluginManager.ActionKeywordRegistered(newActionKeywords);
-
         [RelayCommand]
         private void SetActionKeywords()
         {