Cutlery/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2025-06-03 05:34:16 -04:00
Code Issues Packages Projects Releases Wiki Activity

Added Required attribute to Text property of MessageCommand.

Browse Source
This commit is contained in:
Brian Arnold 2021-03-29 10:10:44 -04:00
parent 998833ea6e
commit ca25301e64
2 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
Jellyfin.Api/Controllers/SessionController.cs
View File

@ -323,13 +323,6 @@ namespace Jellyfin.Api.Controllers
[FromRoute, Required] string sessionId, [FromRoute, Required] string sessionId,
[FromBody, Required] MessageCommand command) [FromBody, Required] MessageCommand command)
{ {
// Need to check if message.Text is null, since [Required] can't be applied to properties of a deserialized object.
if (string.IsNullOrWhiteSpace(command.Text))
{
throw new ArgumentNullException("Message Text may not be empty.");
}
var nullCorrectedCommand = new MessageCommand var nullCorrectedCommand = new MessageCommand
{ {
Header = string.IsNullOrWhiteSpace(command.Header) ? "Message from Server" : command.Header, Header = string.IsNullOrWhiteSpace(command.Header) ? "Message from Server" : command.Header,

4
MediaBrowser.Model/Session/MessageCommand.cs
View File

@ -1,12 +1,14 @@
#nullable disable #nullable disable
#pragma warning disable CS1591 #pragma warning disable CS1591
using System.ComponentModel.DataAnnotations;
namespace MediaBrowser.Model.Session namespace MediaBrowser.Model.Session
{ {
public class MessageCommand public class MessageCommand
{ {
public string Header { get; set; } public string Header { get; set; }
[Required(AllowEmptyStrings = false)]
public string Text { get; set; } public string Text { get; set; }
public long? TimeoutMs { get; set; } public long? TimeoutMs { get; set; }