merge dev to test

.github/workflows/create-release.yml

@@ -13,6 +13,9 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
+      - name: "debug info"
+        run: |
+          echo ${{github}}
       - name: "Calculate versions"
         id: calculate-version
         run: |

Dockerfile

@@ -65,6 +65,7 @@ RUN ln -snf /usr/share/zoneinfo/$TZ /etc/localtime && echo $TZ > /etc/timezone
 # systeme pour les crontab
 # cronie: remplacant de crond qui support le CTRL+C dans docker (sans ce système c'est compliqué de stopper le conteneur)
 # gettext: pour avoir envsubst qui permet de gérer le template tasks.tmpl
+RUN yum install -y procps
 RUN dnf install -y cronie gettext && \
     crond -V && rm -rf /etc/cron.*/*
 COPY ./docker/batch/tasks.tmpl /etc/cron.d/tasks.tmpl
@@ -89,6 +90,8 @@ COPY ./docker/batch/itemBatchTraiterLigneFichierModif.sh /scripts/itemBatchTrait
 RUN chmod +x /scripts/itemBatchTraiterLigneFichierModif.sh
 COPY ./docker/batch/itemBatchTraiterLigneFichierRecouv.sh /scripts/itemBatchTraiterLigneFichierRecouv.sh
 RUN chmod +x /scripts/itemBatchTraiterLigneFichierRecouv.sh
+COPY ./docker/batch/itemBatchTraiterLigneFichierSupp.sh /scripts/itemBatchTraiterLigneFichierSupp.sh
+RUN chmod +x /scripts/itemBatchTraiterLigneFichierSupp.sh
 
 COPY --from=build-image /build/batch/target/*.jar /scripts/item-batch.jar
 RUN chmod +x /scripts/item-batch.jar

batch/pom.xml

@@ -3,7 +3,7 @@
     <parent>
         <artifactId>itembackoffice</artifactId>
         <groupId>fr.abes.item</groupId>
-        <version>2.4.11</version>
+        <version>2.4.12-SNAPSHOT</version>
     </parent>
     <modelVersion>4.0.0</modelVersion>
     <packaging>jar</packaging>
@@ -46,6 +46,12 @@
             <groupId>org.apache.httpcomponents</groupId>
             <artifactId>httpclient</artifactId>
             <version>4.5.14</version>
+            <exclusions>
+                <exclusion>
+                    <groupId>commons-logging</groupId>
+                    <artifactId>commons-logging</artifactId>
+                </exclusion>
+            </exclusions>
         </dependency>
         <dependency>
             <groupId>org.apache.httpcomponents</groupId>

batch/src/main/java/fr/abes/item/batch/JobConfiguration.java

@@ -82,7 +82,7 @@ public ExecutionContextSerializer configureSerializer() {
     // ----- CHUNK ------
     @Bean
     public LignesFichierReader reader() {
-        return new LignesFichierReader(proxyRetry);
+        return new LignesFichierReader();
     }
     @Bean
     public LignesFichierProcessor processor() {
@@ -101,6 +101,8 @@ public LignesFichierWriter writer() {
     @Bean
     public Tasklet getNextDemandeRecouvTasklet() { return new GetNextDemandeRecouvTasklet(strategyFactory, minHour, maxHour); }
     @Bean
+    public Tasklet getNextDemandeSuppTasklet() { return new GetNextDemandeSuppTasklet(strategyFactory, minHour, maxHour); }
+    @Bean
     public Tasklet lireLigneFichierTasklet() { return new LireLigneFichierTasklet(strategyFactory, mailAdmin); }
     @Bean
     public Tasklet authentifierSurSudocTasklet()
@@ -179,6 +181,12 @@ public Step stepRecupererNextDemandeRecouv(JobRepository jobRepository, @Qualifi
                 .tasklet(tasklet, transactionManager)
                 .build();
     }
+    @Bean
+    public Step stepRecupererNextDemandeSupp(JobRepository jobRepository, @Qualifier("getNextDemandeSuppTasklet") Tasklet tasklet, PlatformTransactionManager transactionManager) {
+        return new StepBuilder("stepRecupererNextDemandeSupp", jobRepository).allowStartIfComplete(true)
+                .tasklet(tasklet, transactionManager)
+                .build();
+    }
     // Steps pour lancement d'un traitement de modification de masse
     @Bean
     public Step stepLireLigneFichier(JobRepository jobRepository, @Qualifier("lireLigneFichierTasklet") Tasklet tasklet, PlatformTransactionManager transactionManager) {
@@ -334,6 +342,21 @@ public Job jobTraiterLigneFichierRecouv(JobRepository jobRepository, @Qualifier(
                 .build().build();
     }
 
+    //job de lancement d'une demande de suppression
+    @Bean
+    public Job jobTraiterLigneFichierSupp(JobRepository jobRepository, @Qualifier("stepRecupererNextDemandeSupp") Step step1, @Qualifier("stepLireLigneFichier") Step step2, @Qualifier("stepAuthentifierSurSudoc") Step step3, @Qualifier("stepTraiterLigneFichier") Step step4, @Qualifier("stepGenererFichier") Step step5) {
+        return new JobBuilder("traiterLigneFichierSupp", jobRepository).incrementer(incrementer())
+                .start(step1).on(Constant.FAILED).end()
+                .from(step1).on(Constant.AUCUNE_DEMANDE).end()
+                .from(step1).on(Constant.COMPLETED).to(step2)
+                .from(step2).on(Constant.FAILED).end()
+                .from(step2).on(Constant.COMPLETED).to(step3)
+                .from(step3).on(Constant.FAILED).end()
+                .from(step3).on(Constant.COMPLETED).to(step4)
+                .from(step4).on(Constant.FAILED).end()
+                .from(step4).on(Constant.COMPLETED).to(step5)
+                .build().build();
+    }
 
     // Job d'export des statistiques mensuelles
     @Bean

batch/src/main/java/fr/abes/item/batch/mail/impl/Mailer.java

@@ -12,6 +12,7 @@
 import org.apache.http.impl.client.CloseableHttpClient;
 import org.apache.http.impl.client.HttpClients;
 import org.springframework.beans.factory.annotation.Value;
+import org.springframework.core.env.Environment;
 import org.springframework.http.HttpHeaders;
 import org.springframework.http.MediaType;
 import org.springframework.http.converter.StringHttpMessageConverter;
@@ -25,13 +26,18 @@
 
 @Slf4j
 public class Mailer {
+    private final Environment env;
     @Value("${mail.ws.url}")
     protected String url;
 
 
     @Value("${mail.admin}")
     protected String mailAdmin;
 
+    public Mailer(Environment env) {
+        this.env = env;
+    }
+
 
     public void sendMail(String requestJson) {
         RestTemplate restTemplate = new RestTemplate(); //appel ws qui envoie le mail
@@ -93,4 +99,14 @@ protected String mailToJSON(String to, String subject, String text) {
         }
         return json;
     }
+
+    protected String getEnvironnementExecution() {
+        if (env.getActiveProfiles().length>0)
+            if (!"prod".equalsIgnoreCase(env.getActiveProfiles()[0])) {
+                return env.getActiveProfiles()[0];
+            }
+            else
+                return "";
+        return "Local";
+    }
 }

batch/src/main/java/fr/abes/item/batch/mail/impl/MailerExemp.java

@@ -18,12 +18,11 @@
 @Service
 @Strategy(type = IMailer.class, typeDemande = TYPE_DEMANDE.EXEMP)
 public class MailerExemp extends Mailer implements IMailer {
-    private final Environment env;
     private final LigneFichierExempService service;
 
 
     public MailerExemp(Environment env, LigneFichierExempService service) {
-        this.env = env;
+        super(env);
         this.service = service;
     }
 
@@ -35,7 +34,7 @@ public MailerExemp(Environment env, LigneFichierExempService service) {
     @Override
     public void mailDebutTraitement(String mailDestinataire, Demande demande){
         DemandeExemp demandeExemp = (DemandeExemp) demande;
-        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_EXEMPLARISATION_START + ((DemandeExemp) demande).getTypeExemp().getLibelle() + " N°" + demande.getId() + Constant.DEMANDE_MAIL_DEBUT + " - ILN " + demande.getIln()
+        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_EXEMPLARISATION_START + ((DemandeExemp) demande).getTypeExemp().getLibelle() + " N°" + demande.getId() + Constant.DEMANDE_MAIL_DEBUT + " - ILN " + demande.getIln() + " / " + getEnvironnementExecution()
                 , "Bonjour, <br/>Votre exemplarisation - " + demandeExemp.getTypeExemp().getLibelle() + " N° " + demande.getId() + " - est en cours de traitement.<br/>"
                 + "Pour toute information complémentaire, merci de bien vouloir déposer une demande sur le guichet d'assistance : <a href='https://stp.abes.fr/node/3?origine=sudocpro' target='_blank'>https://stp.abes.fr/</a><br/>"
                 + "<br/>Cordialement,<br/>L'équipe ITEM.");
@@ -48,7 +47,7 @@ public void mailFinTraitement(String mailDestinataire, Demande demande, File f,
         DemandeExemp demandeExemp = (DemandeExemp) demande;
         int nbExempCree = service.getNbLigneFichierSuccessByDemande(demandeExemp);
         int nbRechercheTotal = service.getNbLigneFichierTotalByDemande(demandeExemp);
-        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_EXEMPLARISATION_END + demandeExemp.getTypeExemp().getLibelle() + " N°" + demandeExemp.getId() + " - ILN " + demande.getIln(),
+        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_EXEMPLARISATION_END + demandeExemp.getTypeExemp().getLibelle() + " N°" + demandeExemp.getId() + " - ILN " + demande.getIln() + " / " + getEnvironnementExecution(),
                 "Bonjour,<br/>Votre exemplarisation - " + demandeExemp.getTypeExemp().getLibelle() + " N°" + demandeExemp.getId() + " - est terminée." + Constant.HTML_BALISE_BR +
                         "Bilan :" + Constant.HTML_BALISE_BR +
                         "Exemplarisation démarrée le : " + Constant.formatDate.format(dateDebut) + Constant.HTML_BALISE_BR +
@@ -76,7 +75,7 @@ public void mailFinTraitement(String mailDestinataire, Demande demande, File f,
     @Override
     public void mailEchecTraitement(String mailDestinataire, Demande demande, LocalDateTime dateDebut){
         DemandeExemp demandeExemp = (DemandeExemp) demande;
-        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_EXEMPLARISATION_START + demandeExemp.getTypeExemp().getLibelle() + " N°" + demandeExemp.getId() + "-" + Constant.DEMANDE_MAIL_ECHEC + " - ILN " + demande.getIln(),
+        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_EXEMPLARISATION_START + demandeExemp.getTypeExemp().getLibelle() + " N°" + demandeExemp.getId() + "-" + Constant.DEMANDE_MAIL_ECHEC + " - ILN " + demande.getIln() + " / " + getEnvironnementExecution(),
                 "Bonjour,<br/><br/>" +
                      "votre exemplarisation - " + demandeExemp.getTypeExemp().getLibelle() +" N°" + demandeExemp.getId() + " lancée le " + dateDebut + "n'a pas pu être traitée.<br/><br/>" +
                      "Il convient de ne pas relancer l’exemplarisation. Le traitement reprendra à l’endroit où l’erreur s’est produite : c’est-à-dire que le programme se relancera automatiquement dès que le dysfonctionnement sera résolu, sans intervention manuelle de votre part." +
@@ -88,7 +87,7 @@ public void mailEchecTraitement(String mailDestinataire, Demande demande, LocalD
 
     @Override
     public void mailAlertAdmin(String mailDestinataire, Demande demande) {
-        String requestJson = mailToJSON(mailDestinataire+";"+mailAdmin, "Erreur dans Item / Exemplarisation" + " - ILN " + demande.getIln() + " / " + ((env.getActiveProfiles().length>0)?env.getActiveProfiles()[0]:"Local"), "Une erreur vient de se produire sur Item sur la demande" + demande.getId());
+        String requestJson = mailToJSON(mailDestinataire+";"+mailAdmin, "Erreur dans Item / Exemplarisation" + " - ILN " + demande.getIln() + " / " + getEnvironnementExecution(), "Une erreur vient de se produire sur Item sur la demande" + demande.getId());
         sendMail(requestJson);
     }
 }

batch/src/main/java/fr/abes/item/batch/mail/impl/MailerModif.java

@@ -17,11 +17,9 @@
 @Service
 @Strategy(type = IMailer.class, typeDemande = TYPE_DEMANDE.MODIF)
 public class MailerModif extends Mailer implements IMailer {
-    private final Environment env;
-
     private final LigneFichierModifService service;
     public MailerModif(Environment env, LigneFichierModifService service) {
-        this.env = env;
+        super(env);
         this.service = service;
     }
 
@@ -33,21 +31,21 @@ public MailerModif(Environment env, LigneFichierModifService service) {
     @Override
     public void mailDebutTraitement(String mailDestinataire, Demande demande){
 
-        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_MODIFICATION_START + demande.getId() + " lancée - ILN " + demande.getIln()
-                + " lancée.", "Bonjour, <br/>Votre demande de modification d'exemplaire n° " + demande.getId() + " a été lancée.<br />"+
+        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_MODIFICATION_START + demande.getId() + " lancée - ILN " + demande.getIln() + " / " + getEnvironnementExecution()
+                + " lancée.", "Bonjour, <br/>Votre demande de modification d'exemplaires n° " + demande.getId() + " a été lancée.<br />"+
                 "<br />Cordialement.<br/>L'équipe ITEM'.");
         sendMail(requestJson);
     }
 
     @Override
     public void mailFinTraitement(String mailDestinataire, Demande demande, File f, LocalDateTime dateDebut, LocalDateTime dateFin) {
         int numDemande = demande.getId();
-        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_MODIFICATION_START + numDemande + " terminée - ILN " + demande.getIln(),
+        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_MODIFICATION_START + numDemande + " terminée - ILN " + demande.getIln() + " / " + getEnvironnementExecution(),
                 "Bonjour,<br />Votre demande " + numDemande + " de modification d'exemplaires a bien été traitée.<br />" +
                         "Nombre d'exemplaires traités : " + service.getNbLigneFichierTraiteeByDemande(demande) + Constant.HTML_BALISE_BR +
                         "Nombre de traitements effectués avec succès : " + service.getNbLigneFichierSuccessByDemande(demande) + Constant.HTML_BALISE_BR +
                         "Nombre de traitements échoués : " + service.getNbLigneFichierErrorByDemande(demande) + Constant.HTML_BALISE_BR +
-                        "Vous pouvez retrouver le résultat de votre demande depuis <a href='https://item.sudoc.fr/tableau'>le tableau de bord de l'application.</a> <br />" +
+                        "Vous pouvez retrouver le résultat de votre demande depuis <a href='https://item.sudoc.fr/modification-tableau'>le tableau de bord de l'application.</a> <br />" +
                         "Cordialement.<br/>Les services de l'Abes.");
         sendMailWithAttachment(requestJson, f);
     }
@@ -60,7 +58,7 @@ public void mailFinTraitement(String mailDestinataire, Demande demande, File f,
      */
     @Override
     public void mailEchecTraitement(String mailDestinataire, Demande demande, LocalDateTime dateDebut){
-        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_MODIFICATION_START + demande.getId() + Constant.DEMANDE_MAIL_ECHEC + " - ILN " + demande.getIln(),
+        String requestJson = mailToJSON(mailDestinataire, Constant.DEMANDE_MODIFICATION_START + demande.getId() + Constant.DEMANDE_MAIL_ECHEC + " - ILN " + demande.getIln() + " / " + getEnvironnementExecution(),
                 "Bonjour,<br />Votre modification d'exemplaires -  N°" + demande.getId() + "n'a pas pu être exécutée. Une erreur vient de se produire sur ITEM. Dès que l'incident sera résolu vous recevrez un message vous indiquant la reprise du traitement. Cela ne nécessite aucune intervention de votre part." +
                         "Pour toute information complémentaire, merci de bien vouloir déposer une demande sur le guichet d'assistance : <a href=\"https://stp.abes.fr/node/3?origine=sudocpro/\" target=\"_blank\"> https://stp.abes.fr</a>"+
                         "<br />Cordialement.<br/>L'équipe ITEM.");
@@ -69,7 +67,7 @@ public void mailEchecTraitement(String mailDestinataire, Demande demande, LocalD
 
     @Override
     public void mailAlertAdmin(String mailUtilisateur, Demande demande) {
-        String requestJson = mailToJSON(mailAdmin, "Erreur dans Item / Modification" + " - ILN " + demande.getIln() + " / " + ((env.getActiveProfiles().length>0)?env.getActiveProfiles()[0]:"Local"), "Une erreur vient de se produire sur Item / Modification sur la demande" + demande.getId());
+        String requestJson = mailToJSON(mailAdmin, "Erreur dans Item / Modification" + " - ILN " + demande.getIln() + " / " + getEnvironnementExecution(), "Une erreur vient de se produire sur Item / Demande de Modification " + demande.getId());
         sendMail(requestJson);
 
     }