aboutsummaryrefslogtreecommitdiff
path: root/MediaBrowser.Controller
diff options
context:
space:
mode:
authorLuke Pulverenti <luke.pulverenti@gmail.com>2016-03-21 16:15:18 -0400
committerLuke Pulverenti <luke.pulverenti@gmail.com>2016-03-21 16:15:18 -0400
commit95250b9f220dbd3a5bc6c11cbdc983967b8574d4 (patch)
tree31130eced1141c4635aafa2d355be86a3a62ed6e /MediaBrowser.Controller
parent23fe6dc4be28c4d273bb22962ac769c749931090 (diff)
removed dead code
Diffstat (limited to 'MediaBrowser.Controller')
-rw-r--r--MediaBrowser.Controller/Entities/Folder.cs67
-rw-r--r--MediaBrowser.Controller/Persistence/IItemRepository.cs23
2 files changed, 4 insertions, 86 deletions
diff --git a/MediaBrowser.Controller/Entities/Folder.cs b/MediaBrowser.Controller/Entities/Folder.cs
index fa96ba4e51..a1a89d71f6 100644
--- a/MediaBrowser.Controller/Entities/Folder.cs
+++ b/MediaBrowser.Controller/Entities/Folder.cs
@@ -151,16 +151,6 @@ namespace MediaBrowser.Controller.Entities
AddChildInternal(item.Id);
await LibraryManager.CreateItem(item, cancellationToken).ConfigureAwait(false);
-
- if (!EnableNewFolderQuerying())
- {
- await ItemRepository.SaveChildren(Id, ActualChildren.Select(i => i.Id).ToList(), cancellationToken).ConfigureAwait(false);
- }
- }
-
- private static bool EnableNewFolderQuerying()
- {
- return ConfigurationManager.Configuration.MigrationVersion >= 1;
}
protected void AddChildrenInternal(List<Guid> children)
@@ -197,21 +187,11 @@ namespace MediaBrowser.Controller.Entities
/// Removes the child.
/// </summary>
/// <param name="item">The item.</param>
- /// <param name="cancellationToken">The cancellation token.</param>
- /// <returns>Task.</returns>
- /// <exception cref="System.InvalidOperationException">Unable to remove + item.Name</exception>
- public Task RemoveChild(BaseItem item, CancellationToken cancellationToken)
+ public void RemoveChild(BaseItem item)
{
RemoveChildrenInternal(new[] { item.Id }.ToList());
item.SetParent(null);
-
- if (!EnableNewFolderQuerying())
- {
- return ItemRepository.SaveChildren(Id, ActualChildren.Select(i => i.Id).ToList(), cancellationToken);
- }
-
- return Task.FromResult(true);
}
#region Indexing
@@ -500,11 +480,6 @@ namespace MediaBrowser.Controller.Entities
await LibraryManager.CreateItems(newItems, cancellationToken).ConfigureAwait(false);
AddChildrenInternal(newItems.Select(i => i.Id).ToList());
-
- if (!EnableNewFolderQuerying())
- {
- await ItemRepository.SaveChildren(Id, ActualChildren.Select(i => i.Id).ToList(), cancellationToken).ConfigureAwait(false);
- }
}
}
@@ -734,45 +709,11 @@ namespace MediaBrowser.Controller.Entities
/// <returns>IEnumerable{BaseItem}.</returns>
protected IEnumerable<Guid> GetCachedChildren()
{
- if (EnableNewFolderQuerying())
+ return ItemRepository.GetItemIdsList(new InternalItemsQuery
{
- return ItemRepository.GetItemIdsList(new InternalItemsQuery
- {
- ParentId = Id
-
- });
- }
-
- return ItemRepository.GetChildrenItems(Id).Select(RetrieveChild).Where(i => i != null).Select(i => i.Id);
- }
-
- private BaseItem RetrieveChild(BaseItem child)
- {
- if (child == null || child.Id == Guid.Empty)
- {
- Logger.Error("Item found with empty Id: " + (child.Path ?? child.Name));
- return null;
- }
-
- var item = LibraryManager.GetMemoryItemById(child.Id);
-
- if (item != null)
- {
- if (item is IByReferenceItem)
- {
- return LibraryManager.GetOrAddByReferenceItem(item);
- }
-
- item.SetParent(this);
- }
- else
- {
- child.SetParent(this);
- LibraryManager.RegisterItem(child);
- item = child;
- }
+ ParentId = Id
- return item;
+ });
}
public QueryResult<BaseItem> QueryRecursive(InternalItemsQuery query)
diff --git a/MediaBrowser.Controller/Persistence/IItemRepository.cs b/MediaBrowser.Controller/Persistence/IItemRepository.cs
index 533d66b950..15df1f6492 100644
--- a/MediaBrowser.Controller/Persistence/IItemRepository.cs
+++ b/MediaBrowser.Controller/Persistence/IItemRepository.cs
@@ -43,13 +43,6 @@ namespace MediaBrowser.Controller.Persistence
IEnumerable<ItemReview> GetCriticReviews(Guid itemId);
/// <summary>
- /// Gets the children items.
- /// </summary>
- /// <param name="parentId">The parent identifier.</param>
- /// <returns>IEnumerable&lt;BaseItem&gt;.</returns>
- IEnumerable<BaseItem> GetChildrenItems(Guid parentId);
-
- /// <summary>
/// Saves the critic reviews.
/// </summary>
/// <param name="itemId">The item id.</param>
@@ -97,22 +90,6 @@ namespace MediaBrowser.Controller.Persistence
Task SaveChapters(Guid id, IEnumerable<ChapterInfo> chapters, CancellationToken cancellationToken);
/// <summary>
- /// Gets the children.
- /// </summary>
- /// <param name="parentId">The parent id.</param>
- /// <returns>IEnumerable{ChildDefinition}.</returns>
- IEnumerable<Guid> GetChildren(Guid parentId);
-
- /// <summary>
- /// Saves the children.
- /// </summary>
- /// <param name="parentId">The parent id.</param>
- /// <param name="children">The children.</param>
- /// <param name="cancellationToken">The cancellation token.</param>
- /// <returns>Task.</returns>
- Task SaveChildren(Guid parentId, IEnumerable<Guid> children, CancellationToken cancellationToken);
-
- /// <summary>
/// Gets the media streams.
/// </summary>
/// <param name="query">The query.</param>