Browse Source

Update SessionController.cs

Removed unnecessary construction of POCO to represent data from POST body
BrianCArnold 4 years ago
parent
commit
80fe48fda9
1 changed files with 3 additions and 5 deletions
  1. 3 5
      Jellyfin.Api/Controllers/SessionController.cs

+ 3 - 5
Jellyfin.Api/Controllers/SessionController.cs

@@ -323,12 +323,10 @@ namespace Jellyfin.Api.Controllers
             [FromRoute, Required] string sessionId,
             [FromBody, Required] MessageCommand command)
         {
-            var nullCorrectedCommand = new MessageCommand
+            if (string.IsNullOrWhiteSpace(command.Header)) 
             {
-                Header = string.IsNullOrWhiteSpace(command.Header) ? "Message from Server" : command.Header,
-                TimeoutMs = command.TimeoutMs,
-                Text = command.Text
-            };
+                command.Header =  "Message from Server";
+            }
 
             _sessionManager.SendMessageCommand(RequestHelpers.GetSession(_sessionManager, _authContext, Request).Id, sessionId, nullCorrectedCommand, CancellationToken.None);