diff --git a/Emby.Server.Implementations/Data/SqliteItemRepository.cs b/Emby.Server.Implementations/Data/SqliteItemRepository.cs index 5f2a314f9f..1517029050 100644 --- a/Emby.Server.Implementations/Data/SqliteItemRepository.cs +++ b/Emby.Server.Implementations/Data/SqliteItemRepository.cs @@ -389,6 +389,8 @@ namespace Emby.Server.Implementations.Data }); } } + + GC.Collect(); } catch (Exception ex) { diff --git a/Emby.Server.Implementations/Library/LibraryManager.cs b/Emby.Server.Implementations/Library/LibraryManager.cs index 1ff61286f9..5bf53fcb43 100644 --- a/Emby.Server.Implementations/Library/LibraryManager.cs +++ b/Emby.Server.Implementations/Library/LibraryManager.cs @@ -855,7 +855,7 @@ namespace Emby.Server.Implementations.Library SortOrder = SortOrder.Descending, Limit = 1 }; - + return GetItemList(query) .FirstOrDefault(); } @@ -1113,16 +1113,21 @@ namespace Emby.Server.Implementations.Library progress.Report(1); - var userRoot = GetUserRootFolder(); + await GetUserRootFolder().RefreshMetadata(cancellationToken).ConfigureAwait(false); - await userRoot.RefreshMetadata(cancellationToken).ConfigureAwait(false); - - await userRoot.ValidateChildren(new Progress(), cancellationToken, new MetadataRefreshOptions(_fileSystem), recursive: false).ConfigureAwait(false); + await GetUserRootFolder().ValidateChildren(new Progress(), cancellationToken, new MetadataRefreshOptions(_fileSystem), recursive: false).ConfigureAwait(false); progress.Report(2); + // Quickly scan CollectionFolders for changes + foreach (var folder in GetUserRootFolder().Children.OfType().ToList()) + { + await folder.RefreshMetadata(cancellationToken).ConfigureAwait(false); + } + progress.Report(3); + var innerProgress = new ActionableProgress(); - innerProgress.RegisterAction(pct => progress.Report(2 + pct * .73)); + innerProgress.RegisterAction(pct => progress.Report(3 + pct * .72)); // Now validate the entire media library await RootFolder.ValidateChildren(innerProgress, cancellationToken, new MetadataRefreshOptions(_fileSystem), recursive: true).ConfigureAwait(false); @@ -1291,7 +1296,6 @@ namespace Emby.Server.Implementations.Library if (parent != null) { SetTopParentIdsOrAncestors(query, new List { parent }); - query.ParentId = null; } } @@ -1311,7 +1315,6 @@ namespace Emby.Server.Implementations.Library if (parent != null) { SetTopParentIdsOrAncestors(query, new List { parent }); - query.ParentId = null; } } @@ -1456,6 +1459,12 @@ namespace Emby.Server.Implementations.Library // Optimize by querying against top level views query.TopParentIds = parents.SelectMany(i => GetTopParentIdsForQuery(i, query.User)).Select(i => i.ToString("N")).ToArray(); query.AncestorIds = new string[] { }; + + // Prevent searching in all libraries due to empty filter + if (query.TopParentIds.Length == 0) + { + query.TopParentIds = new[] { Guid.NewGuid().ToString("N") }; + } } } @@ -1478,7 +1487,6 @@ namespace Emby.Server.Implementations.Library if (parent != null) { SetTopParentIdsOrAncestors(query, new List { parent }); - query.ParentId = null; } } @@ -1514,12 +1522,26 @@ namespace Emby.Server.Implementations.Library { // Optimize by querying against top level views query.TopParentIds = parents.SelectMany(i => GetTopParentIdsForQuery(i, query.User)).Select(i => i.ToString("N")).ToArray(); + + // Prevent searching in all libraries due to empty filter + if (query.TopParentIds.Length == 0) + { + query.TopParentIds = new[] { Guid.NewGuid().ToString("N") }; + } } else { // We need to be able to query from any arbitrary ancestor up the tree query.AncestorIds = parents.SelectMany(i => i.GetIdsForAncestorQuery()).Select(i => i.ToString("N")).ToArray(); + + // Prevent searching in all libraries due to empty filter + if (query.AncestorIds.Length == 0) + { + query.AncestorIds = new[] { Guid.NewGuid().ToString("N") }; + } } + + query.ParentId = null; } private void AddUserToQuery(InternalItemsQuery query, User user) @@ -1561,7 +1583,7 @@ namespace Emby.Server.Implementations.Library }, CancellationToken.None).Result; - return channelResult.Items.Select(i => i.Id); + return channelResult.Items.Select(i => i.Id); } // Translate view into folders @@ -1602,7 +1624,7 @@ namespace Emby.Server.Implementations.Library { return collectionFolder.PhysicalFolderIds; } - + var topParent = item.GetTopParent(); if (topParent != null) { diff --git a/Emby.Server.Implementations/TV/TVSeriesManager.cs b/Emby.Server.Implementations/TV/TVSeriesManager.cs index 5e6de1d4db..6bf4125253 100644 --- a/Emby.Server.Implementations/TV/TVSeriesManager.cs +++ b/Emby.Server.Implementations/TV/TVSeriesManager.cs @@ -144,23 +144,16 @@ namespace Emby.Server.Implementations.TV // If viewing all next up for all series, remove first episodes // But if that returns empty, keep those first episodes (avoid completely empty view) var alwaysEnableFirstEpisode = !string.IsNullOrWhiteSpace(request.SeriesId); - var isFirstItemAFirstEpisode = true; return allNextUp .Where(i => { if (alwaysEnableFirstEpisode || i.Item1 != DateTime.MinValue) { - isFirstItemAFirstEpisode = false; return true; } - if (isFirstItemAFirstEpisode) - { - return false; - } - - return true; + return false; }) .Select(i => i.Item2()) .Where(i => i != null) diff --git a/MediaBrowser.Api/BaseApiService.cs b/MediaBrowser.Api/BaseApiService.cs index 07ec6d955a..1ee499ed56 100644 --- a/MediaBrowser.Api/BaseApiService.cs +++ b/MediaBrowser.Api/BaseApiService.cs @@ -150,7 +150,7 @@ namespace MediaBrowser.Api options.Fields.Add(Model.Querying.ItemFields.ChildCount); } - if (client.IndexOf("web", StringComparison.OrdinalIgnoreCase) == -1 && + else if (client.IndexOf("web", StringComparison.OrdinalIgnoreCase) == -1 && // covers both emby mobile and emby for android mobile client.IndexOf("mobile", StringComparison.OrdinalIgnoreCase) == -1 && diff --git a/MediaBrowser.Controller/Entities/CollectionFolder.cs b/MediaBrowser.Controller/Entities/CollectionFolder.cs index c505aefb3f..681f16f07c 100644 --- a/MediaBrowser.Controller/Entities/CollectionFolder.cs +++ b/MediaBrowser.Controller/Entities/CollectionFolder.cs @@ -201,6 +201,11 @@ namespace MediaBrowser.Controller.Entities } protected override bool RefreshLinkedChildren(IEnumerable fileSystemChildren) + { + return RefreshLinkedChildrenInternal(true); + } + + private bool RefreshLinkedChildrenInternal(bool setFolders) { var physicalFolders = GetPhysicalFolders(false) .ToList(); @@ -219,7 +224,10 @@ namespace MediaBrowser.Controller.Entities if (!folderIds.SequenceEqual(newFolderIds)) { changed = true; - PhysicalFolderIds = newFolderIds.ToList(); + if (setFolders) + { + PhysicalFolderIds = newFolderIds.ToList(); + } } return changed;