aboutsummaryrefslogtreecommitdiff
path: root/Jellyfin.Api/Controllers/DisplayPreferencesController.cs
diff options
context:
space:
mode:
authorcrobibero <cody@robibe.ro>2020-04-21 07:55:57 -0600
committercrobibero <cody@robibe.ro>2020-04-21 07:55:57 -0600
commit04119c0d409342050cb7624f025a21985e10a412 (patch)
tree2c36116bb86b26b7298c5a845630f30cc8a4fccc /Jellyfin.Api/Controllers/DisplayPreferencesController.cs
parentc5d709f77ed2158bf68b8cc81238067d4525518f (diff)
Remove exception handler
Diffstat (limited to 'Jellyfin.Api/Controllers/DisplayPreferencesController.cs')
-rw-r--r--Jellyfin.Api/Controllers/DisplayPreferencesController.cs51
1 files changed, 18 insertions, 33 deletions
diff --git a/Jellyfin.Api/Controllers/DisplayPreferencesController.cs b/Jellyfin.Api/Controllers/DisplayPreferencesController.cs
index 0554091b4..e15e9c4be 100644
--- a/Jellyfin.Api/Controllers/DisplayPreferencesController.cs
+++ b/Jellyfin.Api/Controllers/DisplayPreferencesController.cs
@@ -1,6 +1,5 @@
#nullable enable
-using System;
using System.ComponentModel.DataAnnotations;
using System.Threading;
using MediaBrowser.Controller.Net;
@@ -45,20 +44,13 @@ namespace Jellyfin.Api.Controllers
[FromQuery] [Required] string userId,
[FromQuery] [Required] string client)
{
- try
+ var result = _displayPreferencesRepository.GetDisplayPreferences(displayPreferencesId, userId, client);
+ if (result == null)
{
- var result = _displayPreferencesRepository.GetDisplayPreferences(displayPreferencesId, userId, client);
- if (result == null)
- {
- return NotFound();
- }
-
- return Ok(result);
- }
- catch (Exception e)
- {
- return StatusCode(StatusCodes.Status500InternalServerError, e.Message);
+ return NotFound();
}
+
+ return Ok(result);
}
/// <summary>
@@ -80,30 +72,23 @@ namespace Jellyfin.Api.Controllers
[FromQuery, BindRequired] string client,
[FromBody, BindRequired] DisplayPreferences displayPreferences)
{
- try
+ if (!ModelState.IsValid)
{
- if (!ModelState.IsValid)
- {
- return BadRequest(ModelState);
- }
-
- if (displayPreferencesId == null)
- {
- // do nothing.
- }
-
- _displayPreferencesRepository.SaveDisplayPreferences(
- displayPreferences,
- userId,
- client,
- CancellationToken.None);
-
- return Ok();
+ return BadRequest(ModelState);
}
- catch (Exception e)
+
+ if (displayPreferencesId == null)
{
- return StatusCode(StatusCodes.Status500InternalServerError, e.Message);
+ // do nothing.
}
+
+ _displayPreferencesRepository.SaveDisplayPreferences(
+ displayPreferences,
+ userId,
+ client,
+ CancellationToken.None);
+
+ return Ok();
}
}
}