ChronosXYZ/InstructorsListAppTA
1
0
Fork 0
mirror of https://github.com/ChronosX88/InstructorsListAppTA.git synced 2024-12-04 22:52:19 +00:00
Code Issues Projects Releases Wiki Activity

[backend] fix: Fix issues with commit transaction, add "exists" condition in UpdateInstructorEntry

Browse Source
This commit is contained in:
ChronosX88 2019-09-25 18:40:52 +04:00
parent 7d55a9b7a1
commit 38d05d3a89
Signed by: ChronosXYZ
GPG Key ID: 085A69A82C8C511A
1 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
Controllers/InstructorsDataController.cs
View File

@ -28,7 +28,7 @@ namespace InstructorsListApp.Controllers
if(ModelState.IsValid) {
instructor.Id = Guid.NewGuid().ToString();
databaseContext.Instructors.Add(instructor);
databaseContext.SaveChangesAsync();
databaseContext.SaveChanges();
return Ok(instructor.Id);
}
return BadRequest(ModelState);
@ -37,9 +37,15 @@ namespace InstructorsListApp.Controllers
[HttpPut("{id}")]
public IActionResult UpdateInstructorEntry(string id, [FromBody] Instructor instructor) {
if(ModelState.IsValid) {
databaseContext.Instructors.Update(instructor);
databaseContext.SaveChangesAsync();
return Ok();
instructor.Id = id;
var instructorEntry = databaseContext.Instructors.FirstOrDefault(i => i.Id == id);
if(instructorEntry != null) {
databaseContext.Entry<Instructor>(instructorEntry).State = Microsoft.EntityFrameworkCore.EntityState.Detached;
databaseContext.Instructors.Update(instructor);
databaseContext.SaveChanges();
return Ok();
}
return NotFound("Given instructor isn't found");
}
return BadRequest(ModelState);
}
@ -49,10 +55,10 @@ namespace InstructorsListApp.Controllers
Instructor instructor = databaseContext.Instructors.FirstOrDefault(i => i.Id == id);
if(instructor != null) {
databaseContext.Instructors.Remove(instructor);
databaseContext.SaveChangesAsync();
databaseContext.SaveChanges();
return Ok();
}
return BadRequest("No such instructor");
return NotFound("No such instructor");
}
}
}