mirror of https://github.com/jellyfin/jellyfin.git
Merge pull request #3941 from crobibero/no-more-form
Conver all remaining form request to body
This commit is contained in:
commit
92c491397e
|
@ -117,7 +117,7 @@ namespace Jellyfin.Api.Controllers
|
||||||
[HttpPost("MediaEncoder/Path")]
|
[HttpPost("MediaEncoder/Path")]
|
||||||
[Authorize(Policy = Policies.FirstTimeSetupOrElevated)]
|
[Authorize(Policy = Policies.FirstTimeSetupOrElevated)]
|
||||||
[ProducesResponseType(StatusCodes.Status204NoContent)]
|
[ProducesResponseType(StatusCodes.Status204NoContent)]
|
||||||
public ActionResult UpdateMediaEncoderPath([FromForm, Required] MediaEncoderPathDto mediaEncoderPath)
|
public ActionResult UpdateMediaEncoderPath([FromBody, Required] MediaEncoderPathDto mediaEncoderPath)
|
||||||
{
|
{
|
||||||
_mediaEncoder.UpdateEncoderPath(mediaEncoderPath.Path, mediaEncoderPath.PathType);
|
_mediaEncoder.UpdateEncoderPath(mediaEncoderPath.Path, mediaEncoderPath.PathType);
|
||||||
return NoContent();
|
return NoContent();
|
||||||
|
|
|
@ -1,3 +1,4 @@
|
||||||
|
using System.ComponentModel.DataAnnotations;
|
||||||
using System.Linq;
|
using System.Linq;
|
||||||
using System.Threading.Tasks;
|
using System.Threading.Tasks;
|
||||||
using Jellyfin.Api.Constants;
|
using Jellyfin.Api.Constants;
|
||||||
|
@ -64,21 +65,16 @@ namespace Jellyfin.Api.Controllers
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Sets the initial startup wizard configuration.
|
/// Sets the initial startup wizard configuration.
|
||||||
/// </summary>
|
/// </summary>
|
||||||
/// <param name="uiCulture">The UI language culture.</param>
|
/// <param name="startupConfiguration">The updated startup configuration.</param>
|
||||||
/// <param name="metadataCountryCode">The metadata country code.</param>
|
|
||||||
/// <param name="preferredMetadataLanguage">The preferred language for metadata.</param>
|
|
||||||
/// <response code="204">Configuration saved.</response>
|
/// <response code="204">Configuration saved.</response>
|
||||||
/// <returns>A <see cref="NoContentResult"/> indicating success.</returns>
|
/// <returns>A <see cref="NoContentResult"/> indicating success.</returns>
|
||||||
[HttpPost("Configuration")]
|
[HttpPost("Configuration")]
|
||||||
[ProducesResponseType(StatusCodes.Status204NoContent)]
|
[ProducesResponseType(StatusCodes.Status204NoContent)]
|
||||||
public ActionResult UpdateInitialConfiguration(
|
public ActionResult UpdateInitialConfiguration([FromBody, Required] StartupConfigurationDto startupConfiguration)
|
||||||
[FromForm] string? uiCulture,
|
|
||||||
[FromForm] string? metadataCountryCode,
|
|
||||||
[FromForm] string? preferredMetadataLanguage)
|
|
||||||
{
|
{
|
||||||
_config.Configuration.UICulture = uiCulture;
|
_config.Configuration.UICulture = startupConfiguration.UICulture;
|
||||||
_config.Configuration.MetadataCountryCode = metadataCountryCode;
|
_config.Configuration.MetadataCountryCode = startupConfiguration.MetadataCountryCode;
|
||||||
_config.Configuration.PreferredMetadataLanguage = preferredMetadataLanguage;
|
_config.Configuration.PreferredMetadataLanguage = startupConfiguration.PreferredMetadataLanguage;
|
||||||
_config.SaveConfiguration();
|
_config.SaveConfiguration();
|
||||||
return NoContent();
|
return NoContent();
|
||||||
}
|
}
|
||||||
|
@ -86,16 +82,15 @@ namespace Jellyfin.Api.Controllers
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Sets remote access and UPnP.
|
/// Sets remote access and UPnP.
|
||||||
/// </summary>
|
/// </summary>
|
||||||
/// <param name="enableRemoteAccess">Enable remote access.</param>
|
/// <param name="startupRemoteAccessDto">The startup remote access dto.</param>
|
||||||
/// <param name="enableAutomaticPortMapping">Enable UPnP.</param>
|
|
||||||
/// <response code="204">Configuration saved.</response>
|
/// <response code="204">Configuration saved.</response>
|
||||||
/// <returns>A <see cref="NoContentResult"/> indicating success.</returns>
|
/// <returns>A <see cref="NoContentResult"/> indicating success.</returns>
|
||||||
[HttpPost("RemoteAccess")]
|
[HttpPost("RemoteAccess")]
|
||||||
[ProducesResponseType(StatusCodes.Status204NoContent)]
|
[ProducesResponseType(StatusCodes.Status204NoContent)]
|
||||||
public ActionResult SetRemoteAccess([FromForm] bool enableRemoteAccess, [FromForm] bool enableAutomaticPortMapping)
|
public ActionResult SetRemoteAccess([FromBody, Required] StartupRemoteAccessDto startupRemoteAccessDto)
|
||||||
{
|
{
|
||||||
_config.Configuration.EnableRemoteAccess = enableRemoteAccess;
|
_config.Configuration.EnableRemoteAccess = startupRemoteAccessDto.EnableRemoteAccess;
|
||||||
_config.Configuration.EnableUPnP = enableAutomaticPortMapping;
|
_config.Configuration.EnableUPnP = startupRemoteAccessDto.EnableAutomaticPortMapping;
|
||||||
_config.SaveConfiguration();
|
_config.SaveConfiguration();
|
||||||
return NoContent();
|
return NoContent();
|
||||||
}
|
}
|
||||||
|
@ -131,7 +126,7 @@ namespace Jellyfin.Api.Controllers
|
||||||
/// </returns>
|
/// </returns>
|
||||||
[HttpPost("User")]
|
[HttpPost("User")]
|
||||||
[ProducesResponseType(StatusCodes.Status204NoContent)]
|
[ProducesResponseType(StatusCodes.Status204NoContent)]
|
||||||
public async Task<ActionResult> UpdateStartupUser([FromForm] StartupUserDto startupUserDto)
|
public async Task<ActionResult> UpdateStartupUser([FromBody] StartupUserDto startupUserDto)
|
||||||
{
|
{
|
||||||
var user = _userManager.Users.First();
|
var user = _userManager.Users.First();
|
||||||
|
|
||||||
|
|
|
@ -0,0 +1,22 @@
|
||||||
|
using System.ComponentModel.DataAnnotations;
|
||||||
|
|
||||||
|
namespace Jellyfin.Api.Models.StartupDtos
|
||||||
|
{
|
||||||
|
/// <summary>
|
||||||
|
/// Startup remote access dto.
|
||||||
|
/// </summary>
|
||||||
|
public class StartupRemoteAccessDto
|
||||||
|
{
|
||||||
|
/// <summary>
|
||||||
|
/// Gets or sets a value indicating whether enable remote access.
|
||||||
|
/// </summary>
|
||||||
|
[Required]
|
||||||
|
public bool EnableRemoteAccess { get; set; }
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// Gets or sets a value indicating whether enable automatic port mapping.
|
||||||
|
/// </summary>
|
||||||
|
[Required]
|
||||||
|
public bool EnableAutomaticPortMapping { get; set; }
|
||||||
|
}
|
||||||
|
}
|
Loading…
Reference in New Issue