Fixed: Allow repeated import attempts until downloaded files appear

pull/2370/head
ta264 3 years ago
parent f7925ff5d6
commit 9f1d4446e8

@ -167,7 +167,7 @@ namespace NzbDrone.Core.Test.Download.CompletedDownloadServiceTests
Subject.Import(_trackedDownload); Subject.Import(_trackedDownload);
_trackedDownload.State.Should().Be(TrackedDownloadState.Importing); _trackedDownload.State.Should().Be(TrackedDownloadState.ImportPending);
} }
[Test] [Test]

@ -109,6 +109,7 @@ namespace NzbDrone.Core.Download
if (importResults.Empty()) if (importResults.Empty())
{ {
trackedDownload.Warn("No files found are eligible for import in {0}", outputPath); trackedDownload.Warn("No files found are eligible for import in {0}", outputPath);
trackedDownload.State = TrackedDownloadState.ImportPending;
return; return;
} }
@ -116,10 +117,6 @@ namespace NzbDrone.Core.Download
importResults.Count(c => c.Result == ImportResultType.Imported) >= importResults.Count(c => c.Result == ImportResultType.Imported) >=
Math.Max(1, trackedDownload.RemoteAlbum.Albums.Sum(x => x.AlbumReleases.Value.Where(y => y.Monitored).Sum(z => z.TrackCount))); Math.Max(1, trackedDownload.RemoteAlbum.Albums.Sum(x => x.AlbumReleases.Value.Where(y => y.Monitored).Sum(z => z.TrackCount)));
Console.WriteLine($"allimported: {allTracksImported}");
Console.WriteLine($"count: {importResults.Count(c => c.Result == ImportResultType.Imported)}");
Console.WriteLine($"max: {Math.Max(1, trackedDownload.RemoteAlbum.Albums.Sum(x => x.AlbumReleases.Value.Where(y => y.Monitored).Sum(z => z.TrackCount)))}");
if (allTracksImported) if (allTracksImported)
{ {
trackedDownload.State = TrackedDownloadState.Imported; trackedDownload.State = TrackedDownloadState.Imported;
@ -148,11 +145,6 @@ namespace NzbDrone.Core.Download
trackedDownload.State = TrackedDownloadState.ImportPending; trackedDownload.State = TrackedDownloadState.ImportPending;
if (importResults.Empty())
{
trackedDownload.Warn("No files found are eligible for import in {0}", outputPath);
}
if (importResults.Any(c => c.Result != ImportResultType.Imported)) if (importResults.Any(c => c.Result != ImportResultType.Imported))
{ {
trackedDownload.State = TrackedDownloadState.ImportFailed; trackedDownload.State = TrackedDownloadState.ImportFailed;

Loading…
Cancel
Save