aboutsummaryrefslogtreecommitdiff
path: root/MediaBrowser.Providers/Manager/MetadataService.cs
diff options
context:
space:
mode:
Diffstat (limited to 'MediaBrowser.Providers/Manager/MetadataService.cs')
-rw-r--r--MediaBrowser.Providers/Manager/MetadataService.cs41
1 files changed, 31 insertions, 10 deletions
diff --git a/MediaBrowser.Providers/Manager/MetadataService.cs b/MediaBrowser.Providers/Manager/MetadataService.cs
index 6860aeff3..414cadf82 100644
--- a/MediaBrowser.Providers/Manager/MetadataService.cs
+++ b/MediaBrowser.Providers/Manager/MetadataService.cs
@@ -99,7 +99,6 @@ namespace MediaBrowser.Providers.Manager
var updateType = ItemUpdateType.None;
var refreshResult = GetLastResult(item);
- refreshResult.LastErrorMessage = string.Empty;
var itemImageProvider = new ItemImageProvider(Logger, ProviderManager, ServerConfigurationManager, FileSystem);
var localImagesFailed = false;
@@ -119,7 +118,6 @@ namespace MediaBrowser.Providers.Manager
{
localImagesFailed = true;
Logger.ErrorException("Error validating images for {0}", ex, item.Path ?? item.Name ?? "Unknown name");
- refreshResult.AddStatus(ex.Message);
}
var metadataResult = new MetadataResult<TItemType>
@@ -150,7 +148,6 @@ namespace MediaBrowser.Providers.Manager
var result = await RefreshWithProviders(metadataResult, id, refreshOptions, providers, itemImageProvider, cancellationToken).ConfigureAwait(false);
updateType = updateType | result.UpdateType;
- refreshResult.AddStatus(result.ErrorMessage);
if (result.Failures == 0)
{
refreshResult.SetDateLastMetadataRefresh(DateTime.UtcNow);
@@ -172,7 +169,6 @@ namespace MediaBrowser.Providers.Manager
var result = await itemImageProvider.RefreshImages(itemOfType, providers, refreshOptions, config, cancellationToken).ConfigureAwait(false);
updateType = updateType | result.UpdateType;
- refreshResult.AddStatus(result.ErrorMessage);
if (result.Failures == 0)
{
refreshResult.SetDateLastImagesRefresh(DateTime.UtcNow);
@@ -231,22 +227,32 @@ namespace MediaBrowser.Providers.Manager
private DateTime GetLastRefreshDate(IHasMetadata item)
{
- if (item.DateLastRefreshed != default(DateTime))
+ if (EnableDateLastRefreshed(item))
{
return item.DateLastRefreshed;
}
+ return item.DateLastSaved;
+ }
+
+ private bool EnableDateLastRefreshed(IHasMetadata item)
+ {
if (ServerConfigurationManager.Configuration.EnableDateLastRefresh)
{
- return item.DateLastRefreshed;
+ return true;
+ }
+
+ if (item.DateLastRefreshed != default(DateTime))
+ {
+ return true;
}
if (item is BoxSet || (item is IItemByName && !(item is MusicArtist)))
{
- return item.DateLastRefreshed;
+ return true;
}
- return item.DateLastSaved;
+ return false;
}
protected async Task SaveItem(MetadataResult<TItemType> result, ItemUpdateType reason, CancellationToken cancellationToken)
@@ -668,7 +674,14 @@ namespace MediaBrowser.Providers.Manager
{
try
{
- return changeMonitor.HasChanged(item, status, directoryService);
+ var hasChanged = changeMonitor.HasChanged(item, status, directoryService);
+
+ //if (hasChanged)
+ //{
+ // Logger.Debug("{0} reports change to {1}", changeMonitor.GetType().Name, item.Path ?? item.Name);
+ //}
+
+ return hasChanged;
}
catch (Exception ex)
{
@@ -681,7 +694,15 @@ namespace MediaBrowser.Providers.Manager
{
try
{
- return changeMonitor.HasChanged(item, directoryService, date);
+ var hasChanged = changeMonitor.HasChanged(item, directoryService, date);
+
+ //if (hasChanged)
+ //{
+ // Logger.Debug("{0} reports change to {1} since {2}", changeMonitor.GetType().Name,
+ // item.Path ?? item.Name, date);
+ //}
+
+ return hasChanged;
}
catch (Exception ex)
{