Fixed: Don't process tracked download if RemoteAlbum is null (#716)

Fixes LIDARR-1V4
6 years ago · 6afece237c
parent 57df287547
commit 6afece237c
2 changed files with 4 additions and 2 deletions
--- a/src/NzbDrone.Core/Download/CompletedDownloadService.cs
+++ b/src/NzbDrone.Core/Download/CompletedDownloadService.cs
@ -50,7 +50,8 @@ namespace NzbDrone.Core.Download

        public void Process(TrackedDownload trackedDownload, bool ignoreWarnings = false)
        {
-            if (trackedDownload.DownloadItem.Status != DownloadItemStatus.Completed)
+            if (trackedDownload.DownloadItem.Status != DownloadItemStatus.Completed ||
+                trackedDownload.RemoteAlbum == null)
            {
                return;
            }
@ -80,7 +81,7 @@ namespace NzbDrone.Core.Download
                    return;
                }

-                var artist = trackedDownload.RemoteAlbum?.Artist;
+                var artist = trackedDownload.RemoteAlbum.Artist;

                if (artist == null)
                {
--- a/src/NzbDrone.Core/Download/TrackedDownloads/TrackedDownloadService.cs
+++ b/src/NzbDrone.Core/Download/TrackedDownloads/TrackedDownloadService.cs
@ -156,6 +156,7 @@ namespace NzbDrone.Core.Download.TrackedDownloads
                if (trackedDownload.RemoteAlbum == null)
                {
                    _logger.Trace("No Album found for download '{0}'", trackedDownload.DownloadItem.Title);
+                    trackedDownload.Warn("No Album found for download '{0}'", trackedDownload.DownloadItem.Title);
                }
            }
            catch (Exception e)