From ba96fe0b228cc5aa4e2dc7df60aa939f044135a9 Mon Sep 17 00:00:00 2001 From: anomny Date: Mon, 6 Oct 2025 21:05:58 +0200 Subject: [PATCH] Use new response model for create call --- src/Controllers/AssetsController.cs | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/src/Controllers/AssetsController.cs b/src/Controllers/AssetsController.cs index 5305d59..b5072a1 100644 --- a/src/Controllers/AssetsController.cs +++ b/src/Controllers/AssetsController.cs @@ -37,12 +37,16 @@ public class AssetsController : Controller } [HttpPost("Create")] - public async Task Create(AssetsCreateRequestModel assetModel) + public async Task Create(AssetsCreateRequestModel assetModel) { + AssetsResponseModel result; if (assetModel is null) { - _logger.LogError("Unable to create an asset because the AssetModel is null."); - return false; + result = new AssetsResponseModel( + successful: false, + exception: "Unable to create an asset because the AssetsCreateRequestModel is null."); + + return result; } try @@ -63,7 +67,7 @@ public class AssetsController : Controller if (assetModel.Location != null) { attributeSet.Add(new LdapAttribute("l", assetModel.Location)); - } + } if (assetModel.Owner != null) { var ownerDn = $"uid={assetModel.Owner}"; @@ -108,14 +112,14 @@ public class AssetsController : Controller } await _ldap.CreateAsset(attributeSet); - - return true; + result = new AssetsResponseModel(successful: true); } catch (Exception e) { - _logger.LogError($"Unable to create an asset because of the exception: {e.Message}", e); - return false; - } + result = new AssetsResponseModel(successful: false, exception: e.Message); + } + + return result; } [HttpDelete("Delete")]