From d2a57496e566454ce8af66d09a3b8f0e810c2305 Mon Sep 17 00:00:00 2001 From: Bjarke Berg Date: Mon, 23 Nov 2020 08:08:52 +0100 Subject: [PATCH] Fix issue found in test. Return IActionResult instead of HttpResponseMessage --- .../Controllers/EntityController.cs | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/src/Umbraco.Web.BackOffice/Controllers/EntityController.cs b/src/Umbraco.Web.BackOffice/Controllers/EntityController.cs index 5b104d2f9b..28c077a63c 100644 --- a/src/Umbraco.Web.BackOffice/Controllers/EntityController.cs +++ b/src/Umbraco.Web.BackOffice/Controllers/EntityController.cs @@ -250,7 +250,7 @@ namespace Umbraco.Web.BackOffice.Controllers /// The culture to fetch the URL for /// The URL or path to the item [DetermineAmbiguousActionByPassingParameters] - public HttpResponseMessage GetUrl(Udi udi, string culture = "*") + public IActionResult GetUrl(Udi udi, string culture = "*") { var intId = _entityService.GetId(udi); if (!intId.Success) @@ -284,7 +284,7 @@ namespace Umbraco.Web.BackOffice.Controllers /// We are not restricting this with security because there is no sensitive data /// [DetermineAmbiguousActionByPassingParameters] - public HttpResponseMessage GetUrl(int id, UmbracoEntityTypes type, string culture = null) + public IActionResult GetUrl(int id, UmbracoEntityTypes type, string culture = null) { culture = culture ?? ClientCulture(); @@ -297,10 +297,7 @@ namespace Umbraco.Web.BackOffice.Controllers { returnUrl = foundUrl; - return new HttpResponseMessage(HttpStatusCode.OK) - { - Content = new StringContent(returnUrl) - }; + return Ok(returnUrl); } } @@ -314,10 +311,7 @@ namespace Umbraco.Web.BackOffice.Controllers returnUrl = "/" + string.Join("/", ancestors.Select(x => x.Name)); - return new HttpResponseMessage(HttpStatusCode.OK) - { - Content = new StringContent(returnUrl) - }; + return Ok(returnUrl); }