Merge pull request #65 from abes-esr/ITEM-304-back-corriger-les-ligne…

…s-vides-en-fin-de-fichier ITEM-72 Message erreur fichier de traitement non conforme
abes-esr · Oct 21, 2024 · 2f8c24d · 2f8c24d
2 parents e4df772 + f925664
commit 2f8c24d
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 13 deletions.
diff --git a/core/src/main/java/fr/abes/item/core/components/AbstractFichier.java b/core/src/main/java/fr/abes/item/core/components/AbstractFichier.java
@@ -99,7 +99,7 @@ protected void checkRcr(String rcrFichier, String rcr, int ligneCourante) throws
      * @throws FileCheckingException : erreur de format de fichier
      */
     protected void checkPpn(String ppn, int ligneCourante) throws FileCheckingException {
-        if (!ppn.matches("^\\d{8}[0-9X]$")){
+        if (!ppn.matches("^(\\d{8}[0-9X])?$")){
             throw new FileCheckingException(Constant.ERR_FILE_ERRLINE + ligneCourante + Constant.ERR_FILE_WRONGPPN);
         }
     }
@@ -110,7 +110,7 @@ protected void checkPpn(String ppn, int ligneCourante) throws FileCheckingExcept
      * @throws FileCheckingException: erreur de format de l'epn
      */
     protected void checkEpn(String epn, int ligneCourante) throws FileCheckingException {
-        if (!epn.matches("^\\d{8}[0-9X]$")) {
+        if (!epn.matches("^(\\d{8}[0-9X])?$")) {
             throw new FileCheckingException(Constant.ERR_FILE_ERRLINE + ligneCourante + Constant.ERR_FILE_WRONGEPN);
         }
     }

diff --git a/core/src/main/java/fr/abes/item/core/components/FichierPrepare.java b/core/src/main/java/fr/abes/item/core/components/FichierPrepare.java
@@ -13,7 +13,6 @@
 
 import java.io.*;
 import java.util.ArrayList;
-import java.util.Collections;
 import java.util.List;
 
 @Slf4j
@@ -40,13 +39,14 @@ public int getType() {
 		return Constant.ETATDEM_PREPAREE;
 	}
 
-
 	@Override
-	public TYPE_DEMANDE getDemandeType() {return TYPE_DEMANDE.MODIF; }
+	public TYPE_DEMANDE getDemandeType() {
+		return TYPE_DEMANDE.MODIF;
+	}
 
 	@Override
 	public void checkFileContent(Demande demandeModif) {
-	//nothing to do
+		//nothing to do
 	}
 
 	@Override
@@ -98,8 +98,8 @@ public void alimenterEpn(String input, String listeppn, String rcr) {
 	 */
 	public void alimenterPpn(String input, String listeEpn, String rcr) {
 		try (FileWriter fw = new FileWriter(path.resolve(filename).toString(), true);
-			 BufferedWriter bw = new BufferedWriter(fw);
-			 PrintWriter out = new PrintWriter(bw)) {
+			BufferedWriter bw = new BufferedWriter(fw);
+			PrintWriter out = new PrintWriter(bw)) {
 			String[] tabEpn = listeEpn.split(",");
 			Multimap<String, String> resJson = Utilitaires.parseJson(input, true);
 			for (String epn : tabEpn) {
@@ -125,9 +125,10 @@ public void trierLignesDeCorrespondances() throws IOException {
 
 		List<String> correspondanceSortList = new ArrayList<>();
 		String header = reader.readLine();//cette ligne enleve le header et le stock
-		correspondanceSortList.add(header + "\n");
+		correspondanceSortList.add(header);
 		reader.lines().sorted().forEach(line -> {
-			correspondanceSortList.add(line+"\n");
+			correspondanceSortList.add("\n");
+			correspondanceSortList.add(line);
 		});
 		reader.close();
 		fileReader.close();

diff --git a/core/src/main/java/fr/abes/item/core/service/impl/DemandeSuppService.java b/core/src/main/java/fr/abes/item/core/service/impl/DemandeSuppService.java
@@ -160,7 +160,7 @@ private void stockerFichierOnDisk(MultipartFile file, Fichier fichier, DemandeSu
         }
     }
 
-    private void checkEtatDemande(DemandeSupp demande) throws DemandeCheckingException, IOException, FileTypeException {
+    private void checkEtatDemande(DemandeSupp demande) throws DemandeCheckingException, IOException, FileTypeException, FileCheckingException {
         int etat = demande.getEtatDemande().getNumEtat();
         switch (etat) {
             case Constant.ETATDEM_PREPARATION -> preparerFichierEnPrep(demande);
@@ -177,16 +177,16 @@ private void checkEtatDemande(DemandeSupp demande) throws DemandeCheckingExcepti
         }
     }
 
-    private void preparerFichierEnPrep(DemandeSupp demande) throws IOException, DemandeCheckingException, FileTypeException {
+    private void preparerFichierEnPrep(DemandeSupp demande) throws IOException, DemandeCheckingException, FileTypeException, FileCheckingException {
         if (demande.getTypeSuppression() != null) {
             //Suppression d'un éventuel fichier existant sur le disque
             storageService.delete(fichierPrepare.getFilename());
             //Ecriture ligne d'en-tête dans FichierApresWS
             fichierPrepare.ecrireEnTete();
             //Alimentation du fichier par appel à la procédure Oracle ppntoepn
             appelProcStockee(demande.getRcr(), demande.getTypeSuppression());
-            demande.setEtatDemande(new EtatDemande(Constant.ETATDEM_PREPAREE));
             fichierPrepare.trierLignesDeCorrespondances();
+            demande.setEtatDemande(new EtatDemande(Constant.ETATDEM_PREPAREE));
             save(demande);
             checkEtatDemande(demande);
         }