Merge pull request #49 from abes-esr/develop

develop to main
abes-esr · Sep 30, 2024 · 79bb169 · 79bb169
2 parents b9eefd4 + f6aa200
commit 79bb169
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/.../java/fr/abes/item/batch/traitement/traiterlignesfichierchunk/LignesFichierProcessor.java b/.../java/fr/abes/item/batch/traitement/traiterlignesfichierchunk/LignesFichierProcessor.java
@@ -144,7 +144,9 @@ private LigneFichierDtoSupp processDemandeSupp(LigneFichierDto ligneFichierDto)
         //récupération des exemplaires existants pour cette ligne
         List<Exemplaire> exemplairesExistants = ((DemandeSuppService) strategyFactory.getStrategy(IDemandeService.class, TYPE_DEMANDE.SUPP))
                 .getExemplairesExistants(ligneFichierDtoSupp.getPpn());
-        this.fichierSauvegardeSupp.writePpnInFile(ligneFichierDtoSupp.getPpn(), exemplairesExistants);
+        if (!exemplairesExistants.isEmpty()){
+            this.fichierSauvegardeSupp.writePpnInFile(ligneFichierDtoSupp.getPpn(), exemplairesExistants);
+        }
         //supprimer l'exemplaire
         this.proxyRetry.deleteExemplaire(demandeSupp, ligneFichierDtoSupp);
         ligneFichierDtoSupp.setRetourSudoc(Constant.EXEMPLAIRE_SUPPRIME);

diff --git a/core/src/main/java/fr/abes/item/core/entities/item/DemandeSupp.java b/core/src/main/java/fr/abes/item/core/entities/item/DemandeSupp.java
@@ -1,5 +1,6 @@
 package fr.abes.item.core.entities.item;
 
+import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import fr.abes.item.core.constant.TYPE_DEMANDE;
 import fr.abes.item.core.constant.TYPE_SUPPRESSION;
 import jakarta.persistence.*;
@@ -8,12 +9,14 @@
 import lombok.Setter;
 
 import java.util.Date;
+import java.util.Set;
 
 @Entity
 @Table(name = "DEMANDE_SUPP")
 @NoArgsConstructor
 @Getter
 @Setter
+@JsonIgnoreProperties({"ligneFichierSupps"})
 public class DemandeSupp extends Demande {
     @Column(name = "TYPE_SUPPRESSION")
     @Enumerated(EnumType.STRING)
@@ -28,6 +31,9 @@ public TYPE_DEMANDE getTypeDemande() {
         return TYPE_DEMANDE.SUPP;
     }
 
+    @Getter @Setter @OneToMany(mappedBy = "demandeSupp", fetch = FetchType.LAZY, cascade = CascadeType.REMOVE)
+    private Set<LigneFichierSupp> ligneFichierSupps;
+
     public DemandeSupp(String rcr, Date dateCreation, Date dateModification, TYPE_SUPPRESSION typeSuppression,
                         String comment, EtatDemande etatDemande, Utilisateur utilisateur) {
         super(rcr, dateCreation, dateModification, etatDemande, comment, utilisateur);

diff --git a/core/src/main/java/fr/abes/item/core/service/FileSystemStorageService.java b/core/src/main/java/fr/abes/item/core/service/FileSystemStorageService.java
@@ -88,7 +88,7 @@ public Resource loadAsResource(String filename) {
             if (resource.exists() && resource.isReadable()) {
                 return resource;
             } else {
-                throw new StorageFileNotFoundException(Constant.ERR_FILE_READING + filename + " sur le chemin : " + file.toUri());
+                throw new StorageFileNotFoundException(Constant.ERR_FILE_READING + filename);
             }
         } catch (MalformedURLException e) {
             throw new StorageFileNotFoundException(Constant.ERR_FILE_READING + filename, e);