Browse Source

Merge pull request #11526 from jellyfin/don't-assume-library-change-only-on-item-count

Use actual item removal count for library change determination
Joshua M. Boniface 1 year ago
parent
commit
afd1d3be32
1 changed files with 3 additions and 4 deletions
  1. 3 4
      MediaBrowser.Controller/Entities/Folder.cs

+ 3 - 4
MediaBrowser.Controller/Entities/Folder.cs

@@ -421,13 +421,12 @@ namespace MediaBrowser.Controller.Entities
                     validChildren.Add(child);
                 }
 
+                // That's all the new and changed ones - now see if any have been removed and need cleanup
+                var itemsRemoved = currentChildren.Values.Except(validChildren).ToList();
                 var shouldRemove = !IsRoot || allowRemoveRoot;
                 // If it's an AggregateFolder, don't remove
-                if (shouldRemove && currentChildren.Count != validChildren.Count)
+                if (shouldRemove && itemsRemoved.Count > 0)
                 {
-                    // That's all the new and changed ones - now see if there are any that are missing
-                    var itemsRemoved = currentChildren.Values.Except(validChildren).ToList();
-
                     foreach (var item in itemsRemoved)
                     {
                         if (item.IsFileProtocol)