add migration for missing media_id column on translation_requests

Author: rowanfuchs

Lingarr.Core/LingarrVersion.cs

@@ -5,7 +5,7 @@ namespace Lingarr.Core;
 public static class LingarrVersion
 {
     public const string Name = "Lingarr";
-    public const string Number = "1.0.6";
+    public const string Number = "1.0.7";
 
     public static async Task<VersionInfo> CheckForUpdates(object? lingarrApiService = null)
     {

Lingarr.Migrations/Migrations/M0006_AddTranslationRequestMediaId.cs

@@ -0,0 +1,21 @@
+using FluentMigrator;
+
+namespace Lingarr.Migrations.Migrations;
+
+[Migration(6)]
+public class M0006_AddTranslationRequestMediaId : Migration
+{
+    public override void Up()
+    {
+        if (!Schema.Table("translation_requests").Column("media_id").Exists())
+        {
+            Alter.Table("translation_requests")
+                .AddColumn("media_id").AsInt32().Nullable();
+        }
+    }
+
+    public override void Down()
+    {
+        Delete.Column("media_id").FromTable("translation_requests");
+    }
+}

Lingarr.Server/Services/MediaSubtitleProcessor.cs

@@ -111,7 +111,7 @@ private async Task<bool> ProcessSubtitles(
             return false;
         }
 
-        var languagesToTranslate = targetLanguages.Except(selected.AvailableLanguages);
+        var languagesToTranslate = targetLanguages.Except(selected.AvailableLanguages).ToList();
         if (ignoreCaptions == "true")
         {
             var targetLanguagesWithCaptions = subtitles
@@ -130,24 +130,21 @@ private async Task<bool> ProcessSubtitles(
             }
         }
 
-        // Exclude languages that already have a pending, in-progress, or completed translation request
-        // to prevent duplicate translations (see: https://github.com/lingarr-translate/lingarr/issues/312)
         var activeStatuses = new[]
         {
             TranslationStatus.Pending,
             TranslationStatus.InProgress,
             TranslationStatus.Completed
         };
-        var existingLanguages = await _dbContext.TranslationRequests
-            .Where(r => r.MediaId == _media.Id
-                        && r.MediaType == _mediaType
-                        && activeStatuses.Contains(r.Status))
-            .Select(r => r.TargetLanguage)
+        var existingTranslationRequests = await _dbContext.TranslationRequests
+            .Where(translationRequest => translationRequest.MediaId == _media.Id
+                                         && translationRequest.MediaType == _mediaType
+                                         && activeStatuses.Contains(translationRequest.Status))
+            .Select(translationRequest => translationRequest.TargetLanguage)
             .Distinct()
             .ToListAsync();
 
-        languagesToTranslate = languagesToTranslate.Except(existingLanguages);
-        
+        languagesToTranslate = languagesToTranslate.Except(existingTranslationRequests).ToList();
         if (!languagesToTranslate.Any())
         {
             await UpdateHash();