diff options
| author | Claus Vium <cvium@users.noreply.github.com> | 2022-03-11 08:15:12 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-03-11 08:15:12 +0100 |
| commit | 53209830e7b566949c16b43c864f6f85336cb92c (patch) | |
| tree | 59a99d347cffe722da175269dfd99bac08fb7fb0 /Jellyfin.Api/Controllers/PersonsController.cs | |
| parent | 21ef6661d6ef48fb18da10b8e0abf2c1d1345ed1 (diff) | |
| parent | f50a250cd9fac47bcbd9a05e99c8ffe4d294e320 (diff) | |
Merge pull request #7346 from Bond-009/guid
Optimize Guid comparisons
Diffstat (limited to 'Jellyfin.Api/Controllers/PersonsController.cs')
| -rw-r--r-- | Jellyfin.Api/Controllers/PersonsController.cs | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/Jellyfin.Api/Controllers/PersonsController.cs b/Jellyfin.Api/Controllers/PersonsController.cs index ffc748a6e..be4b9eded 100644 --- a/Jellyfin.Api/Controllers/PersonsController.cs +++ b/Jellyfin.Api/Controllers/PersonsController.cs @@ -82,12 +82,9 @@ namespace Jellyfin.Api.Controllers .AddClientFields(Request) .AddAdditionalDtoOptions(enableImages, enableUserData, imageTypeLimit, enableImageTypes); - User? user = null; - - if (userId.HasValue && !userId.Equals(Guid.Empty)) - { - user = _userManager.GetUserById(userId.Value); - } + User? user = userId is null || userId.Value.Equals(default) + ? null + : _userManager.GetUserById(userId.Value); var isFavoriteInFilters = filters.Any(f => f == ItemFilter.IsFavorite); var peopleItems = _libraryManager.GetPeopleItems(new InternalPeopleQuery( @@ -127,7 +124,7 @@ namespace Jellyfin.Api.Controllers return NotFound(); } - if (userId.HasValue && !userId.Equals(Guid.Empty)) + if (userId.HasValue && !userId.Value.Equals(default)) { var user = _userManager.GetUserById(userId.Value); return _dtoService.GetBaseItemDto(item, dtoOptions, user); |
