Fixed: Extra new line in release restrictions causing all releases to be rejected

pull/6/head
Mark McDowall 11 years ago
parent 2dd7fb9fdf
commit 885872ff3a

@ -49,5 +49,12 @@ namespace NzbDrone.Core.Test.DecisionEngineTests
Mocker.GetMock<IConfigService>().SetupGet(c => c.ReleaseRestrictions).Returns(restrictions); Mocker.GetMock<IConfigService>().SetupGet(c => c.ReleaseRestrictions).Returns(restrictions);
Subject.IsSatisfiedBy(_parseResult, null).Should().BeTrue(); Subject.IsSatisfiedBy(_parseResult, null).Should().BeTrue();
} }
[Test]
public void should_not_try_to_find_empty_string_as_a_match()
{
Mocker.GetMock<IConfigService>().SetupGet(c => c.ReleaseRestrictions).Returns("test\n");
Subject.IsSatisfiedBy(_parseResult, null).Should().BeTrue();
}
} }
} }

@ -231,8 +231,8 @@ namespace NzbDrone.Core.Configuration
public string ReleaseRestrictions public string ReleaseRestrictions
{ {
get { return GetValue("ReleaseRestrictions", String.Empty); } get { return GetValue("ReleaseRestrictions", String.Empty).Trim('\r', '\n'); }
set { SetValue("ReleaseRestrictions", value); } set { SetValue("ReleaseRestrictions", value.Trim('\r', '\n')); }
} }
public Int32 RssSyncInterval public Int32 RssSyncInterval

@ -41,6 +41,8 @@ namespace NzbDrone.Core.DecisionEngine.Specifications
foreach (var restriction in restrictions) foreach (var restriction in restrictions)
{ {
if (String.IsNullOrWhiteSpace(restriction)) continue;
if (subject.Release.Title.ToLowerInvariant().Contains(restriction.ToLowerInvariant())) if (subject.Release.Title.ToLowerInvariant().Contains(restriction.ToLowerInvariant()))
{ {
_logger.Trace("{0} is restricted: {1}", subject, restriction); _logger.Trace("{0} is restricted: {1}", subject, restriction);

Loading…
Cancel
Save