Fix COMException(CLIPBRD_E_CANT_OPEN) thrown by Clipboard.SetText

Use Clipboard.SetDataObject(text, true) and try multiple times within a loop and delay in between.

Author: CodingOctocat

IPConfig/Helpers/ClipboardHelper.cs

@@ -0,0 +1,30 @@
+using System.Runtime.InteropServices;
+using System.Threading.Tasks;
+using System.Windows;
+
+using HandyControl.Controls;
+
+namespace IPConfig.Helpers;
+
+public static class ClipboardHelper
+{
+    private const int CLIPBRD_E_CANT_OPEN = unchecked((int)0x800401D0);
+
+    public static async Task SetTextAsync(string text)
+    {
+        for (int i = 0; i < 10; i++)
+        {
+            try
+            {
+                Clipboard.SetDataObject(text, true);
+                Growl.Error(i.ToString());
+
+                return;
+            }
+            catch (COMException ex) when (ex.ErrorCode == CLIPBRD_E_CANT_OPEN)
+            { }
+
+            await Task.Delay(100);
+        }
+    }
+}

IPConfig/ViewModels/IPConfigDetailViewModel.cs

@@ -372,9 +372,9 @@ public void RejectChanges()
     #region RelayCommands
 
     [RelayCommand]
-    private static void CopyDns(string dns)
+    private static async Task CopyDnsAsync(string dns)
     {
-        Clipboard.SetText(dns);
+        await ClipboardHelper.SetTextAsync(dns);
     }
 
     [RelayCommand(AllowConcurrentExecutions = true)]

IPConfig/ViewModels/IPConfigListViewModel.cs

@@ -7,6 +7,7 @@
 using System.Linq;
 using System.Text;
 using System.Text.RegularExpressions;
+using System.Threading.Tasks;
 using System.Windows;
 using System.Windows.Data;
 
@@ -201,7 +202,7 @@ where match.Success
     }
 
     [RelayCommand]
-    private void CopySelectedIPConfigsAsText()
+    private async Task CopySelectedIPConfigsAsTextAsync()
     {
         string sep = new('=', 64);
         var sb = new StringBuilder();
@@ -213,7 +214,7 @@ private void CopySelectedIPConfigsAsText()
         }
 
         string text = sb.ToString().Trim()[..^64];
-        Clipboard.SetText(text);
+        await ClipboardHelper.SetTextAsync(text);
     }
 
     [RelayCommand(CanExecute = nameof(CanDeleteIPConfig))]

IPConfig/ViewModels/MainViewModel.cs

@@ -295,10 +295,10 @@ private void Closing(CancelEventArgs e)
     }
 
     [RelayCommand(CanExecute = nameof(IsSelectedNicNotNull))]
-    private void CopySelectedNicIPConfigAsText()
+    private async Task CopySelectedNicIPConfigAsTextAsync()
     {
         string text = $"{SelectedNic}\n\n{SelectedNicIPConfig}".Trim();
-        Clipboard.SetText(text);
+        await ClipboardHelper.SetTextAsync(text);
     }
 
     [RelayCommand]

IPConfig/ViewModels/NicConfigDetailViewModel.cs

@@ -5,7 +5,6 @@
 using System.Text.RegularExpressions;
 using System.Threading.Tasks;
 using System.Timers;
-using System.Windows;
 
 using CommunityToolkit.Mvvm.ComponentModel;
 using CommunityToolkit.Mvvm.Input;
@@ -69,7 +68,7 @@ public void Receive(PropertyChangedMessage<Nic> message)
     #region Relay Commands
 
     [RelayCommand]
-    private void CopyAllContentsAsText()
+    private async Task CopyAllContentsAsTextAsync()
     {
         string iPv4Header = Nic.SupportsIPv4 ? Lang.TcpIPv4_Header : $"{Lang.AdapterNotSupported} {Lang.TcpIPv4_Header}";
         string iPv6Header = Nic.SupportsIPv6 ? Lang.TcpIPv6_Header : $"{Lang.AdapterNotSupported} {Lang.TcpIPv6_Header}";
@@ -104,51 +103,51 @@ private void CopyAllContentsAsText()
             {Lang.NotSupported}
             """;
 
-        Clipboard.SetText(text);
+        await ClipboardHelper.SetTextAsync(text);
     }
 
     [RelayCommand]
-    private void CopyContentsAsText(string tag)
+    private async Task CopyContentsAsTextAsync(string tag)
     {
         if (tag == Lang.AdapterProperties_Header)
         {
-            Clipboard.SetText(Nic.ToString());
+            await ClipboardHelper.SetTextAsync(Nic.ToString());
         }
         else if (tag == Lang.TcpIPv4_Header)
         {
-            Clipboard.SetText(GetIPv4PropertyContents());
+            await ClipboardHelper.SetTextAsync(GetIPv4PropertyContents());
         }
         else if (tag == Lang.IPv4GeneralProperties_Header)
         {
-            Clipboard.SetText(IPv4AdvancedCofnig.FormatGeneralProperties());
+            await ClipboardHelper.SetTextAsync(IPv4AdvancedCofnig.FormatGeneralProperties());
         }
         else if (tag == Lang.IPv4Lifetimes_Header)
         {
-            Clipboard.SetText(IPv4AdvancedCofnig.FormatLifetimes());
+            await ClipboardHelper.SetTextAsync(IPv4AdvancedCofnig.FormatLifetimes());
         }
         else if (tag == Lang.IPv4AdvancedProperties_Header)
         {
-            Clipboard.SetText(FormatIPv4InterfaceProperties());
+            await ClipboardHelper.SetTextAsync(FormatIPv4InterfaceProperties());
         }
         else if (tag == Lang.IPv4Statistics_Header)
         {
-            Clipboard.SetText(FormatIPv4InterfaceStatistics());
+            await ClipboardHelper.SetTextAsync(FormatIPv4InterfaceStatistics());
         }
         else if (tag == Lang.TcpIPv6_Header)
         {
-            Clipboard.SetText(GetIPv6PropertyContents());
+            await ClipboardHelper.SetTextAsync(GetIPv6PropertyContents());
         }
         else if (tag == Lang.IPv6GeneralProperties_Header)
         {
-            Clipboard.SetText(IPv6AdvancedCofnig.FormatGeneralProperties());
+            await ClipboardHelper.SetTextAsync(IPv6AdvancedCofnig.FormatGeneralProperties());
         }
         else if (tag == Lang.IPv6Lifetimes_Header)
         {
-            Clipboard.SetText(IPv6AdvancedCofnig.FormatLifetimes());
+            await ClipboardHelper.SetTextAsync(IPv6AdvancedCofnig.FormatLifetimes());
         }
         else if (tag == Lang.IPv6AdvancedProperties_Header)
         {
-            Clipboard.SetText(FormatIPv6InterfaceProperties());
+            await ClipboardHelper.SetTextAsync(FormatIPv6InterfaceProperties());
         }
         else if (tag == Lang.IPv6Statistics_Header)
         {