fixed checking against wrong provider id

MediaBrowser.Controller/Providers/Movies/FanArtMovieProvider.cs

@@ -120,12 +120,6 @@ namespace MediaBrowser.Controller.Providers.Movies
                 return false;
             }
             
-            // Refresh if tmdb id has changed
-            if (providerInfo.Data != GetComparisonData(item.GetProviderId(MetadataProviders.Tmdb)))
-            {
-                return true;
-            }
-            
             if (!ConfigurationManager.Configuration.DownloadMovieImages.Art &&
                 !ConfigurationManager.Configuration.DownloadMovieImages.Logo &&
                 !ConfigurationManager.Configuration.DownloadMovieImages.Disc &&
@@ -136,6 +130,12 @@ namespace MediaBrowser.Controller.Providers.Movies
                 return false;
             }
 
+            // Refresh if tmdb id has changed
+            if (providerInfo.Data != GetComparisonData(item.GetProviderId(MetadataProviders.Tmdb)))
+            {
+                return true;
+            }
+
             return base.NeedsRefreshInternal(item, providerInfo);
         }
 

MediaBrowser.Controller/Providers/TV/FanArtTVProvider.cs

@@ -203,7 +203,7 @@ namespace MediaBrowser.Controller.Providers.TV
                 }
             }
 
-            data.Data = GetComparisonData(item.GetProviderId(MetadataProviders.Tvcom));
+            data.Data = GetComparisonData(item.GetProviderId(MetadataProviders.Tvdb));
             SetLastRefreshed(series, DateTime.UtcNow, status);
 
             return true;

MediaBrowser.Controller/Providers/TV/RemoteEpisodeProvider.cs

@@ -121,8 +121,8 @@ namespace MediaBrowser.Controller.Providers.TV
 
                 if (seriesId != null)
                 {
-                    await FetchEpisodeData(episode, seriesId, cancellationToken).ConfigureAwait(false);
+                    var status = await FetchEpisodeData(episode, seriesId, cancellationToken).ConfigureAwait(false);
-                    SetLastRefreshed(item, DateTime.UtcNow);
+                    SetLastRefreshed(item, DateTime.UtcNow, status);
                     return true;
                 }
                 Logger.Info("Episode provider cannot determine Series Id for " + item.Path);
@@ -140,18 +140,18 @@ namespace MediaBrowser.Controller.Providers.TV
         /// <param name="seriesId">The series id.</param>
         /// <param name="cancellationToken">The cancellation token.</param>
         /// <returns>Task{System.Boolean}.</returns>
-        private async Task<bool> FetchEpisodeData(Episode episode, string seriesId, CancellationToken cancellationToken)
+        private async Task<ProviderRefreshStatus> FetchEpisodeData(Episode episode, string seriesId, CancellationToken cancellationToken)
         {
-
-            string name = episode.Name;
             string location = episode.Path;
 
             string epNum = TVUtils.EpisodeNumberFromFile(location, episode.Season != null);
 
+            var status = ProviderRefreshStatus.Success;
+
             if (epNum == null)
             {
                 Logger.Warn("TvDbProvider: Could not determine episode number for: " + episode.Path);
-                return false;
+                return status;
             }
 
             var episodeNumber = Int32.Parse(epNum);
@@ -159,7 +159,7 @@ namespace MediaBrowser.Controller.Providers.TV
             episode.IndexNumber = episodeNumber;
             var usingAbsoluteData = false;
 
-            if (string.IsNullOrEmpty(seriesId)) return false;
+            if (string.IsNullOrEmpty(seriesId)) return status;
 
             var seasonNumber = "";
             if (episode.Parent is Season)
@@ -240,10 +240,7 @@ namespace MediaBrowser.Controller.Providers.TV
                         }
                         catch (HttpException)
                         {
-                        }
+                            status = ProviderRefreshStatus.CompletedWithErrors;
-                        catch (IOException)
-                        {
-
                         }
                     }
 
@@ -303,12 +300,12 @@ namespace MediaBrowser.Controller.Providers.TV
                         await _providerManager.SaveToLibraryFilesystem(episode, Path.Combine(episode.MetaLocation, Path.GetFileNameWithoutExtension(episode.Path) + ".xml"), ms, cancellationToken).ConfigureAwait(false);
                     }
 
-                    return true;
+                    return status;
                 }
 
             }
 
-            return false;
+            return status;
         }
 
         /// <summary>

MediaBrowser.Controller/Providers/TV/RemoteSeasonProvider.cs

@@ -55,7 +55,8 @@ namespace MediaBrowser.Controller.Providers.TV
         /// <value>The priority.</value>
         public override MetadataProviderPriority Priority
         {
-            get { return MetadataProviderPriority.Second; }
+            // Run after fanart
+            get { return MetadataProviderPriority.Fourth; }
         }
 
         /// <summary>