Fixed: Mark as Failed errors (#5939)

Originally by Markus McDowall (cherry picked from commit 12fafb24578d39df2eabd2331558b2da961fe8b6)

Co-Authored-By: Mark McDowall <markus101@users.noreply.github.com>

Fixes: #5907
Fixes: #5938

frontend/src/Store/Actions/historyActions.js

@@ -232,11 +232,9 @@ export const actionHandlers = handleThunks({
     }));
 
     const promise = createAjaxRequest({
-      url: '/history/failed',
+      url: `/history/failed/${id}`,
       method: 'POST',
-      data: {
+      dataType: 'json'
-        id
-      }
     }).request;
 
     promise.done(() => {

frontend/src/Store/Actions/movieHistoryActions.js

@@ -77,11 +77,8 @@ export const actionHandlers = handleThunks({
     } = payload;
 
     const promise = createAjaxRequest({
-      url: '/history/failed',
+      url: `/history/failed/${historyId}`,
-      method: 'POST',
+      method: 'POST'
-      data: {
-        id: historyId
-      }
     }).request;
 
     promise.done(() => {

src/Radarr.Api.V3/History/HistoryModule.cs

@@ -1,7 +1,6 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using Nancy;
 using NzbDrone.Core.CustomFormats;
 using NzbDrone.Core.Datastore;
 using NzbDrone.Core.DecisionEngine.Specifications;
@@ -39,6 +38,7 @@ namespace Radarr.Api.V3.History
             Get("/since", x => GetHistorySince());
             Get("/movie", x => GetMovieHistory());
             Post("/failed", x => MarkAsFailed());
+            Post(@"/failed/(?<id>[\d]{1,10})", x => MarkAsFailed((int)x.Id));
         }
 
         protected HistoryResource MapToResource(MovieHistory model, bool includeMovie)
@@ -130,10 +130,18 @@ namespace Radarr.Api.V3.History
             return _historyService.GetByMovieId(movieId, eventType).Select(h => MapToResource(h, includeMovie)).ToList();
         }
 
+        // v4 TODO: Getting the ID from the form is atypical, consider removing.
         private object MarkAsFailed()
         {
             var id = (int)Request.Form.Id;
+
+            return MarkAsFailed(id);
+        }
+
+        private object MarkAsFailed(int id)
+        {
             _failedDownloadService.MarkAsFailed(id);
+
             return new object();
         }
     }