aboutsummaryrefslogtreecommitdiff
path: root/MediaBrowser.Controller/Entities/UserViewBuilder.cs
diff options
context:
space:
mode:
Diffstat (limited to 'MediaBrowser.Controller/Entities/UserViewBuilder.cs')
-rw-r--r--MediaBrowser.Controller/Entities/UserViewBuilder.cs51
1 files changed, 24 insertions, 27 deletions
diff --git a/MediaBrowser.Controller/Entities/UserViewBuilder.cs b/MediaBrowser.Controller/Entities/UserViewBuilder.cs
index f9ad2d86e6..82256cd964 100644
--- a/MediaBrowser.Controller/Entities/UserViewBuilder.cs
+++ b/MediaBrowser.Controller/Entities/UserViewBuilder.cs
@@ -455,25 +455,34 @@ namespace MediaBrowser.Controller.Entities
{
var itemList = filtered.ToList();
var folderIds = itemList.OfType<Folder>().Select(f => f.Id).ToList();
+ var leaves = itemList.Where(i => i is not Folder).ToList();
+ var isPlayedValue = query.IsPlayed.Value;
- if (folderIds.Count > 0)
- {
- var counts = libraryManager.GetPlayedAndTotalCountBatch(folderIds, user);
- var isPlayedValue = query.IsPlayed.Value;
+ var counts = folderIds.Count > 0
+ ? libraryManager.GetPlayedAndTotalCountBatch(folderIds, user)
+ : null;
+
+ // A movie held as several files is watched once any of its versions is watched.
+ var resumeData = leaves.Count > 0
+ ? userDataManager.GetResumeUserDataBatch(leaves, user)
+ : null;
- return itemList.Where(item =>
+ return itemList.Where(item =>
+ {
+ if (item is Folder)
{
- if (item is Folder)
- {
- var itemCount = counts.GetValueOrDefault(item.Id);
- return (itemCount.Played >= itemCount.Total) == isPlayedValue;
- }
+ var itemCount = counts?.GetValueOrDefault(item.Id) ?? default;
+ return (itemCount.Played >= itemCount.Total) == isPlayedValue;
+ }
- return true;
- });
- }
+ var played = userDataManager.GetUserData(user, item)?.Played ?? false;
+ if (!played && resumeData is not null && resumeData.TryGetValue(item.Id, out var versionData))
+ {
+ played = versionData.UserData.Played;
+ }
- return itemList;
+ return played == isPlayedValue;
+ });
}
return filtered;
@@ -606,19 +615,7 @@ namespace MediaBrowser.Controller.Entities
}
}
- if (query.IsPlayed.HasValue)
- {
- // Folder.IsPlayed() hits the DB per-item (N+1 queries).
- // Folders are batch-filtered by the collection Filter() overload.
- if (!item.IsFolder)
- {
- userData ??= userDataManager.GetUserData(user, item);
- if (item.IsPlayed(user, userData) != query.IsPlayed.Value)
- {
- return false;
- }
- }
- }
+ // IsPlayed is answered by the collection Filter() overload for folders and leaves alike.
if (query.IsLocked.HasValue)
{