Fix #29 to edit/delete secrets from context menu

timheuer · timheuer · commit 5f7be619b223 · 2023-11-21T11:00:47.000-08:00
diff --git a/src/ToolWindows/GHActionsToolWindow.xaml.cs b/src/ToolWindows/GHActionsToolWindow.xaml.cs
@@ -439,7 +439,7 @@ private async void EditSecret_Click(object sender, RoutedEventArgs e)
     {
         MenuItem menuItem = (MenuItem)sender;
         TextBlock tvi = GetParentTreeViewItem(menuItem);
-        if (tvi is not null && !tvi.Text.ToLowerInvariant().Contains(" (")) // yes a hack
+        if (tvi is not null && tvi.Text.ToLowerInvariant().Contains(" (")) // yes a hack
         {
             string header = tvi.Text.ToString();
             string secretName = header.Substring(0, header.IndexOf(" ("));
@@ -466,7 +466,7 @@ private async void DeleteSecret_Click(object sender, RoutedEventArgs e)
         MenuItem menuItem = (MenuItem)sender;
         TextBlock tvi = GetParentTreeViewItem(menuItem);
 
-        if (tvi is not null && !tvi.Text.ToLowerInvariant().Contains(" (")) // yes a hack
+        if (tvi is not null && tvi.Text.ToLowerInvariant().Contains(" (")) // yes a hack
         {
             await ThreadHelper.JoinableTaskFactory.SwitchToMainThreadAsync();
             // confirm the delete first

Original file line number	Diff line number	Diff line change
`@@ -439,7 +439,7 @@ private async void EditSecret_Click(object sender, RoutedEventArgs e)`
`439`	`439`	`{`
`440`	`440`	`MenuItem menuItem = (MenuItem)sender;`
`441`	`441`	`TextBlock tvi = GetParentTreeViewItem(menuItem);`
`442`		`- if (tvi is not null && !tvi.Text.ToLowerInvariant().Contains(" (")) // yes a hack`
	`442`	`+ if (tvi is not null && tvi.Text.ToLowerInvariant().Contains(" (")) // yes a hack`
`443`	`443`	`{`
`444`	`444`	`string header = tvi.Text.ToString();`
`445`	`445`	`string secretName = header.Substring(0, header.IndexOf(" ("));`
`@@ -466,7 +466,7 @@ private async void DeleteSecret_Click(object sender, RoutedEventArgs e)`
`466`	`466`	`MenuItem menuItem = (MenuItem)sender;`
`467`	`467`	`TextBlock tvi = GetParentTreeViewItem(menuItem);`
`468`	`468`
`469`		`- if (tvi is not null && !tvi.Text.ToLowerInvariant().Contains(" (")) // yes a hack`
	`469`	`+ if (tvi is not null && tvi.Text.ToLowerInvariant().Contains(" (")) // yes a hack`
`470`	`470`	`{`
`471`	`471`	`await ThreadHelper.JoinableTaskFactory.SwitchToMainThreadAsync();`
`472`	`472`	`// confirm the delete first`