empty string case should not be only for the contains case

pull/2/head
geogolem 7 years ago
parent ff11388009
commit 7d394dcff2

@ -141,21 +141,22 @@ namespace NzbDrone.Api.Movie
} }
if (pagingResource.FilterKey == "title") if (pagingResource.FilterKey == "title")
{
if (pagingResource.FilterType == "contains")
{ {
if (pagingResource.FilterValue == string.Empty || pagingResource.FilterValue == null) if (pagingResource.FilterValue == string.Empty || pagingResource.FilterValue == null)
{ {
pagingSpec.FilterExpression = v => true; pagingSpec.FilterExpression = v => true;
} }
else else
{
if (pagingResource.FilterType == "contains")
{ {
pagingSpec.FilterExpression = v => v.CleanTitle.Contains(pagingResource.FilterValue); pagingSpec.FilterExpression = v => v.CleanTitle.Contains(pagingResource.FilterValue);
} }
} else else
{ {
pagingSpec.FilterExpression = v => v.CleanTitle == pagingResource.FilterValue; pagingSpec.FilterExpression = v => v.CleanTitle == pagingResource.FilterValue;
} }
}
} }
return ApplyToPage(_moviesService.Paged, pagingSpec, MovieResourceMapper.ToResource); return ApplyToPage(_moviesService.Paged, pagingSpec, MovieResourceMapper.ToResource);

Loading…
Cancel
Save