HAM/Berufsschule_HAM
1
0
Fork 1
mirror of https://github.com/LD-Reborn/Berufsschule_HAM.git synced 2025-12-20 06:51:55 +00:00
Code Issues Packages Projects 1 Releases Wiki Activity

Use new response model for create call

Browse Source
This commit is contained in:
anomny
2025-10-06 21:05:58 +02:00
parent 89d7cb6315
commit ba96fe0b22
1 changed files with 13 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/Controllers/AssetsController.cs
View File

@@ -37,12 +37,16 @@ public class AssetsController : Controller
}
[HttpPost("Create")]
public async Task<bool> Create(AssetsCreateRequestModel assetModel)
public async Task<AssetsResponseModel> 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")]