diff options
| author | Vasily <JustAMan@users.noreply.github.com> | 2020-03-02 12:36:44 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-03-02 12:36:44 +0300 |
| commit | 9a9f2aa293b04ae315905ad7dff03184e281ae16 (patch) | |
| tree | cc126001d1e62ddc8e149ac02951932d305a824a | |
| parent | 7f38af3701aa06e2281799ffeac59e875eaec5db (diff) | |
Apply suggestions from code review
Co-Authored-By: Bond-009 <bond.009@outlook.com>
| -rw-r--r-- | MediaBrowser.Controller/Entities/Folder.cs | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/MediaBrowser.Controller/Entities/Folder.cs b/MediaBrowser.Controller/Entities/Folder.cs index cda4c31ad..04e07b0f2 100644 --- a/MediaBrowser.Controller/Entities/Folder.cs +++ b/MediaBrowser.Controller/Entities/Folder.cs @@ -807,7 +807,7 @@ namespace MediaBrowser.Controller.Entities return false; } - private IReadOnlyList<BaseItem> SortItemsByRequest(InternalItemsQuery query, IReadOnlyList<BaseItem> items) + private static BaseItem[] SortItemsByRequest(InternalItemsQuery query, IReadOnlyList<BaseItem> items) { var ids = query.ItemIds; int size = items.Count; @@ -825,10 +825,11 @@ namespace MediaBrowser.Controller.Entities } var newItems = new BaseItem[size]; - foreach(var item in items) + foreach (var item in items) { newItems[positions[item.Id]] = item; } + return newItems; } @@ -842,6 +843,7 @@ namespace MediaBrowser.Controller.Entities { result.Items = SortItemsByRequest(query, result.Items); } + return result; } @@ -860,6 +862,7 @@ namespace MediaBrowser.Controller.Entities { return SortItemsByRequest(query, result); } + return result.ToArray(); } |
