Move id outside of the Put endpoint body for categories

Author: deyvidnenchev

NorthwindCRUD/Controllers/CategoriesController.cs

@@ -198,16 +198,16 @@ public ActionResult<CategoryDetailsDto> Create(CategoryDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<CategoryDto> Update(CategoryDto model)
+        public ActionResult<CategoryDto> Update(int id, CategoryUpdateDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
-                    var mappedModel = this.mapper.Map<CategoryDto, CategoryDb>(model);
-                    var category = this.categoryService.Update(mappedModel);
+                    var mappedModel = this.mapper.Map<CategoryUpdateDto, CategoryDb>(model);
+                    var category = this.categoryService.Update(id, mappedModel);
 
                     if (category != null)
                     {

NorthwindCRUD/ControllersGraphQL/CategoryController.cs

@@ -50,10 +50,10 @@ public CategoryDto Create(CategoryDto model)
         }
 
         [Mutation]
-        public CategoryDto? Update(CategoryDto model)
+        public CategoryDto? Update(int id, CategoryUpdateDto model)
         {
-            var mappedModel = this.mapper.Map<CategoryDto, CategoryDb>(model);
-            var category = this.categoryService.Update(mappedModel);
+            var mappedModel = this.mapper.Map<CategoryUpdateDto, CategoryDb>(model);
+            var category = this.categoryService.Update(id, mappedModel);
 
             return category != null ? this.mapper.Map<CategoryDb, CategoryDto>(category) : null;
         }

NorthwindCRUD/Helpers/MappingProfiles.cs

@@ -9,6 +9,7 @@ public class MappingProfiles : Profile
         public MappingProfiles()
         {
             CreateMap<CategoryDto, CategoryDb>().ReverseMap();
+            CreateMap<CategoryUpdateDto, CategoryDb>().ReverseMap();
             CreateMap<CategoryDb, CategoryDetailsDto>();
 
             CreateMap<ProductDto, ProductDb>().ReverseMap();

NorthwindCRUD/Models/Dtos/CategoryUpdateDto.cs

@@ -0,0 +1,14 @@
+using System.ComponentModel.DataAnnotations;
+
+namespace NorthwindCRUD.Models.Dtos
+{
+    public class CategoryUpdateDto
+    {
+        [StringLength(500, ErrorMessage = "Description cannot exceed 500 characters.")]
+        public string Description { get; set; }
+
+        [Required(ErrorMessage = "Name is required.")]
+        [StringLength(50, MinimumLength = 3, ErrorMessage = "Name must be between 3 and 50 characters.")]
+        public string Name { get; set; }
+    }
+}

NorthwindCRUD/Services/CategoryService.cs

@@ -48,9 +48,9 @@ public CategoryDb Create(CategoryDb model)
             return categoryEntity.Entity;
         }
 
-        public CategoryDb? Update(CategoryDb model)
+        public CategoryDb? Update(int id, CategoryDb model)
         {
-            var categoryEntity = this.dataContext.Categories.FirstOrDefault(c => c.CategoryId == model.CategoryId);
+            var categoryEntity = this.dataContext.Categories.FirstOrDefault(c => c.CategoryId == id);
             if (categoryEntity != null)
             {
                 categoryEntity.Description = model.Description != null ? model.Description : categoryEntity.Description;