fix(put): update endpoints to include notfound

Author: Svetloslav15

NorthwindCRUD/Controllers/CategoryController.cs

@@ -46,7 +46,6 @@ public ActionResult<CategoryInputModel> GetById(int id)
             try
             {
                 var category = this.categoryService.GetById(id);
-
                 if (category != null)
                 {
                     return Ok(this.mapper.Map<CategoryDb, CategoryInputModel>(category));
@@ -93,7 +92,13 @@ public ActionResult<CategoryInputModel> Update(CategoryInputModel model)
                 {
                     var mappedModel = this.mapper.Map<CategoryInputModel, CategoryDb>(model);
                     var category = this.categoryService.Update(mappedModel);
-                    return Ok(this.mapper.Map<CategoryDb, CategoryInputModel>(category));
+
+                    if (category != null)
+                    {
+                        return Ok(this.mapper.Map<CategoryDb, CategoryInputModel>(category));
+                    }
+
+                    return NotFound();
                 }
 
                 return BadRequest(ModelState);
@@ -112,7 +117,12 @@ public ActionResult<CategoryInputModel> Delete(int id)
             try
             {
                 var category = this.categoryService.Delete(id);
-                return Ok(this.mapper.Map<CategoryDb, CategoryInputModel>(category));
+                if (category != null)
+                {
+                    return Ok(this.mapper.Map<CategoryDb, CategoryInputModel>(category));
+                }
+
+                return NotFound();
             }
             catch (Exception error)
             {

NorthwindCRUD/Controllers/CustomerController.cs

@@ -3,6 +3,7 @@
     using AutoMapper;
     using Microsoft.AspNetCore.Authorization;
     using Microsoft.AspNetCore.Mvc;
+    using NorthwindCRUD.Models.Contracts;
     using NorthwindCRUD.Models.DbModels;
     using NorthwindCRUD.Models.InputModels;
     using NorthwindCRUD.Services;
@@ -92,7 +93,13 @@ public ActionResult<CustomerInputModel> Update(CustomerInputModel model)
                 {
                     var mappedModel = this.mapper.Map<CustomerInputModel, CustomerDb>(model);
                     var customer = this.customerService.Update(mappedModel);
-                    return Ok(this.mapper.Map<CustomerDb, CustomerInputModel>(customer));
+
+                    if (customer != null)
+                    {
+                        return Ok(this.mapper.Map<CustomerDb, CustomerInputModel>(customer));
+                    }
+
+                    return NotFound();
                 }
 
                 return BadRequest(ModelState);
@@ -111,7 +118,12 @@ public ActionResult<CustomerInputModel> Delete(string id)
             try
             {
                 var customer = this.customerService.Delete(id);
-                return Ok(this.mapper.Map<CustomerDb, CustomerInputModel>(customer));
+                if (customer != null)
+                {
+                    return Ok(this.mapper.Map<CustomerDb, CustomerInputModel>(customer));
+                }
+
+                return NotFound();
             }
             catch (Exception error)
             {

NorthwindCRUD/Controllers/EmployeeController.cs

@@ -3,6 +3,7 @@
     using AutoMapper;
     using Microsoft.AspNetCore.Authorization;
     using Microsoft.AspNetCore.Mvc;
+    using NorthwindCRUD.Models.Contracts;
     using NorthwindCRUD.Models.DbModels;
     using NorthwindCRUD.Models.InputModels;
     using NorthwindCRUD.Services;
@@ -111,7 +112,13 @@ public ActionResult<EmployeeInputModel> Delete(int id)
             try
             {
                 var employee = this.employeeService.Delete(id);
-                return Ok(this.mapper.Map<EmployeeDb, EmployeeInputModel>(employee));
+
+                if (employee != null)
+                {
+                    return Ok(this.mapper.Map<EmployeeDb, EmployeeInputModel>(employee));
+                }
+
+                return NotFound();
             }
             catch (Exception error)
             {

NorthwindCRUD/Controllers/OrderController.cs

@@ -92,7 +92,12 @@ public ActionResult<OrderInputModel> Update(OrderInputModel model)
                 {
                     var mappedModel = this.mapper.Map<OrderInputModel, OrderDb>(model);
                     var order = this.orderService.Update(mappedModel);
-                    return Ok(this.mapper.Map<OrderDb, OrderInputModel>(order));
+                    if (order != null)
+                    {
+                        return Ok(this.mapper.Map<OrderDb, OrderInputModel>(order));
+                    }
+
+                    return NotFound();
                 }
 
                 return BadRequest(ModelState);
@@ -111,7 +116,12 @@ public ActionResult<OrderInputModel> Delete(int id)
             try
             {
                 var order = this.orderService.Delete(id);
-                return Ok(this.mapper.Map<OrderDb, OrderInputModel>(order));
+                if (order != null)
+                {
+                    return Ok(this.mapper.Map<OrderDb, OrderInputModel>(order));
+                }
+
+                return NotFound();
             }
             catch (Exception error)
             {