fix: display all people of structure

GIP-RECIA · Oct 16, 2023 · c6f40d2 · c6f40d2
1 parent 6176bba
commit c6f40d2
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 5 deletions.
diff --git a/src/main/java/fr/recia/glc/db/repositories/fonction/FonctionRepository.java b/src/main/java/fr/recia/glc/db/repositories/fonction/FonctionRepository.java
@@ -54,9 +54,8 @@ public interface FonctionRepository<T extends Fonction> extends AbstractReposito
   @Query("SELECT new fr.recia.glc.db.dto.fonction.FonctionDto(f.personne.id, f.disciplinePoste.id, f.filiere.id, " +
     "f.source) " +
     "FROM Fonction f " +
-    "WHERE f.structure.id = :structureId " +
-    "AND (f.source = :source OR f.source = CONCAT('SarapisUi_', :source))")
-  List<FonctionDto> findByStructureIdAndSource(Long structureId, String source);
+    "WHERE f.structure.id = :structureId")
+  List<FonctionDto> findByStructureId(Long structureId);
 
   @Query("SELECT f.id " +
     "FROM Fonction f " +

diff --git a/src/main/java/fr/recia/glc/web/rest/EtablissementController.java b/src/main/java/fr/recia/glc/web/rest/EtablissementController.java
@@ -60,6 +60,7 @@
 import java.util.stream.Collectors;
 
 import static fr.recia.glc.configuration.Constants.SANS_OBJET;
+import static fr.recia.glc.configuration.Constants.SARAPISUI_;
 
 @Slf4j
 @RestController
@@ -186,11 +187,23 @@ public ResponseEntity<EtablissementDto> getEtablissement(@PathVariable Long id)
   private List<TypeFonctionFiliereDto> getFilieresWithDisciplinesAndUsers(
     Long structureId, String source, List<SimplePersonneDto> personnes
   ) {
-    List<FonctionDto> fonctions = fonctionRepository.findByStructureIdAndSource(structureId, source);
+    List<FonctionDto> fonctions = fonctionRepository.findByStructureId(structureId);
+    if (fonctions.isEmpty()) return Collections.emptyList();
+
     List<TypeFonctionFiliereDto> typesFonctionFiliere = typeFonctionFiliereRepository.findBySourceSarapis(source);
     List<DisciplineDto> disciplines = disciplineRepository.findBySourceSarapis(source);
 
-    if (fonctions.isEmpty()) return Collections.emptyList();
+    if (log.isDebugEnabled())
+      log.debug(
+        "<==\n\t- structure : {}\n\t- filieres : {}\n\t- disciplines : {}\n\t- personnes : {}\n\t- source : {}\n\t- filieres of source : {}\n\t- disciplines of source : {}\n==>",
+        structureId,
+        fonctions.stream().map(FonctionDto::getFiliere).collect(Collectors.toSet()),
+        fonctions.stream().map(FonctionDto::getDisciplinePoste).collect(Collectors.toSet()),
+        fonctions.stream().map(FonctionDto::getPersonne).collect(Collectors.toSet()),
+        source.startsWith(SARAPISUI_) ? source : source + " AND " + SARAPISUI_ + source,
+        typesFonctionFiliere.stream().map(TypeFonctionFiliereDto::getId).collect(Collectors.toSet()),
+        disciplines.stream().map(DisciplineDto::getId).collect(Collectors.toSet())
+      );
 
     return typesFonctionFiliere.stream()
       // Ajout des disciplines aux filières