handle clipboard copying exceptions

pizzaboxer · Jun 18, 2024 · 845b5b7 · 845b5b7
1 parent e8d3114
commit 845b5b7
Show file tree

Hide file tree

Showing 4 changed files with 41 additions and 2 deletions.
diff --git a/Bloxstrap/Resources/Strings.Designer.cs b/Bloxstrap/Resources/Strings.Designer.cs
diff --git a/Bloxstrap/Resources/Strings.resx b/Bloxstrap/Resources/Strings.resx
@@ -126,6 +126,9 @@
   <data name="Bootstrapper.AutoUpdateFailed" xml:space="preserve">
     <value>Bloxstrap was unable to auto-update to {0}. Please update it manually by downloading and running the latest release from the GitHub page.</value>
   </data>
+  <data name="Bootstrapper.ClipboardCopyFailed" xml:space="preserve">
+    <value>Failed to copy to the clipboard: {0}</value>
+  </data>
   <data name="Bootstrapper.ConfirmLaunch" xml:space="preserve">
     <value>Roblox is currently running, and launching another instance will close it. Are you sure you want to continue launching?</value>
   </data>

diff --git a/Bloxstrap/UI/Elements/Dialogs/ExceptionDialog.xaml.cs b/Bloxstrap/UI/Elements/Dialogs/ExceptionDialog.xaml.cs
@@ -1,4 +1,5 @@
 using System.Media;
+using System.Runtime.InteropServices;
 using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Interop;
@@ -30,9 +31,20 @@ public ExceptionDialog(Exception exception)
             LocateLogFileButton.Click += delegate
             {
                 if (App.Logger.Initialized)
+                {
                     Process.Start("explorer.exe", $"/select,\"{App.Logger.FileLocation}\"");
+                }
                 else
-                    Clipboard.SetDataObject(String.Join("\r\n", App.Logger.Backlog));
+                {
+                    try
+                    {
+                        Clipboard.SetText(String.Join("\r\n", App.Logger.Backlog));
+                    }
+                    catch (COMException ex)
+                    {
+                        Frontend.ShowMessageBox(string.Format(Bloxstrap.Resources.Strings.Bootstrapper_ClipboardCopyFailed, ex.Message), MessageBoxImage.Error);
+                    }
+                }
             };
 
             ReportOptions.DropDownClosed += (sender, e) =>

diff --git a/Bloxstrap/UI/Elements/Menu/Pages/FastFlagEditorPage.xaml.cs b/Bloxstrap/UI/Elements/Menu/Pages/FastFlagEditorPage.xaml.cs
@@ -6,6 +6,7 @@
 using Wpf.Ui.Mvvm.Contracts;
 
 using Bloxstrap.UI.Elements.Dialogs;
+using System.Runtime.InteropServices;
 
 namespace Bloxstrap.UI.Elements.Menu.Pages
 {
@@ -341,8 +342,22 @@ private void ToggleButton_Click(object sender, RoutedEventArgs e)
 
         private void ExportJSONButton_Click(object sender, RoutedEventArgs e)
         {
+            const string LOG_IDENT = "FastFlagEditorPage::ExportJSONButton_Click";
+
             string json = JsonSerializer.Serialize(App.FastFlags.Prop, new JsonSerializerOptions { WriteIndented = true });
-            Clipboard.SetText(json);
+
+            try
+            {
+                Clipboard.SetText(json);
+            }
+            catch (COMException ex)
+            {
+                App.Logger.WriteLine(LOG_IDENT, "Failed to copy to the clipboard");
+                App.Logger.WriteException(LOG_IDENT, ex);
+                Frontend.ShowMessageBox(string.Format(Bloxstrap.Resources.Strings.Bootstrapper_ClipboardCopyFailed, ex.Message), MessageBoxImage.Error);
+                return;
+            }
+
             Frontend.ShowMessageBox(Bloxstrap.Resources.Strings.Menu_FastFlagEditor_JsonCopiedToClipboard, MessageBoxImage.Information);
         }