PSP-8316 | Lease consultation updates

source/backend/api/Areas/Leases/Controllers/ConsultationController.cs

@@ -0,0 +1,199 @@
+
+using System;
+using System.Collections.Generic;
+using MapsterMapper;
+using Microsoft.AspNetCore.Authorization;
+using Microsoft.AspNetCore.Mvc;
+using Microsoft.Extensions.Logging;
+using Pims.Api.Helpers.Exceptions;
+using Pims.Api.Models.Concepts.Lease;
+using Pims.Api.Policies;
+using Pims.Api.Services;
+using Pims.Core.Extensions;
+using Pims.Core.Json;
+using Pims.Dal.Security;
+using Swashbuckle.AspNetCore.Annotations;
+
+namespace Pims.Api.Areas.Lease.Controllers
+{
+    /// <summary>
+    /// ConsultationController class, provides endpoints for interacting with lease files consultation.
+    /// </summary>
+    [Authorize]
+    [ApiController]
+    [ApiVersion("1.0")]
+    [Area("leases")]
+    [Route("v{version:apiVersion}/[area]")]
+    [Route("[area]")]
+    public class ConsultationController : ControllerBase
+    {
+        #region Variables
+        private readonly ILeaseService _leaseService;
+        private readonly IMapper _mapper;
+        private readonly ILogger _logger;
+
+        #endregion
+
+        #region Constructors
+
+        /// <summary>
+        /// Creates a new instance of a ConsultationController class, initializes it with the specified arguments.
+        /// </summary>
+        /// <param name="leaseService"></param>
+        /// <param name="mapper"></param>
+        /// <param name="logger"></param>
+        public ConsultationController(ILeaseService leaseService, IMapper mapper, ILogger<ConsultationController> logger)
+        {
+            _leaseService = leaseService;
+            _mapper = mapper;
+            _logger = logger;
+        }
+        #endregion
+
+        #region Endpoints
+
+        /// <summary>
+        /// Get the lease file consultation.
+        /// </summary>
+        /// <returns>The consultation items.</returns>
+        [HttpGet("{id:long}/consultations")]
+        [HasPermission(Permissions.LeaseView)]
+        [Produces("application/json")]
+        [ProducesResponseType(typeof(IEnumerable<ConsultationLeaseModel>), 200)]
+        [SwaggerOperation(Tags = new[] { "leasefile" })]
+        public IActionResult GetLeaseConsultations([FromRoute] long id)
+        {
+            _logger.LogInformation(
+                "Request received by Controller: {Controller}, Action: {ControllerAction}, User: {User}, DateTime: {DateTime}",
+                nameof(ConsultationController),
+                nameof(GetLeaseConsultations),
+                User.GetUsername(),
+                DateTime.Now);
+
+
+            var consultation = _leaseService.GetConsultations(id);
+            return new JsonResult(_mapper.Map<IEnumerable<ConsultationLeaseModel>>(consultation));
+        }
+
+        /// <summary>
+        /// Create the lease file consultation to the lease file.
+        /// </summary>
+        /// <returns>The consultation items.</returns>
+        [HttpPost("{leaseId:long}/consultations")]
+        [HasPermission(Permissions.LeaseEdit)]
+        [Produces("application/json")]
+        [ProducesResponseType(typeof(ConsultationLeaseModel), 201)]
+        [TypeFilter(typeof(NullJsonResultFilter))]
+        [SwaggerOperation(Tags = new[] { "leasefile" })]
+        public IActionResult AddLeaseConsultation([FromRoute] long leaseId, [FromBody] ConsultationLeaseModel consultation)
+        {
+            _logger.LogInformation(
+                "Request received by Controller: {Controller}, Action: {ControllerAction}, User: {User}, DateTime: {DateTime}",
+                nameof(ConsultationController),
+                nameof(AddLeaseConsultation),
+                User.GetUsername(),
+                DateTime.Now);
+
+            if (leaseId != consultation.LeaseId)
+            {
+                throw new BadRequestException("Invalid LeaseId.");
+            }
+
+            var newConsultation = _leaseService.AddConsultation(_mapper.Map<Dal.Entities.PimsLeaseConsultation>(consultation));
+
+            return new JsonResult(_mapper.Map<ConsultationLeaseModel>(newConsultation));
+        }
+
+        /// <summary>
+        /// Get the lease file consultation by Id.
+        /// </summary>
+        /// <returns>The consultation items.</returns>
+        [HttpGet("{leaseId:long}/consultations/{consultationId:long}")]
+        [HasPermission(Permissions.LeaseView)]
+        [Produces("application/json")]
+        [ProducesResponseType(typeof(ConsultationLeaseModel), 200)]
+        [SwaggerOperation(Tags = new[] { "leasefile" })]
+        public IActionResult GetLeaseConsultationById([FromRoute] long leaseId, [FromRoute] long consultationId)
+        {
+            _logger.LogInformation(
+                "Request received by Controller: {Controller}, Action: {ControllerAction}, User: {User}, DateTime: {DateTime}",
+                nameof(ConsultationController),
+                nameof(GetLeaseConsultationById),
+                User.GetUsername(),
+                DateTime.Now);
+
+            var consultation = _leaseService.GetConsultationById(consultationId);
+
+            if (consultation.LeaseConsultationId != leaseId)
+            {
+                throw new BadRequestException("Invalid lease id for the given consultation.");
+            }
+
+            return new JsonResult(_mapper.Map<ConsultationLeaseModel>(consultation));
+        }
+
+        /// <summary>
+        /// Update the lease file consultation by Id.
+        /// </summary>
+        /// <returns>The consultation item updated.</returns>
+        [HttpPut("{id:long}/consultations/{consultationId:long}")]
+        [HasPermission(Permissions.LeaseEdit)]
+        [Produces("application/json")]
+        [ProducesResponseType(typeof(ConsultationLeaseModel), 200)]
+        [TypeFilter(typeof(NullJsonResultFilter))]
+        [SwaggerOperation(Tags = new[] { "leasefile" })]
+        public IActionResult UpdateLeaseConsultation([FromRoute] long id, [FromRoute] long consultationId, [FromBody] ConsultationLeaseModel consultation)
+        {
+            _logger.LogInformation(
+                "Request received by Controller: {Controller}, Action: {ControllerAction}, User: {User}, DateTime: {DateTime}",
+                nameof(ConsultationController),
+                nameof(UpdateLeaseConsultation),
+                User.GetUsername(),
+                DateTime.Now);
+
+            if (id != consultation.LeaseId)
+            {
+                throw new BadRequestException("Invalid LeaseId.");
+            }
+            if (consultationId != consultation.Id)
+            {
+                throw new BadRequestException("Invalid consultationId.");
+            }
+
+            var updatedConsultation = _leaseService.UpdateConsultation(_mapper.Map<Dal.Entities.PimsLeaseConsultation>(consultation));
+
+            return new JsonResult(_mapper.Map<ConsultationLeaseModel>(updatedConsultation));
+        }
+
+        /// <summary>
+        /// Delete the lease file consultation by Id.
+        /// </summary>
+        /// <returns>The consultation item updated.</returns>
+        [HttpDelete("{leaseId:long}/consultations/{consultationId:long}")]
+        [HasPermission(Permissions.LeaseEdit)]
+        [Produces("application/json")]
+        [ProducesResponseType(typeof(bool), 200)]
+        [SwaggerOperation(Tags = new[] { "leasefile" })]
+        public IActionResult DeleteLeaseConsultation([FromRoute] long leaseId, [FromRoute] long consultationId)
+        {
+            _logger.LogInformation(
+                "Request received by Controller: {Controller}, Action: {ControllerAction}, User: {User}, DateTime: {DateTime}",
+                nameof(ConsultationController),
+                nameof(DeleteLeaseConsultation),
+                User.GetUsername(),
+                DateTime.Now);
+
+            var existingConsultation = _leaseService.GetConsultationById(consultationId);
+            if (existingConsultation.LeaseConsultationId != leaseId)
+            {
+                throw new BadRequestException("Invalid lease id for the given consultation.");
+            }
+
+            var result = _leaseService.DeleteConsultation(consultationId);
+
+            return new JsonResult(result);
+        }
+
+        #endregion
+    }
+}

