Skip to content

Commit b47e1fe

Browse files
fix: NPE personnes dans les fonctions mais pas dans un etab cas spécifique CFA
1 parent 2674bc3 commit b47e1fe

1 file changed

Lines changed: 7 additions & 3 deletions

File tree

‎src/main/java/fr/recia/manager/services/db/StructureService.java‎

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -225,8 +225,12 @@ public List<TypeFonctionFiliereDto> getComposition(StructureDto etablissement, L
225225
Long disciplineId = fonctionDto.getDiscipline();
226226
// Cas spécial pour les CFA
227227
if (disciplineId == null) {
228-
DatabasePersonneDto databasePersonneDto = personnesMap.get(fonctionDto.getPersonne());
229-
typeFonctionFiliereDto.getPersonnesWithoutDiscipline().add(databasePersonneDto);
228+
if (personnesMap.containsKey(fonctionDto.getPersonne())) {
229+
DatabasePersonneDto databasePersonneDto = personnesMap.get(fonctionDto.getPersonne());
230+
typeFonctionFiliereDto.getPersonnesWithoutDiscipline().add(databasePersonneDto);
231+
} else {
232+
log.warn("person {} in functions {} but not in structure {}", fonctionDto.getPersonne(), typeFonctionFiliereDto.getId(), etablissement.getId());
233+
}
230234
} else {
231235
if (disciplines.containsKey(disciplineId)) {
232236
DisciplineDto disciplineDto;
@@ -244,7 +248,7 @@ public List<TypeFonctionFiliereDto> getComposition(StructureDto etablissement, L
244248
disciplineDto.getCategories().add(databasePersonneDto.getCategorie());
245249
} else {
246250
// TODO : que faire dans ce cas si on se retrouve avec une discipline sans personne dedans ?
247-
log.warn("person in functions but not in structure for {} : {}", disciplineDto.getId(), fonctionDto.getPersonne());
251+
log.warn("person {} in functions {} but not in structure {}", fonctionDto.getPersonne(), disciplineDto.getId(), etablissement.getId());
248252
}
249253
} else {
250254
log.warn("discipline {} is not in known in disciplines for {}", disciplineId, etablissement.getSource());

0 commit comments

Comments
 (0)