From 9092130350024331a1c3b34cd4f9d3932a1348c7 Mon Sep 17 00:00:00 2001
From: Tim Eisele <Ghost_of_Stone@web.de>
Date: Sat, 26 Apr 2025 17:36:17 +0200
Subject: Optimize migrations (#13855)

---
 .../Jellyfin.Database.Implementations/IJellyfinDatabaseProvider.cs | 4 ++--
 .../Jellyfin.Database.Providers.Sqlite/SqliteDatabaseProvider.cs   | 7 ++-----
 2 files changed, 4 insertions(+), 7 deletions(-)

(limited to 'src')

diff --git a/src/Jellyfin.Database/Jellyfin.Database.Implementations/IJellyfinDatabaseProvider.cs b/src/Jellyfin.Database/Jellyfin.Database.Implementations/IJellyfinDatabaseProvider.cs
index 566b521dd0..34ac7dc836 100644
--- a/src/Jellyfin.Database/Jellyfin.Database.Implementations/IJellyfinDatabaseProvider.cs
+++ b/src/Jellyfin.Database/Jellyfin.Database.Implementations/IJellyfinDatabaseProvider.cs
@@ -50,7 +50,7 @@ public interface IJellyfinDatabaseProvider
     /// <summary>
     /// Runs a full Database backup that can later be restored to.
     /// </summary>
-    /// <param name="cancellationToken">A cancelation token.</param>
+    /// <param name="cancellationToken">A cancellation token.</param>
     /// <returns>A key to identify the backup.</returns>
     /// <exception cref="NotImplementedException">May throw an NotImplementException if this operation is not supported for this database.</exception>
     Task<string> MigrationBackupFast(CancellationToken cancellationToken);
@@ -59,7 +59,7 @@ public interface IJellyfinDatabaseProvider
     /// Restores a backup that has been previously created by <see cref="MigrationBackupFast(CancellationToken)"/>.
     /// </summary>
     /// <param name="key">The key to the backup from which the current database should be restored from.</param>
-    /// <param name="cancellationToken">A cancelation token.</param>
+    /// <param name="cancellationToken">A cancellation token.</param>
     /// <returns>A <see cref="Task"/> representing the result of the asynchronous operation.</returns>
     Task RestoreBackupFast(string key, CancellationToken cancellationToken);
 }
diff --git a/src/Jellyfin.Database/Jellyfin.Database.Providers.Sqlite/SqliteDatabaseProvider.cs b/src/Jellyfin.Database/Jellyfin.Database.Providers.Sqlite/SqliteDatabaseProvider.cs
index 927ba63b97..ef1bf1769d 100644
--- a/src/Jellyfin.Database/Jellyfin.Database.Providers.Sqlite/SqliteDatabaseProvider.cs
+++ b/src/Jellyfin.Database/Jellyfin.Database.Providers.Sqlite/SqliteDatabaseProvider.cs
@@ -98,10 +98,7 @@ public sealed class SqliteDatabaseProvider : IJellyfinDatabaseProvider
         var key = DateTime.UtcNow.ToString("yyyyMMddhhmmss", CultureInfo.InvariantCulture);
         var path = Path.Combine(_applicationPaths.DataPath, "jellyfin.db");
         var backupFile = Path.Combine(_applicationPaths.DataPath, BackupFolderName);
-        if (!Directory.Exists(backupFile))
-        {
-            Directory.CreateDirectory(backupFile);
-        }
+        Directory.CreateDirectory(backupFile);
 
         backupFile = Path.Combine(backupFile, $"{key}_jellyfin.db");
         File.Copy(path, backupFile);
@@ -118,7 +115,7 @@ public sealed class SqliteDatabaseProvider : IJellyfinDatabaseProvider
 
         if (!File.Exists(backupFile))
         {
-            _logger.LogCritical("Tried to restore a backup that does not exist.");
+            _logger.LogCritical("Tried to restore a backup that does not exist: {Key}", key);
             return Task.CompletedTask;
         }
 
-- 
cgit v1.2.3