Code Quality: Allow multiple exception types to be ignored

src/Files.App/Extensions/DispatcherQueueExtensions.cs

@@ -26,7 +26,7 @@ public static Task EnqueueOrInvokeAsync(this DispatcherQueue? dispatcher, Func<T
 				}
 
 				return function();
-			}, App.Logger, typeof(COMException));
+			}, App.Logger, typeof(COMException), typeof(InvalidOperationException));
 		}
 
 		public static Task<T?> EnqueueOrInvokeAsync<T>(this DispatcherQueue? dispatcher, Func<Task<T>> function, DispatcherQueuePriority priority = DispatcherQueuePriority.Normal)
@@ -47,7 +47,7 @@ public static Task EnqueueOrInvokeAsync(this DispatcherQueue? dispatcher, Func<T
 				}
 
 				return function();
-			}, App.Logger, typeof(COMException));
+			}, App.Logger, typeof(COMException), typeof(InvalidOperationException));
 		}
 
 		public static Task EnqueueOrInvokeAsync(this DispatcherQueue? dispatcher, Action function, DispatcherQueuePriority priority = DispatcherQueuePriority.Normal)
@@ -69,7 +69,7 @@ public static Task EnqueueOrInvokeAsync(this DispatcherQueue? dispatcher, Action
 
 				function();
 				return Task.CompletedTask;
-			}, App.Logger, typeof(COMException));
+			}, App.Logger, typeof(COMException), typeof(InvalidOperationException));
 		}
 
 		public static Task<T?> EnqueueOrInvokeAsync<T>(this DispatcherQueue? dispatcher, Func<T> function, DispatcherQueuePriority priority = DispatcherQueuePriority.Normal)
@@ -90,7 +90,7 @@ public static Task EnqueueOrInvokeAsync(this DispatcherQueue? dispatcher, Action
 				}
 
 				return Task.FromResult(function());
-			}, App.Logger, typeof(COMException));
+			}, App.Logger, typeof(COMException), typeof(InvalidOperationException));
 		}
 
 	}

src/Files.Shared/Extensions/SafetyExtensions.cs

@@ -9,7 +9,7 @@
 {
 	public static class SafetyExtensions
 	{
-		public static bool IgnoreExceptions(Action action, ILogger? logger = null, Type? exceptionToIgnore = null)
+		public static bool IgnoreExceptions(Action action, ILogger? logger = null, params Type[]? exceptionsToIgnore)
 		{
 			try
 			{
@@ -18,18 +18,30 @@
 			}
 			catch (Exception ex)
 			{
-				if (exceptionToIgnore is null || exceptionToIgnore.IsAssignableFrom(ex.GetType()))
+				bool shouldIgnore = exceptionsToIgnore is null || exceptionsToIgnore.Length == 0;
+				if (!shouldIgnore)
 				{
-					logger?.LogInformation(ex, ex.Message);
+					foreach (var type in exceptionsToIgnore)
+					{
+						if (type.IsAssignableFrom(ex.GetType()))
+						{
+							shouldIgnore = true;
+							break;
+						}
+					}
+				}
 
+				if (shouldIgnore)
+				{
+					logger?.LogInformation(ex, ex.Message);
 					return false;
 				}
-				else
-					throw;
+
+				throw;
 			}
 		}
 
-		public static async Task<bool> IgnoreExceptions(Func<Task> action, ILogger? logger = null, Type? exceptionToIgnore = null)
+		public static async Task<bool> IgnoreExceptions(Func<Task> action, ILogger? logger = null, params Type[]? exceptionsToIgnore)
 		{
 			try
 			{
@@ -39,52 +51,88 @@
 			}
 			catch (Exception ex)
 			{
-				if (exceptionToIgnore is null || exceptionToIgnore.IsAssignableFrom(ex.GetType()))
+				bool shouldIgnore = exceptionsToIgnore is null || exceptionsToIgnore.Length == 0;
+				if (!shouldIgnore)
 				{
-					logger?.LogInformation(ex, ex.Message);
+					foreach (var type in exceptionsToIgnore)
+					{
+						if (type.IsAssignableFrom(ex.GetType()))
+						{
+							shouldIgnore = true;
+							break;
+						}
+					}
+				}
 
+				if (shouldIgnore)
+				{
+					logger?.LogInformation(ex, ex.Message);
 					return false;
 				}
-				else
-					throw;
+
+				throw;
 			}
 		}
 
-		public static T? IgnoreExceptions<T>(Func<T> action, ILogger? logger = null, Type? exceptionToIgnore = null)
+		public static T? IgnoreExceptions<T>(Func<T> action, ILogger? logger = null, params Type[]? exceptionsToIgnore)
 		{
 			try
 			{
 				return action();
 			}
 			catch (Exception ex)
 			{
-				if (exceptionToIgnore is null || exceptionToIgnore.IsAssignableFrom(ex.GetType()))
+				bool shouldIgnore = exceptionsToIgnore is null || exceptionsToIgnore.Length == 0;
+				if (!shouldIgnore)
 				{
-					logger?.LogInformation(ex, ex.Message);
+					foreach (var type in exceptionsToIgnore)
+					{
+						if (type.IsAssignableFrom(ex.GetType()))
+						{
+							shouldIgnore = true;
+							break;
+						}
+					}
+				}
 
+				if (shouldIgnore)
+				{
+					logger?.LogInformation(ex, ex.Message);
 					return default;
 				}
-				else
-					throw;
+
+				throw;
 			}
 		}
 
-		public static async Task<T?> IgnoreExceptions<T>(Func<Task<T>> action, ILogger? logger = null, Type? exceptionToIgnore = null)
+		public static async Task<T?> IgnoreExceptions<T>(Func<Task<T>> action, ILogger? logger = null, params Type[]? exceptionsToIgnore)
 		{
 			try
 			{
 				return await action();
 			}
 			catch (Exception ex)
 			{
-				if (exceptionToIgnore is null || exceptionToIgnore.IsAssignableFrom(ex.GetType()))
+				bool shouldIgnore = exceptionsToIgnore is null || exceptionsToIgnore.Length == 0;
+				if (!shouldIgnore)
 				{
-					logger?.LogInformation(ex, ex.Message);
+					foreach (var type in exceptionsToIgnore)
+					{
+						if (type.IsAssignableFrom(ex.GetType()))
+						{
+							shouldIgnore = true;
+							break;
+						}
+					}
+				}
 
+				if (shouldIgnore)
+				{
+					logger?.LogInformation(ex, ex.Message);
 					return default;
 				}
-				else
-					throw;
+
+				throw;
 			}
 		}