Avoid import loop for already imported movies

(cherry picked from commit b183743d9f0a0b15e4a9db0a9d3d2d1c238b0d9c) Closes #9325
Radarr · Feb 2, 2024 · 20ac0bb · 20ac0bb
1 parent 9ffa1cc
commit 20ac0bb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/...ne.Core.Test/MediaFiles/MovieImport/Specifications/AlreadyImportedSpecificationFixture.cs b/...ne.Core.Test/MediaFiles/MovieImport/Specifications/AlreadyImportedSpecificationFixture.cs
@@ -115,7 +115,7 @@ public void should_reject_if_episode_imported_after_being_grabbed()
 
             GivenHistory(history);
 
-            Subject.IsSatisfiedBy(_localMovie, _downloadClientItem).Accepted.Should().BeTrue();
+            Subject.IsSatisfiedBy(_localMovie, _downloadClientItem).Accepted.Should().BeFalse();
         }
     }
 }
diff --git a/src/NzbDrone.Core/MediaFiles/MovieImport/Specifications/AlreadyImportedSpecification.cs b/src/NzbDrone.Core/MediaFiles/MovieImport/Specifications/AlreadyImportedSpecification.cs
@@ -67,6 +67,11 @@ public Decision IsSatisfiedBy(LocalMovie localMovie, DownloadClientItem download
                     return Decision.Reject("Movie file already imported at {0}", lastImported.Date.ToLocalTime());
                 }
             }
+            else
+            {
+                _logger.Debug("Movie file previously imported at {0}", lastImported.Date);
+                return Decision.Reject("Movie file already imported at {0}", lastImported.Date.ToLocalTime());
+            }
 
             return Decision.Accept();
         }