mirror/Lidarr
1
0
Fork 0
mirror of https://github.com/lidarr/Lidarr synced 2025-02-25 15:22:42 +00:00
Code Issues Releases Wiki Activity

Simplify ManualImportModule null check

Browse source
This commit is contained in:
Qstick 2020-09-06 22:51:25 -04:00
parent e8e4d76d73
commit 6713525757
2 changed files with 16 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/Lidarr.Api.V1/ManualImport/ManualImportModule.cs
View file

@ -47,14 +47,11 @@ private List<ManualImportResource> GetMediaFiles()
var downloadId = (string)Request.Query.downloadId; var downloadId = (string)Request.Query.downloadId;
NzbDrone.Core.Music.Artist artist = null; NzbDrone.Core.Music.Artist artist = null;
var artistIdQuery = Request.Query.artistId; var artistIdQuery = Request.GetNullableIntegerQueryParameter("artistId", null);
if (artistIdQuery.HasValue)
if (artistIdQuery.HasValue && artistIdQuery.Value > 0)
{ {
var artistId = Convert.ToInt32(artistIdQuery.Value); artist = _artistService.GetArtist(Convert.ToInt32(artistIdQuery.Value));
if (artistId > 0)
{
artist = _artistService.GetArtist(Convert.ToInt32(artistIdQuery.Value));
}
} }
var filter = Request.GetBooleanQueryParameter("filterExistingFiles", true) ? FilterFilesType.Matched : FilterFilesType.None; var filter = Request.GetBooleanQueryParameter("filterExistingFiles", true) ? FilterFilesType.Matched : FilterFilesType.None;

12
src/Lidarr.Http/Extensions/RequestExtensions.cs
View file

@ -78,5 +78,17 @@ public static Guid GetGuidQueryParameter(this Request request, string parameter,
return defaultValue; return defaultValue;
} }
public static int? GetNullableIntegerQueryParameter(this Request request, string parameter, int? defaultValue = null)
{
var parameterValue = request.Query[parameter];
if (parameterValue.HasValue)
{
return int.Parse(parameterValue.Value);
}
return defaultValue;
}
} }
} }