From fb20422e08747375465d53fea2f988f21da79ac6 Mon Sep 17 00:00:00 2001 From: Hugo Bole Date: Mon, 8 Jan 2024 10:21:25 +0100 Subject: [PATCH] =?UTF-8?q?Modifie=20les=20appels=20en=20dur=20aux=20types?= =?UTF-8?q?=20de=20visites=20sous=20forme=20de=20cha=C3=AEne...?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ... pour les remplacer par des appels au type énuméré Change-Id: I0da41c20dd7fd6031a9f4af5a6e7d1c096a45b42 --- .../remocra/repository/HydrantRepository.java | 2 +- .../repository/HydrantVisiteRepository.java | 41 +++++++++++-------- .../TypeHydrantAnomalieRepository.java | 4 +- .../fr/sdis83/remocra/service/XmlService.java | 2 +- 4 files changed, 28 insertions(+), 21 deletions(-) diff --git a/remocra/src/main/java/fr/sdis83/remocra/repository/HydrantRepository.java b/remocra/src/main/java/fr/sdis83/remocra/repository/HydrantRepository.java index 435476576..3d07679d8 100755 --- a/remocra/src/main/java/fr/sdis83/remocra/repository/HydrantRepository.java +++ b/remocra/src/main/java/fr/sdis83/remocra/repository/HydrantRepository.java @@ -704,7 +704,7 @@ private int getNbVisitesCtrlWithCondition(long idHydrant, Condition predicatInst HYDRANT_VISITE .HYDRANT .eq(idHydrant) - .and(TYPE_HYDRANT_SAISIE.CODE.eq("CTRL")) + .and(TYPE_HYDRANT_SAISIE.CODE.eq(GlobalConstants.TypeVisite.CONTROLE.getCode())) .and(HYDRANT_VISITE.CTRL_DEBIT_PRESSION) .and(predicatInstant)) .fetchOneInto(int.class); diff --git a/remocra/src/main/java/fr/sdis83/remocra/repository/HydrantVisiteRepository.java b/remocra/src/main/java/fr/sdis83/remocra/repository/HydrantVisiteRepository.java index 7f5332154..253278760 100644 --- a/remocra/src/main/java/fr/sdis83/remocra/repository/HydrantVisiteRepository.java +++ b/remocra/src/main/java/fr/sdis83/remocra/repository/HydrantVisiteRepository.java @@ -1,5 +1,6 @@ package fr.sdis83.remocra.repository; +import static fr.sdis83.remocra.GlobalConstants.TypeVisite; import static fr.sdis83.remocra.db.model.remocra.Tables.HYDRANT; import static fr.sdis83.remocra.db.model.remocra.Tables.HYDRANT_ANOMALIES; import static fr.sdis83.remocra.db.model.remocra.Tables.HYDRANT_PIBI; @@ -86,26 +87,28 @@ public HydrantVisite addVisiteFromFiche(Long id, String visiteData) throws Excep .where(HYDRANT_VISITE.HYDRANT.eq(id)) .fetchOneInto(Integer.class); - String typeVisite = + String strTypeVisite = context .select(TYPE_HYDRANT_SAISIE.CODE) .from(TYPE_HYDRANT_SAISIE) .where(TYPE_HYDRANT_SAISIE.ID.eq(JSONUtil.getLong(data, "type"))) .fetchOneInto(String.class); - if (nbVisites == 0 && !typeVisite.toUpperCase().equals("CREA")) { + TypeVisite typeVisite = TypeVisite.fromCode(strTypeVisite.toUpperCase()); + + if (nbVisites == 0 && !typeVisite.equals(TypeVisite.CREATION)) { throw new Exception( "Le contexte de visite doit être de type CREA (première visite du PEI)"); - } else if (nbVisites == 1 && !typeVisite.toUpperCase().equals("RECEP")) { + } else if (nbVisites == 1 && !typeVisite.equals(TypeVisite.RECEPTION)) { throw new Exception( "Le contexte de visite doit être de type RECEP (deuxième visite du PEI)"); } else if (nbVisites > 1 - && (!typeVisite.toUpperCase().equals("NP") - && !typeVisite.toUpperCase().equals("RECO") - && !typeVisite.equals("CTRL"))) { + && (!typeVisite.equals(TypeVisite.NON_PROGRAMMEE) + && !typeVisite.equals(TypeVisite.RECONNAISSANCE) + && !typeVisite.equals(TypeVisite.CONTROLE))) { throw new Exception( "Une visite de type " - + typeVisite.toUpperCase() + + typeVisite.getCode() + " existe déjà. Veuillez utiliser une visite de type NP, RECO ou CTRL"); } @@ -268,17 +271,18 @@ public HydrantVisite addVisite(HydrantVisite visite) { .fetchOneInto(String.class); TableField field = null; - switch (codeTypeVisite) { - case "CREA": + TypeVisite typeVisite = TypeVisite.fromCode(codeTypeVisite); + switch (typeVisite) { + case CREATION: field = HYDRANT.DATE_CREA; break; - case "RECEP": + case RECEPTION: field = HYDRANT.DATE_RECEP; break; - case "RECO": + case RECONNAISSANCE: field = HYDRANT.DATE_RECO; break; - case "CTRL": + case CONTROLE: field = HYDRANT.DATE_CONTR; break; } @@ -375,17 +379,18 @@ private void deleteVisite(HydrantVisite visite) { } // On met à jour la date dans la table hydrant TableField field = null; - switch (codeVisite) { - case "CREA": + TypeVisite typeVisite = TypeVisite.fromCode(codeVisite); + switch (typeVisite) { + case CREATION: field = HYDRANT.DATE_CREA; break; - case "RECEP": + case RECEPTION: field = HYDRANT.DATE_RECEP; break; - case "RECO": + case RECONNAISSANCE: field = HYDRANT.DATE_RECO; break; - case "CTRL": + case CONTROLE: field = HYDRANT.DATE_CONTR; break; } @@ -451,7 +456,7 @@ public Long getIdControle() { return context .select(TYPE_HYDRANT_SAISIE.ID) .from(TYPE_HYDRANT_SAISIE) - .where(TYPE_HYDRANT_SAISIE.CODE.equalIgnoreCase("CTRL")) + .where(TYPE_HYDRANT_SAISIE.CODE.equalIgnoreCase(TypeVisite.CONTROLE.getCode())) .fetchOneInto(Long.class); } } diff --git a/remocra/src/main/java/fr/sdis83/remocra/repository/TypeHydrantAnomalieRepository.java b/remocra/src/main/java/fr/sdis83/remocra/repository/TypeHydrantAnomalieRepository.java index fa17829c8..1bcbd1959 100644 --- a/remocra/src/main/java/fr/sdis83/remocra/repository/TypeHydrantAnomalieRepository.java +++ b/remocra/src/main/java/fr/sdis83/remocra/repository/TypeHydrantAnomalieRepository.java @@ -5,6 +5,7 @@ import static fr.sdis83.remocra.db.model.remocra.Tables.TYPE_HYDRANT_ANOMALIE_NATURE_SAISIES; import static fr.sdis83.remocra.db.model.remocra.Tables.TYPE_HYDRANT_SAISIE; +import fr.sdis83.remocra.GlobalConstants; import fr.sdis83.remocra.db.model.remocra.Tables; import fr.sdis83.remocra.db.model.remocra.tables.pojos.TypeHydrantAnomalie; import java.util.List; @@ -39,7 +40,8 @@ public List getIdsAnomaliesCTP() { TYPE_HYDRANT_ANOMALIE_NATURE.ID)) .join(TYPE_HYDRANT_SAISIE) .on(TYPE_HYDRANT_SAISIE.ID.eq(TYPE_HYDRANT_ANOMALIE_NATURE_SAISIES.SAISIES)) - .where(TYPE_HYDRANT_SAISIE.CODE.equalIgnoreCase("CTRL")) + .where( + TYPE_HYDRANT_SAISIE.CODE.equalIgnoreCase(GlobalConstants.TypeVisite.CONTROLE.getCode())) .fetchInto(Long.class); } } diff --git a/remocra/src/main/java/fr/sdis83/remocra/service/XmlService.java b/remocra/src/main/java/fr/sdis83/remocra/service/XmlService.java index c97ef845b..d8dcd9b02 100644 --- a/remocra/src/main/java/fr/sdis83/remocra/service/XmlService.java +++ b/remocra/src/main/java/fr/sdis83/remocra/service/XmlService.java @@ -1404,7 +1404,7 @@ private Hydrant.TYPE_SAISIE getTypeSaisie(Hydrant hydrant, String typeSaisie) return Hydrant.TYPE_SAISIE.RECEP; } else if (authUtils.hasRight(TypeDroitEnum.HYDRANTS_CONTROLE_C) && authUtils.hasRight(TypeDroitEnum.HYDRANTS_RECONNAISSANCE_C)) { - return (typeSaisie != null && "CTRL".equals(typeSaisie)) + return (GlobalConstants.TypeVisite.CONTROLE.getCode().equals(typeSaisie)) ? Hydrant.TYPE_SAISIE.CTRL : Hydrant.TYPE_SAISIE.RECO; } else if (authUtils.hasRight(TypeDroitEnum.HYDRANTS_CONTROLE_C)) {