mirror of
https://github.com/claunia/marechai.git
synced 2025-12-16 19:14:25 +00:00
Refactor controllers to use route parameters for UpdateAsync methods and improve consistency in model retrieval
This commit is contained in:
@@ -80,18 +80,18 @@ public class MagazineIssuesController(MarechaiContext context) : ControllerBase
|
||||
})
|
||||
.FirstOrDefaultAsync();
|
||||
|
||||
[HttpPost]
|
||||
[HttpPut("{id:long}")]
|
||||
[Authorize(Roles = "Admin,UberAdmin")]
|
||||
[ProducesResponseType(StatusCodes.Status200OK)]
|
||||
[ProducesResponseType(StatusCodes.Status400BadRequest)]
|
||||
[ProducesResponseType(StatusCodes.Status404NotFound)]
|
||||
[ProducesResponseType(StatusCodes.Status401Unauthorized)]
|
||||
public async Task<ActionResult> UpdateAsync([FromBody] MagazineIssueDto dto)
|
||||
public async Task<ActionResult> UpdateAsync(long id, [FromBody] MagazineIssueDto dto)
|
||||
{
|
||||
string userId = User.FindFirstValue(ClaimTypes.Sid);
|
||||
|
||||
if(userId is null) return Unauthorized();
|
||||
MagazineIssue model = await context.MagazineIssues.FindAsync(dto.Id);
|
||||
MagazineIssue model = await context.MagazineIssues.FindAsync(id);
|
||||
|
||||
if(model is null) return NotFound();
|
||||
|
||||
|
||||
Reference in New Issue
Block a user