From 0189f4c49dc89654e6aa10c5dd0fc50a0984bfec Mon Sep 17 00:00:00 2001 From: Luke Pulverenti Date: Wed, 26 Oct 2016 14:25:03 -0400 Subject: move provider project towards portability --- .../Persistence/SqliteItemRepository.cs | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) (limited to 'MediaBrowser.Server.Implementations') diff --git a/MediaBrowser.Server.Implementations/Persistence/SqliteItemRepository.cs b/MediaBrowser.Server.Implementations/Persistence/SqliteItemRepository.cs index bc62516c7..667f6b89a 100644 --- a/MediaBrowser.Server.Implementations/Persistence/SqliteItemRepository.cs +++ b/MediaBrowser.Server.Implementations/Persistence/SqliteItemRepository.cs @@ -1112,7 +1112,10 @@ namespace MediaBrowser.Server.Implementations.Persistence private string SerializeProviderIds(BaseItem item) { - var ids = item.ProviderIds.ToList(); + // Ideally we shouldn't need this IsNullOrWhiteSpace check but we're seeing some cases of bad data slip through + var ids = item.ProviderIds + .Where(i => !string.IsNullOrWhiteSpace(i.Value)) + .ToList(); if (ids.Count == 0) { @@ -4889,6 +4892,15 @@ namespace MediaBrowser.Server.Implementations.Persistence foreach (var pair in newValues) { + if (string.IsNullOrWhiteSpace(pair.Key)) + { + continue; + } + if (string.IsNullOrWhiteSpace(pair.Value)) + { + continue; + } + _saveProviderIdsCommand.GetParameter(0).Value = itemId; _saveProviderIdsCommand.GetParameter(1).Value = pair.Key; _saveProviderIdsCommand.GetParameter(2).Value = pair.Value; -- cgit v1.2.3