source/backend/api/Services/ILeaseService.cs

@@ -38,5 +38,15 @@ public interface ILeaseService
         PimsLease UpdateChecklistItems(long leaseId, IList<PimsLeaseChecklistItem> checklistItems);
 
         IEnumerable<PimsLeaseStakeholderType> GetAllStakeholderTypes();
+
+        IEnumerable<PimsLeaseConsultation> GetConsultations(long leaseId);
+
+        PimsLeaseConsultation GetConsultationById(long consultationId);
+
+        PimsLeaseConsultation AddConsultation(PimsLeaseConsultation consultation);
+
+        PimsLeaseConsultation UpdateConsultation(PimsLeaseConsultation consultation);
+
+        bool DeleteConsultation(long consultationId);
     }
 }

source/backend/api/Services/LeaseService.cs

@@ -32,6 +32,7 @@ public class LeaseService : BaseService, ILeaseService
         private readonly IInsuranceRepository _insuranceRepository;
         private readonly ILeaseStakeholderRepository _stakeholderRepository;
         private readonly ILeaseRenewalRepository _renewalRepository;
+        private readonly IConsultationRepository _consultationRepository;
         private readonly IUserRepository _userRepository;
         private readonly IPropertyService _propertyService;
         private readonly ILookupRepository _lookupRepository;
