From 4394e4248455637ab7836756058ac872fdf4af10 Mon Sep 17 00:00:00 2001 From: wayne <wayne8692wayne8692@gmail.com> Date: 星期四, 17 二月 2022 11:39:43 +0800 Subject: [PATCH] Merge branch 'pollex-dev' into sit --- pamapi/src/main/java/com/pollex/pam/service/SatisfactionService.java | 38 +++++++++++++++++++++++++------------- 1 files changed, 25 insertions(+), 13 deletions(-) diff --git a/pamapi/src/main/java/com/pollex/pam/service/SatisfactionService.java b/pamapi/src/main/java/com/pollex/pam/service/SatisfactionService.java index 0d8a786..1bf3c20 100644 --- a/pamapi/src/main/java/com/pollex/pam/service/SatisfactionService.java +++ b/pamapi/src/main/java/com/pollex/pam/service/SatisfactionService.java @@ -6,7 +6,8 @@ import com.pollex.pam.enums.SatisfactionTypeEnum; import com.pollex.pam.security.SecurityUtils; -import com.pollex.pam.service.dto.SatisfactionSystemScoreDTO; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.stereotype.Service; import org.springframework.transaction.annotation.Transactional; @@ -24,10 +25,13 @@ import com.pollex.pam.service.mapper.SatisfactionMapper; import com.pollex.pam.web.rest.errors.SatisfactionAlreadyExistException; import com.pollex.pam.web.rest.errors.SatisfactionNotFoundException; +import org.springframework.util.Assert; @Service @Transactional public class SatisfactionService { + + private static final Logger log = LoggerFactory.getLogger(SatisfactionService.class); @Autowired SatisfactionRepository satisfactionRepository; @@ -55,19 +59,27 @@ public Satisfaction save(Satisfaction satisfaction) { satisfaction = satisfactionRepository.save(satisfaction); - consultantService.setConsultantAvgScore(satisfaction); + if(satisfaction.getType() == SatisfactionTypeEnum.APPOINTMENT) { + consultantService.setConsultantAvgScore(satisfaction); + } return satisfaction; } public Satisfaction scorefaction(SatisfactionCustomerScoreDTO scoreDTO) { - Optional<Satisfaction> satisfactionOP = getByAppointmentIdAndType(scoreDTO.getAppointmentId(), SatisfactionTypeEnum.APPOINTMENT); + Optional<Satisfaction> satisfactionOP = satisfactionRepository.findOneByAppointmentIdAndType(scoreDTO.getAppointmentId(), scoreDTO.getType()); Satisfaction satisfaction = satisfactionOP.orElseThrow(SatisfactionNotFoundException::new); + + boolean isSameCustomer = satisfaction.getCustomerId().equals(SecurityUtils.getCustomerDBId()); + Assert.isTrue(isSameCustomer, "The currently logged in customer has a different ID than the customer on Satisfaction"); + satisfaction.setScore(scoreDTO.getScore()); satisfaction.setStatus(SatisfactionStatusEnum.FILLED); - satisfaction.setType(SatisfactionTypeEnum.APPOINTMENT); save(satisfaction); - personalNotificationService.createScorefactionToConsultant(satisfaction); - return satisfaction; + + if(satisfaction.getType() == SatisfactionTypeEnum.APPOINTMENT) { + personalNotificationService.createScorefactionToConsultant(satisfaction); + } + return satisfaction; } public Satisfaction createAppointmentSatisfaction(Appointment appointment) { @@ -89,8 +101,8 @@ return satisfactionMapper.toDTO(satisfactionList); } - public List<SatisfactionDTO> getByCustomerIdAndType(Long customerDBId, SatisfactionTypeEnum type) { - List<Satisfaction> satisfactionList = satisfactionRepository.findByCustomerIdAndType(customerDBId, type); + public List<SatisfactionDTO> getByCustomerId(Long customerDBId) { + List<Satisfaction> satisfactionList = satisfactionRepository.findByCustomerId(customerDBId); return satisfactionMapper.toDTO(satisfactionList); } @@ -110,13 +122,13 @@ return satisfactionList; } - public Satisfaction createSystemSatisfaction(SatisfactionSystemScoreDTO scoreDTO) { + public void createUnfilledSystemSatisfaction(Appointment appointment) { Satisfaction satisfaction = new Satisfaction(); + satisfaction.setAppointmentId(appointment.getId()); satisfaction.setCustomerId(SecurityUtils.getCustomerDBId()); - satisfaction.setAppointmentId(scoreDTO.getAppointmentId()); - satisfaction.setStatus(SatisfactionStatusEnum.FILLED); - satisfaction.setScore(scoreDTO.getScore()); + satisfaction.setStatus(SatisfactionStatusEnum.UNFILLED); satisfaction.setType(SatisfactionTypeEnum.SYSTEM); - return satisfactionRepository.save(satisfaction); + + satisfactionRepository.save(satisfaction); } } -- Gitblit v1.8.0