Use transactions

src/Imprink.WebApi/Controllers/Products/CategoriesController.cs

@@ -12,7 +12,6 @@ public class CategoriesController(IMediator mediator) : ControllerBase
     [HttpGet]
     public async Task<ActionResult<IEnumerable<CategoryDto>>> GetCategories([FromQuery] GetCategoriesQuery query)
     {
-        var result = await mediator.Send(query);
-        return Ok(result);
+        return Ok(await mediator.Send(query));
     }
 }

src/Imprink.WebApi/Controllers/Products/ProductVariantsController.cs

@@ -13,7 +13,6 @@ public class ProductVariantsController(IMediator mediator) : ControllerBase
     public async Task<ActionResult<IEnumerable<ProductVariantDto>>> GetProductVariants(
         [FromQuery] GetProductVariantsQuery query)
     {
-        var result = await mediator.Send(query);
-        return Ok(result);
+        return Ok(await mediator.Send(query));
     }
 }

src/Imprink.WebApi/Controllers/Products/ProductsController.cs

@@ -17,7 +17,7 @@ public class ProductsController(IMediator mediator) : ControllerBase
     public async Task<ActionResult<PagedResultDto<ProductDto>>> GetProducts(
         [FromQuery] ProductFilterParameters filterParameters)
     {
-        var result = await mediator.Send(new GetProductsQuery { FilterParameters = filterParameters });
+        var result = await mediator.Send(new GetProductsQuery { FilterParameters = filterParameters});
         return Ok(result);
     }
 }

src/Imprink.WebApi/Controllers/Users/UserController.cs

@@ -27,7 +27,6 @@ public class UserController(IMediator mediator) : ControllerBase
         };
         
         await mediator.Send(new SyncUserCommand(auth0User));
-
         return Ok("User Synced.");
     }
 }

src/Imprink.WebApi/Controllers/Users/UserRoleController.cs

@@ -14,35 +14,21 @@ public class UserRoleController(IMediator mediator) : ControllerBase
     [HttpGet("me")]
     public async Task<IActionResult> GetMyRoles()
     {
-        var claims = User.Claims as Claim[] ?? User.Claims.ToArray();
-        var sub = claims.FirstOrDefault(c => c.Type == ClaimTypes.NameIdentifier)?.Value ?? string.Empty;
-        
-        var myRoles = await mediator.Send(new GetUserRolesCommand(sub));
-        
-        return Ok(myRoles);
+        var sub = User.FindFirst(ClaimTypes.NameIdentifier)?.Value ?? string.Empty;
+        return Ok(await mediator.Send(new GetUserRolesCommand(sub)));
     }
     
     [Authorize(Roles = "Admin")]
     [HttpPost("set")]
     public async Task<IActionResult> SetUserRole(SetUserRoleCommand command)
     {
-        var userRole = await mediator.Send(command);
-        
-        if (userRole == null)
-            return BadRequest();
-        
-        return Ok(userRole);
+        return Ok(await mediator.Send(command));
     }
     
     [Authorize(Roles = "Admin")]
     [HttpPost("unset")]
     public async Task<IActionResult> UnsetUserRole(DeleteUserRoleCommand command)
     {
-        var userRole = await mediator.Send(command);
-        
-        if (userRole == null)
-            return BadRequest();
-        
-        return Ok(userRole);
+        return Ok(await mediator.Send(command));
     }
 }