From e7fa4cba194c8e90216f99257653880603e9e734 Mon Sep 17 00:00:00 2001 From: Leonardo Galli Date: Tue, 10 Jan 2017 21:25:36 +0100 Subject: [PATCH] Fix download rejections being ignored. --- src/NzbDrone.Core/DecisionEngine/DownloadDecisionMaker.cs | 4 ++-- src/NzbDrone.Core/Download/ProcessDownloadDecisions.cs | 6 ++++++ 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/src/NzbDrone.Core/DecisionEngine/DownloadDecisionMaker.cs b/src/NzbDrone.Core/DecisionEngine/DownloadDecisionMaker.cs index e2d171ee4..5e3b0d824 100644 --- a/src/NzbDrone.Core/DecisionEngine/DownloadDecisionMaker.cs +++ b/src/NzbDrone.Core/DecisionEngine/DownloadDecisionMaker.cs @@ -83,7 +83,7 @@ namespace NzbDrone.Core.DecisionEngine { if (parsedEpisodeInfo.Quality.HardcodedSubs.IsNotNullOrWhiteSpace()) { - remoteEpisode.DownloadAllowed = true; + remoteEpisode.DownloadAllowed = false; decision = new DownloadDecision(remoteEpisode, new Rejection("Hardcoded subs found: " + parsedEpisodeInfo.Quality.HardcodedSubs)); } else @@ -257,7 +257,7 @@ namespace NzbDrone.Core.DecisionEngine } catch (NotImplementedException e) { - _logger.Info("Spec " + spec.GetType().Name + " does not care about movies."); + _logger.Trace("Spec " + spec.GetType().Name + " does not care about movies."); } catch (Exception e) { diff --git a/src/NzbDrone.Core/Download/ProcessDownloadDecisions.cs b/src/NzbDrone.Core/Download/ProcessDownloadDecisions.cs index adadebdee..70aba1c41 100644 --- a/src/NzbDrone.Core/Download/ProcessDownloadDecisions.cs +++ b/src/NzbDrone.Core/Download/ProcessDownloadDecisions.cs @@ -51,6 +51,12 @@ namespace NzbDrone.Core.Download continue; } + if (report.Rejections.Any()) + { + _logger.Debug("Rejecting release {0} because {1}", report.ToString(), report.Rejections.First().Reason); + continue; + } + if (remoteMovie == null || remoteMovie.Movie == null) { continue;