Make sure temporary files deleted

Author: Jack251970

Plugins/Flow.Launcher.Plugin.BrowserBookmark/FirefoxBookmarkLoader.cs

@@ -41,6 +41,8 @@ protected List<Bookmark> GetBookmarksFromPath(string placesPath)
         if (string.IsNullOrEmpty(placesPath) || !File.Exists(placesPath))
             return bookmarks;
 
+        var tempDbPath = Path.Combine(_faviconCacheDir, $"tempplaces_{Guid.NewGuid()}.sqlite");
+
         try
         {
             // Try to register file monitoring
@@ -54,7 +56,6 @@ protected List<Bookmark> GetBookmarksFromPath(string placesPath)
             }
 
             // Use a copy to avoid lock issues with the original file
-            var tempDbPath = Path.Combine(_faviconCacheDir, $"tempplaces_{Guid.NewGuid()}.sqlite");
             File.Copy(placesPath, tempDbPath, true);
 
             // Connect to database and execute query
@@ -83,31 +84,32 @@ protected List<Bookmark> GetBookmarksFromPath(string placesPath)
             // https://github.com/dotnet/efcore/issues/26580
             SqliteConnection.ClearPool(dbConnection);
             dbConnection.Close();
-
-            // Delete temporary file
-            try
-            {
-                File.Delete(tempDbPath);
-            }
-            catch (Exception ex)
-            {
-                Main._context.API.LogException(ClassName, $"Failed to delete temporary favicon DB: {tempDbPath}", ex);
-            }
         }
         catch (Exception ex)
         {
             Main._context.API.LogException(ClassName, $"Failed to load Firefox bookmarks: {placesPath}", ex);
         }
 
+        // Delete temporary file
+        try
+        {
+            File.Delete(tempDbPath);
+        }
+        catch (Exception ex)
+        {
+            Main._context.API.LogException(ClassName, $"Failed to delete temporary favicon DB: {tempDbPath}", ex);
+        }
+
         return bookmarks;
     }
 
     private void LoadFaviconsFromDb(string faviconDbPath, List<Bookmark> bookmarks)
     {
+        var tempDbPath = Path.Combine(_faviconCacheDir, $"tempfavicons_{Guid.NewGuid()}.sqlite");
+
         try
         {
             // Use a copy to avoid lock issues with the original file
-            var tempDbPath = Path.Combine(_faviconCacheDir, $"tempfavicons_{Guid.NewGuid()}.sqlite");
             File.Copy(faviconDbPath, tempDbPath, true);
 
             var defaultIconPath = Path.Combine(
@@ -177,21 +179,21 @@ ORDER BY i.width DESC  -- Select largest icon available
             // https://github.com/dotnet/efcore/issues/26580
             SqliteConnection.ClearPool(connection);
             connection.Close();
-
-            // Delete temporary file
-            try
-            {
-                File.Delete(tempDbPath);
-            }
-            catch (Exception ex)
-            {
-                Main._context.API.LogException(ClassName, $"Failed to delete temporary favicon DB: {tempDbPath}", ex);
-            }
         }
         catch (Exception ex)
         {
             Main._context.API.LogException(ClassName, $"Failed to load Firefox favicon DB: {faviconDbPath}", ex);
         }
+
+        // Delete temporary file
+        try
+        {
+            File.Delete(tempDbPath);
+        }
+        catch (Exception ex)
+        {
+            Main._context.API.LogException(ClassName, $"Failed to delete temporary favicon DB: {tempDbPath}", ex);
+        }
     }
 
     private static void SaveBitmapData(byte[] imageData, string outputPath)