@@ -48,6 +49,7 @@ public LeaseService(
             IInsuranceRepository insuranceRepository,
             ILeaseStakeholderRepository stakeholderRepository,
             ILeaseRenewalRepository renewalRepository,
+            IConsultationRepository consultationRepository,
             IUserRepository userRepository,
             IPropertyService propertyService,
             ILookupRepository lookupRepository)
@@ -63,6 +65,7 @@ public LeaseService(
             _insuranceRepository = insuranceRepository;
             _stakeholderRepository = stakeholderRepository;
             _renewalRepository = renewalRepository;
+            _consultationRepository = consultationRepository;
             _userRepository = userRepository;
             _propertyService = propertyService;
             _lookupRepository = lookupRepository;
@@ -254,8 +257,6 @@ public PimsLease Update(PimsLease lease, IEnumerable<UserOverrideCode> userOverr
 
             _propertyLeaseRepository.UpdatePropertyLeases(lease.Internal_Id, leaseWithProperties.PimsPropertyLeases);
 
-            _leaseRepository.UpdateLeaseConsultations(lease.Internal_Id, lease.ConcurrencyControlNumber, lease.PimsLeaseConsultations);
-
             _leaseRepository.UpdateLeaseRenewals(lease.Internal_Id, lease.ConcurrencyControlNumber, lease.PimsLeaseRenewals);
 
             List<PimsPropertyLease> differenceSet = currentFileProperties.Where(x => !lease.PimsPropertyLeases.Any(y => y.Internal_Id == x.Internal_Id)).ToList();
@@ -340,6 +341,52 @@ public IEnumerable<PimsLeaseStakeholderType> GetAllStakeholderTypes()
             return _leaseRepository.GetAllLeaseStakeholderTypes();
         }
 
+        public IEnumerable<PimsLeaseConsultation> GetConsultations(long leaseId)
+        {
+            _logger.LogInformation("Getting lease consultations with Lease id: {leaseId}", leaseId);
+            _user.ThrowIfNotAuthorized(Permissions.LeaseView);
+
+            return _consultationRepository.GetConsultationsByLease(leaseId);
+        }
+
+        public PimsLeaseConsultation GetConsultationById(long consultationId)
+        {
+            _logger.LogInformation("Getting consultation with id: {consultationId}", consultationId);
+            _user.ThrowIfNotAuthorized(Permissions.LeaseEdit);
+
+            return _consultationRepository.GetConsultationById(consultationId);
+        }
+
+        public PimsLeaseConsultation AddConsultation(PimsLeaseConsultation consultation)
+        {
+            _user.ThrowIfNotAuthorized(Permissions.LeaseEdit);
+
+            var newConsultation = _consultationRepository.AddConsultation(consultation);
+            _consultationRepository.CommitTransaction();
+
+            return newConsultation;
+        }
+
+        public PimsLeaseConsultation UpdateConsultation(PimsLeaseConsultation consultation)
+        {
+            _user.ThrowIfNotAuthorized(Permissions.LeaseEdit);
+
+            var updatedConsultation = _consultationRepository.UpdateConsultation(consultation);
+            _consultationRepository.CommitTransaction();
+
+            return updatedConsultation;
+        }
+
+        public bool DeleteConsultation(long consultationId)
+        {
+            _user.ThrowIfNotAuthorized(Permissions.LeaseEdit);
+
+            bool deleteResult = _consultationRepository.TryDeleteConsultation(consultationId);
+            _consultationRepository.CommitTransaction();
+
+            return deleteResult;
+        }
+
         private static void ValidateRenewalDates(PimsLease lease, ICollection<PimsLeaseRenewal> renewals)
         {
             if (lease.LeaseStatusTypeCode != PimsLeaseStatusTypes.ACTIVE)

source/backend/apimodels/Models/Concepts/Deposit/SecurityDepositMap.cs

@@ -1,7 +1,5 @@
-using System;
 using Mapster;
 using Pims.Api.Models.Base;
-using Pims.Core.Extensions;
 using Entity = Pims.Dal.Entities;
 
 namespace Pims.Api.Models.Concepts.Deposit

source/backend/apimodels/Models/Concepts/Lease/ConsultationLeaseMap.cs

@@ -1,5 +1,6 @@
 using Mapster;
 using Pims.Api.Models.Base;
+using Pims.Core.Extensions;
 using Pims.Dal.Entities;
 
 namespace Pims.Api.Models.Concepts.Lease
@@ -10,18 +11,38 @@ public void Register(TypeAdapterConfig config)
         {
             config.NewConfig<PimsLeaseConsultation, ConsultationLeaseModel>()
                 .Map(dest => dest.Id, src => src.LeaseConsultationId)
-                .Map(dest => dest.ConsultationType, src => src.ConsultationTypeCodeNavigation)
-                .Map(dest => dest.ConsultationStatusType, src => src.ConsultationStatusTypeCodeNavigation)
-                .Map(dest => dest.ParentLeaseId, src => src.LeaseId)
+                .Map(dest => dest.LeaseId, src => src.LeaseId)
+                .Map(dest => dest.Lease, src => src.Lease)
+                .Map(dest => dest.PersonId, src => src.PersonId)
+                .Map(dest => dest.Person, src => src.Person)
+                .Map(dest => dest.OrganizationId, src => src.OrganizationId)
+                .Map(dest => dest.Organization, src => src.Organization)
+                .Map(dest => dest.PrimaryContactId, src => src.PrimaryContactId)
+                .Map(dest => dest.PrimaryContact, src => src.PrimaryContact)
+                .Map(dest => dest.ConsultationTypeCode, src => src.ConsultationTypeCodeNavigation)
+                .Map(dest => dest.ConsultationStatusTypeCode, src => src.ConsultationStatusTypeCodeNavigation)
                 .Map(dest => dest.OtherDescription, src => src.OtherDescription)
+                .Map(dest => dest.RequestedOn, src => src.RequestedOn.ToNullableDateOnly())
+                .Map(dest => dest.IsResponseReceived, src => src.IsResponseReceived)
+                .Map(dest => dest.ResponseReceivedDate, src => src.ResponseReceivedDate.ToNullableDateOnly())
+                .Map(dest => dest.Comment, src => src.Comment)
                 .Inherits<IBaseAppEntity, BaseAuditModel>();
 
             config.NewConfig<ConsultationLeaseModel, PimsLeaseConsultation>()
                 .Map(dest => dest.LeaseConsultationId, src => src.Id)
-                .Map(dest => dest.LeaseId, src => src.ParentLeaseId)
-                .Map(dest => dest.ConsultationTypeCode, src => src.ConsultationType.Id)
-                .Map(dest => dest.ConsultationStatusTypeCode, src => src.ConsultationStatusType.Id)
+                .Map(dest => dest.LeaseId, src => src.LeaseId)
+                .Map(dest => dest.PersonId, src => src.PersonId)
+                .Map(dest => dest.OrganizationId, src => src.OrganizationId)
+                .Map(dest => dest.Organization, src => src.Organization)
+                .Map(dest => dest.PrimaryContactId, src => src.PrimaryContactId)
+                .Map(dest => dest.PrimaryContact, src => src.PrimaryContact)
+                .Map(dest => dest.ConsultationTypeCode, src => src.ConsultationTypeCode.Id)
+                .Map(dest => dest.ConsultationStatusTypeCode, src => src.ConsultationStatusTypeCode.Id)
                 .Map(dest => dest.OtherDescription, src => src.OtherDescription)
+                .Map(dest => dest.RequestedOn, src => src.RequestedOn.ToNullableDateTime())
+                .Map(dest => dest.IsResponseReceived, src => src.IsResponseReceived)
+                .Map(dest => dest.ResponseReceivedDate, src => src.ResponseReceivedDate.ToNullableDateTime())
+                .Map(dest => dest.Comment, src => src.Comment)
                 .Inherits<BaseAuditModel, IBaseAppEntity>();
         }
     }