Added club POST and PUT

PathfinderHonorManager.Tests/PathfinderHonorManager.Tests.csproj

@@ -17,11 +17,11 @@
     <PackageReference Include="NUnit" Version="4.3.2" />
     <PackageReference Include="NUnit3TestAdapter" Version="5.0.0" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.13.0" />
-    <PackageReference Include="Microsoft.Data.Sqlite.Core" Version="9.0.3" />
-    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="9.0.3" />
+    <PackageReference Include="Microsoft.Data.Sqlite.Core" Version="9.0.4" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="9.0.4" />
     <PackageReference Include="FluentValidation.Validators.UnitTestExtension" Version="1.11.0.2" />
     <PackageReference Include="FluentValidation.DependencyInjectionExtensions" Version="11.11.0" />
-    <PackageReference Include="Microsoft.EntityFrameworkCore.InMemory" Version="9.0.3" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore.InMemory" Version="9.0.4" />
   </ItemGroup>
 
   <ItemGroup>

PathfinderHonorManager/Controllers/ClubController.cs

@@ -9,6 +9,9 @@
 using PathfinderHonorManager.Service.Interfaces;
 using Microsoft.Extensions.Logging;
 using System.Linq;
+using PathfinderHonorManager.Dto.Incoming;
+using FluentValidation;
+using Microsoft.EntityFrameworkCore;
 
 namespace PathfinderHonorManager.Controllers
 {
@@ -19,15 +22,20 @@ namespace PathfinderHonorManager.Controllers
     [Consumes("application/json")]
     [ProducesResponseType(StatusCodes.Status401Unauthorized)]
     [ProducesResponseType(StatusCodes.Status403Forbidden)]
-    public class ClubsController : ControllerBase
+    public class ClubsController : CustomApiController
     {
         private readonly IClubService _clubService;
         private readonly ILogger<ClubsController> _logger;
+        private readonly IValidator<ClubDto> _validator;
 
-        public ClubsController(IClubService clubService, ILogger<ClubsController> logger)
+        public ClubsController(
+            IClubService clubService, 
+            ILogger<ClubsController> logger,
+            IValidator<ClubDto> validator)
         {
             _clubService = clubService;
             _logger = logger;
+            _validator = validator;
         }
 
         // GET Clubs
@@ -80,7 +88,7 @@ public async Task<ActionResult<IEnumerable<Club>>> GetClubs(CancellationToken to
         /// <returns></returns>
         [ProducesResponseType(StatusCodes.Status200OK)]
         [ProducesResponseType(StatusCodes.Status404NotFound)]
-        [HttpGet("{id:guid}")]
+        [HttpGet("{id:guid}", Name = "GetClubById")]
         public async Task<IActionResult> GetByIdAsync(Guid id, CancellationToken token)
         {
             _logger.LogInformation("Getting club with ID {ClubId}", id);
@@ -95,5 +103,54 @@ public async Task<IActionResult> GetByIdAsync(Guid id, CancellationToken token)
             _logger.LogInformation("Retrieved club with ID {ClubId}", id);
             return Ok(club);
         }
+
+        [Authorize("CreateClubs")]
+        [ProducesResponseType(StatusCodes.Status201Created)]
+        [ProducesResponseType(StatusCodes.Status400BadRequest)]
+        [HttpPost]
+        public async Task<IActionResult> CreateAsync([FromBody] ClubDto club, CancellationToken token)
+        {
+            _logger.LogInformation("Creating new club with code {ClubCode}", club.ClubCode);
+
+            try
+            {
+                var createdClub = await _clubService.CreateAsync(club, token);
+                return CreatedAtRoute("GetClubById", new { id = createdClub.ClubID }, createdClub);
+            }
+            catch (FluentValidation.ValidationException ex)
+            {
+                _logger.LogWarning(ex, "Validation failed while creating club with code {ClubCode}", club.ClubCode);
+                UpdateModelState(ex);
+                return ValidationProblem(ModelState);
+            }
+        }
+
+        [Authorize("UpdateClubs")]
+        [ProducesResponseType(StatusCodes.Status200OK)]
+        [ProducesResponseType(StatusCodes.Status400BadRequest)]
+        [ProducesResponseType(StatusCodes.Status404NotFound)]
+        [HttpPut("{id:guid}")]
+        public async Task<IActionResult> UpdateAsync(Guid id, [FromBody] ClubDto club, CancellationToken token)
+        {
+            _logger.LogInformation("Updating club with ID {ClubId}", id);
+
+            try
+            {
+                var updatedClub = await _clubService.UpdateAsync(id, club, token);
+                if (updatedClub == default)
+                {
+                    _logger.LogWarning("Club with ID {ClubId} not found for update", id);
+                    return NotFound();
+                }
+
+                return Ok(updatedClub);
+            }
+            catch (FluentValidation.ValidationException ex)
+            {
+                _logger.LogWarning(ex, "Validation failed while updating club with ID {ClubId}", id);
+                UpdateModelState(ex);
+                return ValidationProblem(ModelState);
+            }
+        }
     }
 }

PathfinderHonorManager/Mapping/AutoMapperConfig.cs

@@ -55,6 +55,7 @@ private void RegisterHonorMappings()
         private void RegisterClubMappings()
         {
             CreateMap<Club, Outgoing.ClubDto>();
+            CreateMap<Incoming.ClubDto, Club>();
         }
 
         private void RegisterPathfinderHonorMappings()

PathfinderHonorManager/PathfinderHonorManager.csproj

@@ -11,23 +11,23 @@
   <ItemGroup>
     <PackageReference Include="FluentValidation" Version="11.11.0" />
     <PackageReference Include="FluentValidation.DependencyInjectionExtensions" Version="11.11.0" />
-    <PackageReference Include="Microsoft.EntityFrameworkCore" Version="9.0.3" />
-    <PackageReference Include="Microsoft.Extensions.Logging" Version="9.0.3" />
-    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="9.0.3" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore" Version="9.0.4" />
+    <PackageReference Include="Microsoft.Extensions.Logging" Version="9.0.4" />
+    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="9.0.4" />
     <PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
     <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="9.0.2" />
     <PackageReference Include="Npgsql" Version="9.0.3" />
-    <PackageReference Include="Microsoft.EntityFrameworkCore.Proxies" Version="9.0.3" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore.Proxies" Version="9.0.4" />
     <PackageReference Include="AutoMapper.Extensions.Microsoft.DependencyInjection" Version="12.0.1" />
     <PackageReference Include="FluentValidation.AspNetCore" Version="11.3.0" />
-    <PackageReference Include="Swashbuckle.AspNetCore" Version="8.1.0" />
-    <PackageReference Include="Microsoft.Extensions.Diagnostics.HealthChecks" Version="9.0.3" /> 
+    <PackageReference Include="Swashbuckle.AspNetCore" Version="8.1.1" />
+    <PackageReference Include="Microsoft.Extensions.Diagnostics.HealthChecks" Version="9.0.4" /> 
     <PackageReference Include="AspNetCore.HealthChecks.NpgSql" Version="9.0.0" />
     <PackageReference Include="Microsoft.Extensions.Logging.AzureAppServices" Version="9.0.2" />
-    <PackageReference Include="Microsoft.AspNetCore.Authentication.MicrosoftAccount" Version="9.0.3" />
-    <PackageReference Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="9.0.3" />
-    <PackageReference Include="Microsoft.AspNetCore.Authorization" Version="9.0.3" />
-    <PackageReference Include="Microsoft.Identity.Web" Version="3.6.2" />
+    <PackageReference Include="Microsoft.AspNetCore.Authentication.MicrosoftAccount" Version="9.0.4" />
+    <PackageReference Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="9.0.4" />
+    <PackageReference Include="Microsoft.AspNetCore.Authorization" Version="9.0.4" />
+    <PackageReference Include="Microsoft.Identity.Web" Version="3.8.4" />
     <PackageReference Include="Microsoft.ApplicationInsights" Version="2.23.0" />
     <PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.22.0" />
   </ItemGroup>

PathfinderHonorManager/Service/ClubService.cs

@@ -5,11 +5,13 @@
 using System.Threading;
 using System.Threading.Tasks;
 using Outgoing = PathfinderHonorManager.Dto.Outgoing;
+using Incoming = PathfinderHonorManager.Dto.Incoming;
 using PathfinderHonorManager.Model;
 using PathfinderHonorManager.DataAccess;
 using PathfinderHonorManager.Service.Interfaces;
 using System.Collections.Generic;
 using System.Linq;
+using FluentValidation;
 
 namespace PathfinderHonorManager.Service
 {
@@ -79,5 +81,53 @@ public ClubService(
             _logger.LogInformation("Retrieved club: {ClubName} (Code: {ClubCode})", entity.Name, code);
             return _mapper.Map<Outgoing.ClubDto>(entity);
         }
+
+        public async Task<Outgoing.ClubDto> CreateAsync(Incoming.ClubDto club, CancellationToken token)
+        {
+            _logger.LogInformation("Creating new club with code: {ClubCode}", club.ClubCode);
+
+            try
+            {
+                var entity = _mapper.Map<Club>(club);
+                await _dbContext.Clubs.AddAsync(entity, token);
+                await _dbContext.SaveChangesAsync(token);
+
+                _logger.LogInformation("Created club: {ClubName} (ID: {ClubId})", entity.Name, entity.ClubID);
+                return _mapper.Map<Outgoing.ClubDto>(entity);
+            }
+            catch (DbUpdateException ex)
+            {
+                _logger.LogError(ex, "Database error while creating club with code {ClubCode}", club.ClubCode);
+                throw new ValidationException("Failed to create club. The club code may already be in use.");
+            }
+        }
+
+        public async Task<Outgoing.ClubDto> UpdateAsync(Guid id, Incoming.ClubDto club, CancellationToken token)
+        {
+            _logger.LogInformation("Updating club with ID: {ClubId}", id);
+
+            try
+            {
+                var entity = await _dbContext.Clubs
+                    .SingleOrDefaultAsync(c => c.ClubID == id, token);
+
+                if (entity == default)
+                {
+                    _logger.LogWarning("Club with ID {ClubId} not found", id);
+                    return default;
+                }
+
+                _mapper.Map(club, entity);
+                await _dbContext.SaveChangesAsync(token);
+
+                _logger.LogInformation("Updated club: {ClubName} (ID: {ClubId})", entity.Name, id);
+                return _mapper.Map<Outgoing.ClubDto>(entity);
+            }
+            catch (DbUpdateException ex)
+            {
+                _logger.LogError(ex, "Database error while updating club with ID {ClubId}", id);
+                throw new ValidationException("Failed to update club. The club code may already be in use.");
+            }
+        }
     }
 }

PathfinderHonorManager/Service/Interfaces/IClubService.cs

@@ -3,6 +3,7 @@
 using System.Threading;
 using System.Threading.Tasks;
 using Outgoing = PathfinderHonorManager.Dto.Outgoing;
+using Incoming = PathfinderHonorManager.Dto.Incoming;
 
 namespace PathfinderHonorManager.Service.Interfaces
 {
@@ -18,5 +19,14 @@ public interface IClubService
 
         Task<ICollection<Outgoing.ClubDto>> GetAllAsync(
             CancellationToken token);
+
+        Task<Outgoing.ClubDto> CreateAsync(
+            Incoming.ClubDto club,
+            CancellationToken token);
+
+        Task<Outgoing.ClubDto> UpdateAsync(
+            Guid id,
+            Incoming.ClubDto club,
+            CancellationToken token);
     }
 }