diff --git a/src/main/java/io/gmss/fiscad/implementations/infocad/metier/EnqueteServiceImpl.java b/src/main/java/io/gmss/fiscad/implementations/infocad/metier/EnqueteServiceImpl.java index 6ad215e..531ebd8 100644 --- a/src/main/java/io/gmss/fiscad/implementations/infocad/metier/EnqueteServiceImpl.java +++ b/src/main/java/io/gmss/fiscad/implementations/infocad/metier/EnqueteServiceImpl.java @@ -102,7 +102,7 @@ public class EnqueteServiceImpl implements EnqueteService { @Transactional public EnquetePayLoadWeb createEnquete(EnquetePayLoadWeb enquetePayLoadWeb) throws BadRequestException { - if((enquetePayLoadWeb.getParcelleQ()==null || enquetePayLoadWeb.getParcelleI()==null || enquetePayLoadWeb.getParcelleP()!=null) && enquetePayLoadWeb.getParcelleNup()==null){ + if((enquetePayLoadWeb.getParcelleQ()==null || enquetePayLoadWeb.getParcelleI()==null || enquetePayLoadWeb.getParcelleP()==null) && enquetePayLoadWeb.getParcelleNup()==null){ throw new BadRequestException("Echec de l'enregistrement : Vous devez préciser le QIP ou le NUP"); } @@ -116,9 +116,11 @@ public class EnqueteServiceImpl implements EnqueteService { throw new BadRequestException("Echec de l'enregistrement : Quartier inexistant"); } - Optional optionalUser = userRepository.findById(enquetePayLoadWeb.getEnqueteurId()); - if (optionalUser.isEmpty()) { - throw new BadRequestException("Echec de l'enregistrement : Enquêteur inexistant"); + if (enquetePayLoadWeb.getEnqueteurId()!=null) { + Optional optionalUser = userRepository.findById(enquetePayLoadWeb.getEnqueteurId()); + if (optionalUser.isEmpty()) { + throw new BadRequestException("Echec de l'enregistrement : Enquêteur inexistant"); + } } if(enquetePayLoadWeb.getPersonneId()==null) { @@ -189,9 +191,6 @@ public class EnqueteServiceImpl implements EnqueteService { parcellePayLoadWeb.setTypeDomaineId(enquetePayLoadWeb.getTypeDomaineId()); parcellePayLoadWeb.setRueId(enquetePayLoadWeb.getRueId()); parcellePayLoadWeb.setProprietaireId(enquetePayLoadWeb.getPersonneId()); - parcellePayLoadWeb.setNatureDomaineId(enquetePayLoadWeb.getNatureDomaineId()); - parcellePayLoadWeb.setNatureDomaineId(enquetePayLoadWeb.getNatureDomaineId()); - return parcellePayLoadWeb; } diff --git a/src/main/java/io/gmss/fiscad/implementations/infocad/metier/ParcelleServiceImpl.java b/src/main/java/io/gmss/fiscad/implementations/infocad/metier/ParcelleServiceImpl.java index 9051598..db03696 100644 --- a/src/main/java/io/gmss/fiscad/implementations/infocad/metier/ParcelleServiceImpl.java +++ b/src/main/java/io/gmss/fiscad/implementations/infocad/metier/ParcelleServiceImpl.java @@ -56,22 +56,23 @@ public class ParcelleServiceImpl implements ParcelleService { @Override public ParcellePayLoadWeb createParcelle(ParcellePayLoadWeb parcellePayLoadWeb) throws BadRequestException { + Optional optionalNatureDomaine = natureDomaineRepository.findById(parcellePayLoadWeb.getNatureDomaineId()); - if (!optionalNatureDomaine.isPresent()) { + if (optionalNatureDomaine.isEmpty()) { throw new BadRequestException("Impossible d'enregistrer une parcelle avec une nature de domaine inexistant"); } + Optional optionalQuartier = quartierRepository.findById(parcellePayLoadWeb.getQuartierId()); - if (!optionalQuartier.isPresent()) { + if (optionalQuartier.isEmpty()) { throw new BadRequestException("Impossible d'enregistrer une parcelle avec un quartier/village inexistant"); } Optional optionalTypeDomaine = typeDomaineRepository.findById(parcellePayLoadWeb.getTypeDomaineId()); - if (!optionalTypeDomaine.isPresent()) { + if (optionalTypeDomaine.isEmpty()) { throw new BadRequestException("Veuillez préciser le type de domaine"); } - Parcelle parcelle = new Parcelle(); - parcelle = entityFromPayLoadService.getParcelleFromPayload(parcellePayLoadWeb); + Parcelle parcelle = entityFromPayLoadService.getParcelleFromPayload(parcellePayLoadWeb); parcelle= parcelleRepository.save(parcelle); return parcelleRepository.findParcelleToDtoById(parcelle.getId()).orElse(null); }