diff --git a/src/NzbDrone.Core/MetadataSource/SkyHook/SkyHookProxy.cs b/src/NzbDrone.Core/MetadataSource/SkyHook/SkyHookProxy.cs index fceb7fed3..366df1f10 100644 --- a/src/NzbDrone.Core/MetadataSource/SkyHook/SkyHookProxy.cs +++ b/src/NzbDrone.Core/MetadataSource/SkyHook/SkyHookProxy.cs @@ -162,8 +162,9 @@ namespace NzbDrone.Core.MetadataSource.SkyHook movie.TitleSlug += "-" + movie.TmdbId.ToString(); - movie.Images.Add(_configService.GetCoverForURL(resource.poster_path, MediaCoverTypes.Poster));//TODO: Update to load image specs from tmdb page! - movie.Images.Add(_configService.GetCoverForURL(resource.backdrop_path, MediaCoverTypes.Fanart)); + movie.Images.Add(MapImage(resource.poster_path, MediaCoverTypes.Poster));//TODO: Update to load image specs from tmdb page! + movie.Images.Add(MapImage(resource.backdrop_path, MediaCoverTypes.Fanart)); + movie.Runtime = resource.runtime; //foreach(Title title in resource.alternative_titles.titles) @@ -422,7 +423,7 @@ namespace NzbDrone.Core.MetadataSource.SkyHook { return new List { GetMovieInfo(parserResult.ImdbId) }; } - catch (Exception e) + catch (Exception) { return new List(); } @@ -540,13 +541,11 @@ namespace NzbDrone.Core.MetadataSource.SkyHook } } } - catch (Exception ex) + catch (Exception) { _logger.Debug("Not a valid date time."); } - - var now = DateTime.Now; //handle the case when we have both theatrical and physical release dates if (imdbMovie.InCinemas.HasValue && imdbMovie.PhysicalRelease.HasValue) @@ -588,10 +587,9 @@ namespace NzbDrone.Core.MetadataSource.SkyHook try { - var imdbPoster = _configService.GetCoverForURL(result.poster_path, MediaCoverTypes.Poster); - imdbMovie.Images.Add(imdbPoster); + imdbMovie.Images.AddIfNotNull(MapImage(result.poster_path, MediaCoverTypes.Poster)); } - catch (Exception e) + catch (Exception) { _logger.Debug(result); } @@ -615,61 +613,14 @@ namespace NzbDrone.Core.MetadataSource.SkyHook return null; } - private static Actor MapActors(ActorResource arg) - { - var newActor = new Actor - { - Name = arg.Name, - Character = arg.Character - }; - - if (arg.Image != null) - { - newActor.Images = new List - { - new MediaCover.MediaCover(MediaCoverTypes.Headshot, arg.Image) - }; - } - - return newActor; - } - - private static Ratings MapRatings(RatingResource rating) + private MediaCover.MediaCover MapImage(string path, MediaCoverTypes type) { - if (rating == null) + if (path.IsNotNullOrWhiteSpace()) { - return new Ratings(); + return _configService.GetCoverForURL(path, type); } - return new Ratings - { - Votes = rating.Count, - Value = rating.Value - }; - } - - private static MediaCover.MediaCover MapImage(ImageResource arg) - { - return new MediaCover.MediaCover - { - Url = arg.Url, - CoverType = MapCoverType(arg.CoverType) - }; - } - - private static MediaCoverTypes MapCoverType(string coverType) - { - switch (coverType.ToLower()) - { - case "poster": - return MediaCoverTypes.Poster; - case "banner": - return MediaCoverTypes.Banner; - case "fanart": - return MediaCoverTypes.Fanart; - default: - return MediaCoverTypes.Unknown; - } + return null; } public Movie MapMovieToTmdbMovie(Movie movie)