revert clean task changes

MediaBrowser.Server.Implementations/Persistence/CleanDatabaseScheduledTask.cs

@@ -142,55 +142,33 @@ namespace MediaBrowser.Server.Implementations.Persistence
             }
         }
 
-        private Task UpdateToLatestSchema(CancellationToken cancellationToken, IProgress<double> progress)
+        private async Task UpdateToLatestSchema(CancellationToken cancellationToken, IProgress<double> progress)
         {
-            return UpdateToLatestSchema(0, 0, null, cancellationToken, progress);
+            var itemIds = _libraryManager.GetItemIds(new InternalItemsQuery
-        }
-
-        private async Task UpdateToLatestSchema(int queryStartIndex, int progressStartIndex, int? totalRecordCount, CancellationToken cancellationToken, IProgress<double> progress)
-        {
-            IEnumerable<BaseItem> items;
-            int numItemsToSave;
-            var pageSize = 1000;
-
-            if (totalRecordCount.HasValue)
-            {
-                var list = _libraryManager.GetItemList(new InternalItemsQuery
-                {
-                    IsCurrentSchema = false,
-                    ExcludeItemTypes = new[] { typeof(LiveTvProgram).Name },
-                    StartIndex = queryStartIndex,
-                    Limit = pageSize
-
-                }).ToList();
-
-                items = list;
-                numItemsToSave = list.Count;
-            }
-            else
-            {
-                var itemsResult = _libraryManager.GetItemsResult(new InternalItemsQuery
             {
                 IsCurrentSchema = false,
-                    ExcludeItemTypes = new[] { typeof(LiveTvProgram).Name },
+                ExcludeItemTypes = new[] { typeof(LiveTvProgram).Name }
-                    StartIndex = queryStartIndex,
-                    Limit = pageSize
             });
 
-                totalRecordCount = itemsResult.TotalRecordCount;
+            var numComplete = 0;
-                items = itemsResult.Items;
+            var numItems = itemIds.Count;
-                numItemsToSave = itemsResult.Items.Length;
-            }
-
-            var numItems = totalRecordCount.Value;
 
             _logger.Debug("Upgrading schema for {0} items", numItems);
 
-            if (numItemsToSave > 0)
+            foreach (var itemId in itemIds)
+            {
+                cancellationToken.ThrowIfCancellationRequested();
+
+                if (itemId != Guid.Empty)
+                {
+                    // Somehow some invalid data got into the db. It probably predates the boundary checking
+                    var item = _libraryManager.GetItemById(itemId);
+
+                    if (item != null)
                     {
                         try
                         {
-                    await _itemRepo.SaveItems(items, cancellationToken).ConfigureAwait(false);
+                            await _itemRepo.SaveItem(item, cancellationToken).ConfigureAwait(false);
                         }
                         catch (OperationCanceledException)
                         {
@@ -200,20 +178,17 @@ namespace MediaBrowser.Server.Implementations.Persistence
                         {
                             _logger.ErrorException("Error saving item", ex);
                         }
+                    }
+                }
 
-                progressStartIndex += pageSize;
+                numComplete++;
-                double percent = progressStartIndex;
+                double percent = numComplete;
                 percent /= numItems;
                 progress.Report(percent * 100);
-
-                var newStartIndex = queryStartIndex + (pageSize - numItemsToSave);
-                await UpdateToLatestSchema(newStartIndex, progressStartIndex, totalRecordCount, cancellationToken, progress).ConfigureAwait(false);
             }
-            else
+
-            {
             progress.Report(100);
         }
-        }
 
         private async Task CleanDeadItems(CancellationToken cancellationToken, IProgress<double> progress)
         {