reduce requests when images already exist

4 files changed, 21 insertions, 2 deletions

diff --git a/MediaBrowser.Controller/Providers/Movies/FanArtMovieProvider.cs b/MediaBrowser.Controller/Providers/Movies/FanArtMovieProvider.cs
index ce9652b59..f1e670862 100644
--- a/MediaBrowser.Controller/Providers/Movies/FanArtMovieProvider.cs
+++ b/MediaBrowser.Controller/Providers/Movies/FanArtMovieProvider.cs
@@ -129,6 +129,16 @@ namespace MediaBrowser.Controller.Providers.Movies
                 return false;
             }
 
+            if (item.HasImage(ImageType.Art) &&
+                item.HasImage(ImageType.Logo) &&
+                item.HasImage(ImageType.Disc) &&
+                item.HasImage(ImageType.Banner) &&
+                item.HasImage(ImageType.Thumb) &&
+                item.BackdropImagePaths.Count > 0)
+            {
+                return false;
+            }
+
             // Refresh if tmdb id has changed
             if (providerInfo.Data != GetComparisonData(item.GetProviderId(MetadataProviders.Tmdb)))
             {
diff --git a/MediaBrowser.Controller/Providers/Movies/MovieDbImagesProvider.cs b/MediaBrowser.Controller/Providers/Movies/MovieDbImagesProvider.cs
index 64f60d17b..5a30f447b 100644
--- a/MediaBrowser.Controller/Providers/Movies/MovieDbImagesProvider.cs
+++ b/MediaBrowser.Controller/Providers/Movies/MovieDbImagesProvider.cs
@@ -153,7 +153,7 @@ namespace MediaBrowser.Controller.Providers.Movies
             }
 
             // Don't refresh if we already have both poster and backdrop and we're not refreshing images
-            if (!ConfigurationManager.Configuration.RefreshItemImages && item.HasImage(ImageType.Primary) && item.BackdropImagePaths.Count > 0)
+            if (item.HasImage(ImageType.Primary) && item.BackdropImagePaths.Count > 0)
             {
                 return false;
             }
diff --git a/MediaBrowser.Controller/Providers/Music/FanArtArtistProvider.cs b/MediaBrowser.Controller/Providers/Music/FanArtArtistProvider.cs
index 7f4ac7c22..fc9500b40 100644
--- a/MediaBrowser.Controller/Providers/Music/FanArtArtistProvider.cs
+++ b/MediaBrowser.Controller/Providers/Music/FanArtArtistProvider.cs
@@ -124,7 +124,7 @@ namespace MediaBrowser.Controller.Providers.Music
               !ConfigurationManager.Configuration.DownloadMusicArtistImages.Logo &&
               !ConfigurationManager.Configuration.DownloadMusicArtistImages.Primary &&
 
-              // The fanart album provider depends on xml downloaded here, so honor it's settings too
+                // The fanart album provider depends on xml downloaded here, so honor it's settings too
                 !ConfigurationManager.Configuration.DownloadMusicAlbumImages.Disc &&
                 !ConfigurationManager.Configuration.DownloadMusicAlbumImages.Primary)
             {
diff --git a/MediaBrowser.Controller/Providers/TV/FanArtTVProvider.cs b/MediaBrowser.Controller/Providers/TV/FanArtTVProvider.cs
index 0ba48d3b0..164c5d0c9 100644
--- a/MediaBrowser.Controller/Providers/TV/FanArtTVProvider.cs
+++ b/MediaBrowser.Controller/Providers/TV/FanArtTVProvider.cs
@@ -70,6 +70,15 @@ namespace MediaBrowser.Controller.Providers.TV
                 return false;
             }
 
+            if (item.HasImage(ImageType.Art) &&
+                item.HasImage(ImageType.Logo) &&
+                item.HasImage(ImageType.Banner) &&
+                item.HasImage(ImageType.Thumb) &&
+                item.BackdropImagePaths.Count > 0)
+            {
+                return false;
+            }
+            
             if (providerInfo.Data != GetComparisonData(item.GetProviderId(MetadataProviders.Tvdb)))
             {
                 return true;