fixes #1787
This commit is contained in:
parent
0af6b57334
commit
dc15f93005
|
@ -52,6 +52,23 @@ namespace MediaBrowser.Api.UserLibrary
|
||||||
/// <param name="dtoService">The dto service.</param>
|
/// <param name="dtoService">The dto service.</param>
|
||||||
public ItemsService(IUserManager userManager, ILibraryManager libraryManager, ILocalizationManager localization, IDtoService dtoService)
|
public ItemsService(IUserManager userManager, ILibraryManager libraryManager, ILocalizationManager localization, IDtoService dtoService)
|
||||||
{
|
{
|
||||||
|
if (userManager == null)
|
||||||
|
{
|
||||||
|
throw new ArgumentNullException("userManager");
|
||||||
|
}
|
||||||
|
if (libraryManager == null)
|
||||||
|
{
|
||||||
|
throw new ArgumentNullException("libraryManager");
|
||||||
|
}
|
||||||
|
if (localization == null)
|
||||||
|
{
|
||||||
|
throw new ArgumentNullException("localization");
|
||||||
|
}
|
||||||
|
if (dtoService == null)
|
||||||
|
{
|
||||||
|
throw new ArgumentNullException("dtoService");
|
||||||
|
}
|
||||||
|
|
||||||
_userManager = userManager;
|
_userManager = userManager;
|
||||||
_libraryManager = libraryManager;
|
_libraryManager = libraryManager;
|
||||||
_localization = localization;
|
_localization = localization;
|
||||||
|
@ -65,6 +82,11 @@ namespace MediaBrowser.Api.UserLibrary
|
||||||
/// <returns>System.Object.</returns>
|
/// <returns>System.Object.</returns>
|
||||||
public async Task<object> Get(GetItems request)
|
public async Task<object> Get(GetItems request)
|
||||||
{
|
{
|
||||||
|
if (request == null)
|
||||||
|
{
|
||||||
|
throw new ArgumentNullException("request");
|
||||||
|
}
|
||||||
|
|
||||||
var result = await GetItems(request).ConfigureAwait(false);
|
var result = await GetItems(request).ConfigureAwait(false);
|
||||||
|
|
||||||
return ToOptimizedSerializedResultUsingCache(result);
|
return ToOptimizedSerializedResultUsingCache(result);
|
||||||
|
@ -78,15 +100,27 @@ namespace MediaBrowser.Api.UserLibrary
|
||||||
private async Task<ItemsResult> GetItems(GetItems request)
|
private async Task<ItemsResult> GetItems(GetItems request)
|
||||||
{
|
{
|
||||||
var user = !string.IsNullOrWhiteSpace(request.UserId) ? _userManager.GetUserById(request.UserId) : null;
|
var user = !string.IsNullOrWhiteSpace(request.UserId) ? _userManager.GetUserById(request.UserId) : null;
|
||||||
|
|
||||||
var result = await GetItemsToSerialize(request, user).ConfigureAwait(false);
|
var result = await GetItemsToSerialize(request, user).ConfigureAwait(false);
|
||||||
|
|
||||||
|
if (result == null)
|
||||||
|
{
|
||||||
|
throw new InvalidOperationException("GetItemsToSerialize returned null");
|
||||||
|
}
|
||||||
|
|
||||||
var dtoOptions = GetDtoOptions(request);
|
var dtoOptions = GetDtoOptions(request);
|
||||||
|
|
||||||
|
var dtoList = await _dtoService.GetBaseItemDtos(result.Items, dtoOptions, user).ConfigureAwait(false);
|
||||||
|
|
||||||
|
if (dtoList == null)
|
||||||
|
{
|
||||||
|
throw new InvalidOperationException("GetBaseItemDtos returned null");
|
||||||
|
}
|
||||||
|
|
||||||
return new ItemsResult
|
return new ItemsResult
|
||||||
{
|
{
|
||||||
TotalRecordCount = result.TotalRecordCount,
|
TotalRecordCount = result.TotalRecordCount,
|
||||||
Items = (await _dtoService.GetBaseItemDtos(result.Items, dtoOptions, user).ConfigureAwait(false)).ToArray()
|
Items = dtoList.ToArray()
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -88,6 +88,16 @@ namespace MediaBrowser.Server.Implementations.Dto
|
||||||
|
|
||||||
public async Task<List<BaseItemDto>> GetBaseItemDtos(IEnumerable<BaseItem> items, DtoOptions options, User user = null, BaseItem owner = null)
|
public async Task<List<BaseItemDto>> GetBaseItemDtos(IEnumerable<BaseItem> items, DtoOptions options, User user = null, BaseItem owner = null)
|
||||||
{
|
{
|
||||||
|
if (items == null)
|
||||||
|
{
|
||||||
|
throw new ArgumentNullException("items");
|
||||||
|
}
|
||||||
|
|
||||||
|
if (options == null)
|
||||||
|
{
|
||||||
|
throw new ArgumentNullException("options");
|
||||||
|
}
|
||||||
|
|
||||||
var syncJobItems = GetSyncedItemProgress(options);
|
var syncJobItems = GetSyncedItemProgress(options);
|
||||||
var syncDictionary = GetSyncedItemProgressDictionary(syncJobItems);
|
var syncDictionary = GetSyncedItemProgressDictionary(syncJobItems);
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user