Apply suggestions from code review
Co-Authored-By: cvium <cvium@users.noreply.github.com>
This commit is contained in:
parent
83d98ac92d
commit
da2c7db0df
|
@ -67,7 +67,7 @@ namespace MediaBrowser.Providers.TV.TheTVDB
|
||||||
cancellationToken);
|
cancellationToken);
|
||||||
if (string.IsNullOrEmpty(episodeTvdbId))
|
if (string.IsNullOrEmpty(episodeTvdbId))
|
||||||
{
|
{
|
||||||
_logger.LogError("Episode {SeasonNumber}x{EpisodeNumber}found for series {SeriesTvdbId}",
|
_logger.LogError("Episode {SeasonNumber}x{EpisodeNumber} not found for series {SeriesTvdbId}",
|
||||||
seasonNumber, episodeNumber);
|
seasonNumber, episodeNumber);
|
||||||
return imageResult;
|
return imageResult;
|
||||||
}
|
}
|
||||||
|
|
|
@ -36,7 +36,7 @@ namespace MediaBrowser.Providers.TV
|
||||||
var list = new List<RemoteSearchResult>();
|
var list = new List<RemoteSearchResult>();
|
||||||
|
|
||||||
// The search query must either provide an episode number or date
|
// The search query must either provide an episode number or date
|
||||||
// TODO premieredate functionality is ded, could grab all episodes and search for it
|
// TODO premieredate functionality is dead, could grab all episodes and search for it
|
||||||
if (!searchInfo.IndexNumber.HasValue || !searchInfo.PremiereDate.HasValue)
|
if (!searchInfo.IndexNumber.HasValue || !searchInfo.PremiereDate.HasValue)
|
||||||
{
|
{
|
||||||
return list;
|
return list;
|
||||||
|
@ -52,7 +52,7 @@ namespace MediaBrowser.Providers.TV
|
||||||
episodeTvdbId = await _tvDbClientManager.GetEpisodeTvdbId(searchInfo, cancellationToken);
|
episodeTvdbId = await _tvDbClientManager.GetEpisodeTvdbId(searchInfo, cancellationToken);
|
||||||
if (string.IsNullOrEmpty(episodeTvdbId))
|
if (string.IsNullOrEmpty(episodeTvdbId))
|
||||||
{
|
{
|
||||||
_logger.LogError("Episode {SeasonNumber}x{EpisodeNumber} found for series {SeriesTvdbId}",
|
_logger.LogError("Episode {SeasonNumber}x{EpisodeNumber} not found for series {SeriesTvdbId}",
|
||||||
searchInfo.ParentIndexNumber, searchInfo.IndexNumber);
|
searchInfo.ParentIndexNumber, searchInfo.IndexNumber);
|
||||||
return list;
|
return list;
|
||||||
}
|
}
|
||||||
|
@ -106,7 +106,7 @@ namespace MediaBrowser.Providers.TV
|
||||||
tvdbId = await _tvDbClientManager.GetEpisodeTvdbId(searchInfo, cancellationToken);
|
tvdbId = await _tvDbClientManager.GetEpisodeTvdbId(searchInfo, cancellationToken);
|
||||||
if (string.IsNullOrEmpty(tvdbId))
|
if (string.IsNullOrEmpty(tvdbId))
|
||||||
{
|
{
|
||||||
_logger.LogError("Episode {SeasonNumber}x{EpisodeNumber}found for series {SeriesTvdbId}",
|
_logger.LogError("Episode {SeasonNumber}x{EpisodeNumber} not found for series {SeriesTvdbId}",
|
||||||
searchInfo.ParentIndexNumber, searchInfo.IndexNumber, tvdbId);
|
searchInfo.ParentIndexNumber, searchInfo.IndexNumber, tvdbId);
|
||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
|
|
|
@ -103,7 +103,7 @@ namespace MediaBrowser.Providers.TV.TheTVDB
|
||||||
VoteCount = image.RatingsInfo.Count,
|
VoteCount = image.RatingsInfo.Count,
|
||||||
Url = TVUtils.BannerUrl + image.FileName,
|
Url = TVUtils.BannerUrl + image.FileName,
|
||||||
ProviderName = ProviderName,
|
ProviderName = ProviderName,
|
||||||
Language = languages.FirstOrDefault(l => l.Id == image.LanguageId)?.Abbreviation,
|
Language = languages.FirstOrDefault(lang => lang.Id == image.LanguageId)?.Abbreviation,
|
||||||
ThumbnailUrl = TVUtils.BannerUrl + image.Thumbnail
|
ThumbnailUrl = TVUtils.BannerUrl + image.Thumbnail
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user