Tech debt| use new api files and null enhancements

source/backend/api/Areas/Acquisition/Controllers/AcquisitionFileController.cs

@@ -68,6 +68,7 @@ public AcquisitionFileController(IAcquisitionFileService acquisitionService, ICo
         [Produces("application/json")]
         [ProducesResponseType(typeof(AcquisitionFileModel), 200)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetAcquisitionFile(long id)
         {
             // RECOMMENDED - Add valuable metadata to logs
@@ -109,6 +110,7 @@ public IActionResult GetLastUpdatedBy(long id)
         [Produces("application/json")]
         [ProducesResponseType(typeof(AcquisitionFileModel), 200)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult AddAcquisitionFile([FromBody] AcquisitionFileModel model, [FromQuery] string[] userOverrideCodes)
         {
             _logger.LogInformation(
@@ -136,6 +138,7 @@ public IActionResult AddAcquisitionFile([FromBody] AcquisitionFileModel model, [
         [ProducesResponseType(typeof(AcquisitionFileModel), 200)]
         [ProducesResponseType(typeof(ErrorResponseModel), 409)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult UpdateAcquisitionFile(long id, [FromBody] AcquisitionFileModel model, [FromQuery] string[] userOverrideCodes)
         {
             _logger.LogInformation(
@@ -161,6 +164,7 @@ public IActionResult UpdateAcquisitionFile(long id, [FromBody] AcquisitionFileMo
         [Produces("application/json")]
         [ProducesResponseType(typeof(AcquisitionFileModel), 200)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult UpdateAcquisitionFileProperties([FromBody] AcquisitionFileModel acquisitionFileModel, [FromQuery] string[] userOverrideCodes)
         {
             var acquisitionFileEntity = _mapper.Map<Dal.Entities.PimsAcquisitionFile>(acquisitionFileModel);
@@ -177,6 +181,7 @@ public IActionResult UpdateAcquisitionFileProperties([FromBody] AcquisitionFileM
         [Produces("application/json")]
         [ProducesResponseType(typeof(IEnumerable<AcquisitionFilePropertyModel>), 200)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetAcquisitionFileProperties(long id)
         {
             var acquisitionFileProperties = _acquisitionService.GetProperties(id);
@@ -193,6 +198,7 @@ public IActionResult GetAcquisitionFileProperties(long id)
         [Produces("application/json")]
         [ProducesResponseType(typeof(IEnumerable<AcquisitionFileOwnerModel>), 200)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetAcquisitionFileOwners([FromRoute] long id)
         {
             var owners = _acquisitionService.GetOwners(id);
@@ -210,6 +216,7 @@ public IActionResult GetAcquisitionFileOwners([FromRoute] long id)
         [Produces("application/json")]
         [ProducesResponseType(typeof(IEnumerable<AcquisitionFileTeamModel>), 200)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetAcquisitionTeamMembers()
         {
             var team = _acquisitionService.GetTeamMembers();
@@ -243,6 +250,7 @@ public IActionResult GetFileCompensations(long id)
         [Produces("application/json")]
         [ProducesResponseType(typeof(IEnumerable<CompensationFinancialModel>), 200)]
         [SwaggerOperation(Tags = new[] { "comp-req-h120s" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetFileCompReqH120(long id, bool? finalOnly)
         {
             _logger.LogInformation(
@@ -269,6 +277,7 @@ public IActionResult GetFileCompReqH120(long id, bool? finalOnly)
         [Produces("application/json")]
         [ProducesResponseType(typeof(CompensationRequisitionModel), 201)]
         [SwaggerOperation(Tags = new[] { "compensation-requisition" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult AddCompensationRequisition([FromRoute] long id, [FromBody] CompensationRequisitionModel compensationRequisition)
         {
             _logger.LogInformation(
@@ -323,6 +332,7 @@ public IActionResult GetAcquisitionFileExpropriationPayments([FromRoute] long id
         [Produces("application/json")]
         [ProducesResponseType(typeof(ExpropriationPaymentModel), 201)]
         [SwaggerOperation(Tags = new[] { "expropriation-payments" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult AddExpropriationPayment([FromRoute] long id, [FromBody] ExpropriationPaymentModel expropriationPayment)
         {
             _logger.LogInformation(

source/backend/api/Areas/Acquisition/Controllers/AgreementController.cs

@@ -5,6 +5,7 @@
 using Pims.Api.Models.Concepts.AcquisitionFile;
 using Pims.Api.Policies;
 using Pims.Api.Services;
+using Pims.Core.Json;
 using Pims.Dal.Security;
 using Swashbuckle.AspNetCore.Annotations;
 
@@ -67,6 +68,7 @@ public IActionResult GetAcquisitionFileAgreements([FromRoute] long id)
         [Produces("application/json")]
         [ProducesResponseType(typeof(AgreementModel), 200)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult UpdateAcquisitionFileAgreements([FromRoute] long id, [FromBody] List<AgreementModel> agreements)
         {
             var agreementEntities = _mapper.Map<List<Dal.Entities.PimsAgreement>>(agreements);

source/backend/api/Areas/Acquisition/Controllers/ChecklistController.cs

@@ -2,10 +2,12 @@
 using MapsterMapper;
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
+using Pims.Api.Helpers.Exceptions;
 using Pims.Api.Models.Concepts.AcquisitionFile;
 using Pims.Api.Models.Concepts.File;
 using Pims.Api.Policies;
 using Pims.Api.Services;
+using Pims.Core.Json;
 using Pims.Dal.Security;
 using Swashbuckle.AspNetCore.Annotations;
 
@@ -53,6 +55,7 @@ public ChecklistController(IAcquisitionFileService acquisitionService, IMapper m
         [Produces("application/json")]
         [ProducesResponseType(typeof(IEnumerable<FileChecklistItemModel>), 200)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetAcquisitionFileChecklist([FromRoute] long id)
         {
             var checklist = _acquisitionService.GetChecklistItems(id);
@@ -63,15 +66,25 @@ public IActionResult GetAcquisitionFileChecklist([FromRoute] long id)
         /// Update the acquisition file checklist.
         /// </summary>
         /// <returns>The updated checklist items.</returns>
-        [HttpPut("{id:long}/checklist")]
+        [HttpPut("{acquisitionFileId:long}/checklist")]
         [HasPermission(Permissions.AcquisitionFileEdit)]
         [Produces("application/json")]
         [ProducesResponseType(typeof(AcquisitionFileModel), 200)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
-        public IActionResult UpdateAcquisitionFileChecklist([FromBody] AcquisitionFileModel acquisitionFileModel)
+        [TypeFilter(typeof(NullJsonResultFilter))]
+        public IActionResult UpdateAcquisitionFileChecklist(long acquisitionFileId, [FromBody] IList<FileChecklistItemModel> checklistItems)
         {
-            var acquisitionFileEntity = _mapper.Map<Dal.Entities.PimsAcquisitionFile>(acquisitionFileModel);
-            var acquisitionFile = _acquisitionService.UpdateChecklistItems(acquisitionFileEntity);
+
+            foreach (var item in checklistItems)
+            {
+                if (item.FileId != acquisitionFileId)
+                {
+                    throw new BadRequestException("All checklist items file id must match the acquisition file id");
+                }
+            }
+
+            var checklistItemEntities = _mapper.Map<IList<Dal.Entities.PimsAcquisitionChecklistItem>>(checklistItems);
+            var acquisitionFile = _acquisitionService.UpdateChecklistItems(checklistItemEntities);
             return new JsonResult(_mapper.Map<AcquisitionFileModel>(acquisitionFile));
         }
 

source/backend/api/Areas/Acquisition/Controllers/SearchController.cs

@@ -15,6 +15,7 @@ namespace Pims.Api.Areas.Acquisition.Controllers
     using Pims.Api.Policies;
     using Pims.Api.Services;
     using Pims.Core.Extensions;
+    using Pims.Core.Json;
     using Pims.Dal.Entities.Models;
     using Pims.Dal.Security;
     using Swashbuckle.AspNetCore.Annotations;
@@ -66,6 +67,7 @@ public SearchController(IAcquisitionFileService acquisitionService, IMapper mapp
         [ProducesResponseType(typeof(IEnumerable<AcquisitionFileModel>), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetAcquisitionFiles()
         {
             var uri = new Uri(this.Request.GetDisplayUrl());
@@ -83,6 +85,7 @@ public IActionResult GetAcquisitionFiles()
         [ProducesResponseType(typeof(IEnumerable<AcquisitionFileModel>), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "acquisitionfile" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetAcquisitionFiles([FromBody] AcquisitionFilterModel filter)
         {
             // RECOMMENDED - Add valuable metadata to logs

source/backend/api/Areas/Admin/Controllers/AccessRequestController.cs

@@ -4,6 +4,7 @@
 using Pims.Api.Models.Base;
 using Pims.Api.Models.Concepts.AccessRequest;
 using Pims.Api.Policies;
+using Pims.Core.Json;
 using Pims.Dal.Repositories;
 using Pims.Dal.Security;
 using Swashbuckle.AspNetCore.Annotations;
@@ -57,6 +58,7 @@ public AccessRequestController(IAccessRequestRepository accessRequestRepository,
         [ProducesResponseType(typeof(PageModel<AccessRequestModel>), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-access-requests" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetPage(int page = 1, int quantity = 10, string searchText = null, string sort = null)
         {
             if (page < 1)
@@ -94,6 +96,7 @@ public IActionResult GetPage(int page = 1, int quantity = 10, string searchText
         [ProducesResponseType(typeof(ErrorResponseModel), 400)]
         [ProducesResponseType(typeof(ErrorResponseModel), 403)]
         [SwaggerOperation(Tags = new[] { "user" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetAccessRequest(long id)
         {
             var accessRequest = _accessRequestRepository.GetById(id);
@@ -111,6 +114,7 @@ public IActionResult GetAccessRequest(long id)
         [ProducesResponseType(typeof(AccessRequestModel), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-access-requests" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult Delete(long id, [FromBody] AccessRequestModel model)
         {
             var entity = _mapper.Map<Entity.PimsAccessRequest>(model);

source/backend/api/Areas/Admin/Controllers/ClaimController.cs

@@ -4,6 +4,7 @@
 using Pims.Api.Models.Base;
 using Pims.Api.Models.Concepts.Claim;
 using Pims.Api.Policies;
+using Pims.Core.Json;
 using Pims.Dal.Repositories;
 using Pims.Dal.Security;
 using Swashbuckle.AspNetCore.Annotations;
@@ -55,6 +56,7 @@ public ClaimController(IClaimRepository claimRepository, IMapper mapper)
         [ProducesResponseType(typeof(PageModel<ClaimModel>), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-claim" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetClaims(int page = 1, int quantity = 10, string name = null)
         {
             if (page < 1)

source/backend/api/Areas/Admin/Controllers/FinancialCodeController.cs

@@ -5,6 +5,7 @@
 using Pims.Api.Policies;
 using Pims.Api.Services;
 using Pims.Core.Exceptions;
+using Pims.Core.Json;
 using Pims.Dal.Security;
 using Swashbuckle.AspNetCore.Annotations;
 
@@ -49,6 +50,7 @@ public FinancialCodeController(IFinancialCodeService financialCodeService)
         [ProducesResponseType(typeof(IEnumerable<FinancialCodeModel>), 200)]
         [ProducesResponseType(typeof(Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-financialcodes" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetFinancialCodes()
         {
             var allCodes = _financialCodeService.GetAllFinancialCodes();
@@ -67,6 +69,7 @@ public IActionResult GetFinancialCodes()
         [ProducesResponseType(typeof(Models.ErrorResponseModel), 400)]
         [ProducesResponseType(typeof(Models.ErrorResponseModel), 409)]
         [SwaggerOperation(Tags = new[] { "admin-financialcodes" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetFinancialCode(FinancialCodeTypes type, long codeId)
         {
             return new JsonResult(_financialCodeService.GetById(type, codeId));
@@ -84,6 +87,7 @@ public IActionResult GetFinancialCode(FinancialCodeTypes type, long codeId)
         [ProducesResponseType(typeof(Models.ErrorResponseModel), 400)]
         [ProducesResponseType(typeof(Models.ErrorResponseModel), 409)]
         [SwaggerOperation(Tags = new[] { "admin-financialcodes" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult AddFinancialCode(FinancialCodeTypes type, [FromBody] FinancialCodeModel codeModel)
         {
             try
@@ -110,6 +114,7 @@ public IActionResult AddFinancialCode(FinancialCodeTypes type, [FromBody] Financ
         [ProducesResponseType(typeof(Models.ErrorResponseModel), 400)]
         [ProducesResponseType(typeof(Models.ErrorResponseModel), 409)]
         [SwaggerOperation(Tags = new[] { "admin-financialcodes" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult UpdateFinancialCode(FinancialCodeTypes type, long codeId, [FromBody] FinancialCodeModel codeModel)
         {
             try

source/backend/api/Areas/Admin/Controllers/RoleController.cs

@@ -4,6 +4,7 @@
 using Pims.Api.Models.Base;
 using Pims.Api.Models.Concepts.Role;
 using Pims.Api.Policies;
+using Pims.Core.Json;
 using Pims.Dal.Repositories;
 using Pims.Dal.Security;
 using Swashbuckle.AspNetCore.Annotations;
@@ -55,6 +56,7 @@ public RoleController(IRoleRepository roleRepository, IMapper mapper)
         [ProducesResponseType(typeof(PageModel<RoleModel>), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-role" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetRoles(int page = 1, int quantity = 10, string name = null)
         {
             if (page < 1)
@@ -87,6 +89,7 @@ public IActionResult GetRoles(int page = 1, int quantity = 10, string name = nul
         [ProducesResponseType(typeof(RoleModel), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-role" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetRole(Guid key)
         {
             var entity = _roleRepository.GetByKey(key);

source/backend/api/Areas/Admin/Controllers/UserController.cs

@@ -5,6 +5,7 @@
 using Pims.Api.Models.Base;
 using Pims.Api.Models.Concepts.User;
 using Pims.Api.Policies;
+using Pims.Core.Json;
 using Pims.Dal.Entities;
 using Pims.Dal.Entities.Models;
 using Pims.Dal.Repositories;
@@ -54,6 +55,7 @@ public UserController(IUserRepository userRepository, IMapper mapper)
         [ProducesResponseType(typeof(PageModel<UserModel>), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-user" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetUsers()
         {
             var uri = new Uri(this.Request.GetDisplayUrl());
@@ -71,6 +73,7 @@ public IActionResult GetUsers()
         [ProducesResponseType(typeof(PageModel<UserModel>), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-user" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetUsers(UserFilter filter)
         {
             var page = _userRepository.GetAllByFilter(filter);
@@ -88,6 +91,7 @@ public IActionResult GetUsers(UserFilter filter)
         [ProducesResponseType(typeof(UserModel), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-user" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetUser(long id)
         {
             var entity = _userRepository.GetById(id);
@@ -105,6 +109,7 @@ public IActionResult GetUser(long id)
         [ProducesResponseType(typeof(UserModel), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-user" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetUser(Guid key)
         {
             var entity = _userRepository.GetByKeycloakUserId(key);
@@ -122,6 +127,7 @@ public IActionResult GetUser(Guid key)
         [ProducesResponseType(typeof(UserModel), 201)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-user" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult AddUser([FromBody] UserModel model)
         {
             var entity = _mapper.Map<PimsUser>(model);
@@ -143,7 +149,7 @@ public IActionResult AddUser([FromBody] UserModel model)
         [ProducesResponseType(typeof(UserModel), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-user" })]
-        [System.Diagnostics.CodeAnalysis.SuppressMessage("Style", "IDE0060:Remove unused parameter", Justification = "Parameter 'id' is required for route.")]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult UpdateUser(Guid key, [FromBody] UserModel model)
         {
             var entity = _mapper.Map<PimsUser>(model);
@@ -164,7 +170,7 @@ public IActionResult UpdateUser(Guid key, [FromBody] UserModel model)
         [ProducesResponseType(typeof(UserModel), 200)]
         [ProducesResponseType(typeof(Api.Models.ErrorResponseModel), 400)]
         [SwaggerOperation(Tags = new[] { "admin-user" })]
-        [System.Diagnostics.CodeAnalysis.SuppressMessage("Style", "IDE0060:Remove unused parameter", Justification = "Parameter 'key' is required for route.")]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult DeleteUser(Guid key, [FromBody] UserModel model)
         {
             var entity = _mapper.Map<PimsUser>(model);

source/backend/api/Areas/CompensationRequisition/Controllers/CompensationRequisitionController.cs

@@ -46,6 +46,7 @@ public CompensationRequisitionController(IMapper mapper, ILogger<CompensationReq
         [Produces("application/json")]
         [ProducesResponseType(typeof(CompensationRequisitionModel), 200)]
         [SwaggerOperation(Tags = new[] { "compensation-requisition" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult GetCompensationRequisitionById([FromRoute] long id)
         {
             _logger.LogInformation(
@@ -66,6 +67,7 @@ public IActionResult GetCompensationRequisitionById([FromRoute] long id)
         [Produces("application/json")]
         [ProducesResponseType(typeof(CompensationRequisitionModel), 200)]
         [SwaggerOperation(Tags = new[] { "compensation-requisition" })]
+        [TypeFilter(typeof(NullJsonResultFilter))]
         public IActionResult UpdateCompensationRequisition([FromRoute] long id, [FromBody] CompensationRequisitionModel compensationRequisition)
         {
             _logger.LogInformation(