From 0591939c5e4bf5a68018e00fd9014ac125f9acad Mon Sep 17 00:00:00 2001
From: Lex Li <support@lextm.com>
Date: Tue, 1 Apr 2025 22:45:10 -0700
Subject: [PATCH] Fixed a few issues in FRT.

---
 .../TraceFailedRequestsPage.Designer.cs                   | 1 +
 .../TraceFailedRequestsPage.cs                            | 8 ++++++++
 .../Wizards/AddTraceWizard/AddTraceWizard.cs              | 5 +++--
 .../Wizards/AddTraceWizard/ContentPage.cs                 | 3 ++-
 4 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/JexusManager.Features.TraceFailedRequests/TraceFailedRequestsPage.Designer.cs b/JexusManager.Features.TraceFailedRequests/TraceFailedRequestsPage.Designer.cs
index e87530e7..2af82c3a 100644
--- a/JexusManager.Features.TraceFailedRequests/TraceFailedRequestsPage.Designer.cs
+++ b/JexusManager.Features.TraceFailedRequests/TraceFailedRequestsPage.Designer.cs
@@ -123,6 +123,7 @@ private void InitializeComponent()
             this.listView1.UseCompatibleStateImageBehavior = false;
             this.listView1.View = System.Windows.Forms.View.Details;
             this.listView1.SelectedIndexChanged += new System.EventHandler(this.ListView1SelectedIndexChanged);
+            this.listView1.MouseDoubleClick += new System.Windows.Forms.MouseEventHandler(this.ListView1_MouseDoubleClick);
             // 
             // chPath
             // 
diff --git a/JexusManager.Features.TraceFailedRequests/TraceFailedRequestsPage.cs b/JexusManager.Features.TraceFailedRequests/TraceFailedRequestsPage.cs
index 7db418c3..9be3b249 100644
--- a/JexusManager.Features.TraceFailedRequests/TraceFailedRequestsPage.cs
+++ b/JexusManager.Features.TraceFailedRequests/TraceFailedRequestsPage.cs
@@ -109,6 +109,14 @@ private void ListView1SelectedIndexChanged(object sender, EventArgs e)
             Refresh();
         }
 
+        private void ListView1_MouseDoubleClick(object sender, MouseEventArgs e)
+        {
+            if (listView1.SelectedItems.Count > 0)
+            {
+                _feature.Edit();
+            }
+        }
+
         protected override bool ShowHelp()
         {
             _feature.ShowHelp();
diff --git a/JexusManager.Features.TraceFailedRequests/Wizards/AddTraceWizard/AddTraceWizard.cs b/JexusManager.Features.TraceFailedRequests/Wizards/AddTraceWizard/AddTraceWizard.cs
index 705d1312..2ca79704 100644
--- a/JexusManager.Features.TraceFailedRequests/Wizards/AddTraceWizard/AddTraceWizard.cs
+++ b/JexusManager.Features.TraceFailedRequests/Wizards/AddTraceWizard/AddTraceWizard.cs
@@ -27,6 +27,7 @@ public AddTraceWizard(IServiceProvider serviceProvider, TraceFailedRequestsItem
             _wizardData = new AddTraceWizardData(config, existing);
             InitializeComponent();
             TaskGlyph = Resources.trace_failed_requests_48;
+            Text = existing == null ? "Add Failed Request Tracing Rule" : "Edit Failed Request Tracing Rule";
         }
 
         protected override object WizardData
@@ -40,7 +41,7 @@ protected override void CompleteWizard()
         {
             Item = _existing == null ? new TraceFailedRequestsItem(null) : _existing;
             _wizardData.Apply(Item);
-            if (_feature.Items.Any(item => item.Match(Item)))
+            if (_existing == null && _feature.Items.Any(item => item.Match(Item)))
             {
                 ShowMessage(
                     "A failed request trace for this content already exists.",
@@ -76,7 +77,7 @@ protected override WizardPage[] GetWizardPages()
             conditionsPage.SetPreviousPage(contentPage);
             conditionsPage.SetNextPage(providersPage);
             providersPage.SetPreviousPage(conditionsPage);
-            return new WizardPage[] { contentPage, conditionsPage, providersPage };
+            return [contentPage, conditionsPage, providersPage];
         }
 
         protected override void ShowHelp()
diff --git a/JexusManager.Features.TraceFailedRequests/Wizards/AddTraceWizard/ContentPage.cs b/JexusManager.Features.TraceFailedRequests/Wizards/AddTraceWizard/ContentPage.cs
index 631f7ed6..bc6205c5 100644
--- a/JexusManager.Features.TraceFailedRequests/Wizards/AddTraceWizard/ContentPage.cs
+++ b/JexusManager.Features.TraceFailedRequests/Wizards/AddTraceWizard/ContentPage.cs
@@ -23,10 +23,11 @@ public ContentPage()
                 .Merge(Observable.FromEventPattern<EventArgs>(rbAspNet, "Click"))
                 .Merge(Observable.FromEventPattern<EventArgs>(rbAsp, "Click"))
                 .Merge(Observable.FromEventPattern<EventArgs>(rbCustom, "Click"))
+                .Merge(Observable.FromEventPattern<EventArgs>(txtContent, "TextChanged"))
                 .ObserveOn(System.Threading.SynchronizationContext.Current)
                 .Subscribe(evt =>
                 {
-                    txtContent.Enabled = rbCustom.Checked;
+                    txtContent.Enabled = rbCustom.Checked && rbCustom.Enabled;
                     UpdateWizard();
                 }));
         }