소스 검색

Extract "migrations" config name to a proper constant

Vasily 5 년 전
부모
커밋
ccafebca68
2개의 변경된 파일8개의 추가작업 그리고 3개의 파일을 삭제
  1. 2 2
      Jellyfin.Server/Migrations/MigrationRunner.cs
  2. 6 1
      Jellyfin.Server/Migrations/MigrationsListStore.cs

+ 2 - 2
Jellyfin.Server/Migrations/MigrationRunner.cs

@@ -27,7 +27,7 @@ namespace Jellyfin.Server.Migrations
         public static void Run(CoreAppHost host, ILoggerFactory loggerFactory)
         public static void Run(CoreAppHost host, ILoggerFactory loggerFactory)
         {
         {
             var logger = loggerFactory.CreateLogger<MigrationRunner>();
             var logger = loggerFactory.CreateLogger<MigrationRunner>();
-            var migrationOptions = ((IConfigurationManager)host.ServerConfigurationManager).GetConfiguration<MigrationOptions>("migrations");
+            var migrationOptions = ((IConfigurationManager)host.ServerConfigurationManager).GetConfiguration<MigrationOptions>(MigrationsListStore.StoreKey);
             var applied = migrationOptions.Applied.ToList();
             var applied = migrationOptions.Applied.ToList();
 
 
             for (var i = 0; i < Migrations.Length; i++)
             for (var i = 0; i < Migrations.Length; i++)
@@ -56,7 +56,7 @@ namespace Jellyfin.Server.Migrations
             {
             {
                 logger.LogInformation("Some migrations were run, saving the state");
                 logger.LogInformation("Some migrations were run, saving the state");
                 migrationOptions.Applied = applied.ToArray();
                 migrationOptions.Applied = applied.ToArray();
-                host.ServerConfigurationManager.SaveConfiguration("migrations", migrationOptions);
+                host.ServerConfigurationManager.SaveConfiguration(MigrationsListStore.StoreKey, migrationOptions);
             }
             }
         }
         }
     }
     }

+ 6 - 1
Jellyfin.Server/Migrations/MigrationsListStore.cs

@@ -7,13 +7,18 @@ namespace Jellyfin.Server.Migrations
     /// </summary>
     /// </summary>
     public class MigrationsListStore : ConfigurationStore
     public class MigrationsListStore : ConfigurationStore
     {
     {
+        /// <summary>
+        /// The name of the configuration in the storage.
+        /// </summary>
+        public static readonly string StoreKey = "migrations";
+
         /// <summary>
         /// <summary>
         /// Initializes a new instance of the <see cref="MigrationsListStore"/> class.
         /// Initializes a new instance of the <see cref="MigrationsListStore"/> class.
         /// </summary>
         /// </summary>
         public MigrationsListStore()
         public MigrationsListStore()
         {
         {
             ConfigurationType = typeof(MigrationOptions);
             ConfigurationType = typeof(MigrationOptions);
-            Key = "migrations";
+            Key = StoreKey;
         }
         }
     }
     }
 }
 }