From 20ac0bb0e150d8909cb009dce2c644874835b282 Mon Sep 17 00:00:00 2001 From: Bogdan Date: Sun, 29 Oct 2023 00:45:47 +0300 Subject: [PATCH] Avoid import loop for already imported movies (cherry picked from commit b183743d9f0a0b15e4a9db0a9d3d2d1c238b0d9c) Closes #9325 --- .../Specifications/AlreadyImportedSpecificationFixture.cs | 2 +- .../Specifications/AlreadyImportedSpecification.cs | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/NzbDrone.Core.Test/MediaFiles/MovieImport/Specifications/AlreadyImportedSpecificationFixture.cs b/src/NzbDrone.Core.Test/MediaFiles/MovieImport/Specifications/AlreadyImportedSpecificationFixture.cs index 6e9033078c9..fc89c43af75 100644 --- a/src/NzbDrone.Core.Test/MediaFiles/MovieImport/Specifications/AlreadyImportedSpecificationFixture.cs +++ b/src/NzbDrone.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 index a466c018a7a..ea2bfbbdd43 100644 --- 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(); }