From 77ad7f6139e4911168e2199fe48e78bf7fdddbf1 Mon Sep 17 00:00:00 2001 From: JPVenson Date: Mon, 7 Apr 2025 22:42:01 +0200 Subject: [PATCH] Fix the migration as the new constraint now uses Value as unique key (#13867) --- Jellyfin.Server/Migrations/Routines/MigrateLibraryDb.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Jellyfin.Server/Migrations/Routines/MigrateLibraryDb.cs b/Jellyfin.Server/Migrations/Routines/MigrateLibraryDb.cs index 3fc9bea842..105fd555f6 100644 --- a/Jellyfin.Server/Migrations/Routines/MigrateLibraryDb.cs +++ b/Jellyfin.Server/Migrations/Routines/MigrateLibraryDb.cs @@ -138,14 +138,14 @@ internal class MigrateLibraryDb : IDatabaseMigrationRoutine """; // EFCores local lookup sucks. We cannot use context.ItemValues.Local here because its just super slow. - var localItems = new Dictionary<(int Type, string CleanValue), (Database.Implementations.Entities.ItemValue ItemValue, List ItemIds)>(); + var localItems = new Dictionary<(int Type, string Value), (Database.Implementations.Entities.ItemValue ItemValue, List ItemIds)>(); using (new TrackedMigrationStep("loading ItemValues", _logger)) { foreach (SqliteDataReader dto in connection.Query(itemValueQuery)) { var itemId = dto.GetGuid(0); var entity = GetItemValue(dto); - var key = ((int)entity.Type, entity.CleanValue); + var key = ((int)entity.Type, entity.Value); if (!localItems.TryGetValue(key, out var existing)) { localItems[key] = existing = (entity, []);