peruste täyttö fix

Opetushallitus · Dec 14, 2023 · 673a209 · 673a209
1 parent 27375c3
commit 673a209
Showing 1 changed file with 21 additions and 18 deletions.
diff --git a/...usteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/JulkaisutServiceImpl.java b/...usteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/JulkaisutServiceImpl.java
@@ -93,6 +93,7 @@
 import org.springframework.transaction.annotation.Propagation;
 import org.springframework.transaction.annotation.Transactional;
 import org.springframework.util.CollectionUtils;
+import org.springframework.util.ObjectUtils;
 import org.springframework.web.HttpMediaTypeNotSupportedException;
 
 import java.io.ByteArrayInputStream;
@@ -520,27 +521,29 @@ private Page<PerusteenJulkaisuData> taytaPerusteet(Page<PerusteenJulkaisuData> j
                 .map(julkaisuData -> julkaisuData.getTutkinnonosa().getKoodiUri())
                 .collect(Collectors.toSet());
 
-        Map<String, List<PerusteenJulkaisuData>> perusteetTutkinnonosanKoodilla = new HashMap<>();
-        julkaisutRepository.findAllJulkaistutPerusteetByKoodi(tutkinnonosaKoodit).stream()
-                .map(this::convertToPerusteData)
-                .forEach(perusteData -> {
-                    perusteData.getKoodit().forEach(koodi -> {
-                        if (!perusteetTutkinnonosanKoodilla.containsKey(koodi)) {
-                            perusteetTutkinnonosanKoodilla.put(koodi, new ArrayList<>());
-                        }
-                        perusteetTutkinnonosanKoodilla.get(koodi).add(perusteData);
+        if (!ObjectUtils.isEmpty(tutkinnonosaKoodit)) {
+            Map<String, List<PerusteenJulkaisuData>> perusteetTutkinnonosanKoodilla = new HashMap<>();
+            julkaisutRepository.findAllJulkaistutPerusteetByKoodi(tutkinnonosaKoodit).stream()
+                    .map(this::convertToPerusteData)
+                    .forEach(perusteData -> {
+                        perusteData.getKoodit().forEach(koodi -> {
+                            if (!perusteetTutkinnonosanKoodilla.containsKey(koodi)) {
+                                perusteetTutkinnonosanKoodilla.put(koodi, new ArrayList<>());
+                            }
+                            perusteetTutkinnonosanKoodilla.get(koodi).add(perusteData);
+                        });
                     });
-                });
 
-        julkaisut.map(julkaisu -> {
-            if (julkaisu.getTutkinnonosa() != null && perusteetTutkinnonosanKoodilla.containsKey(julkaisu.getTutkinnonosa().getKoodiUri())) {
-                if (julkaisu.getPerusteet() == null) {
-                    julkaisu.setPerusteet(new ArrayList<>());
+            julkaisut.map(julkaisu -> {
+                if (julkaisu.getTutkinnonosa() != null && perusteetTutkinnonosanKoodilla.containsKey(julkaisu.getTutkinnonosa().getKoodiUri())) {
+                    if (julkaisu.getPerusteet() == null) {
+                        julkaisu.setPerusteet(new ArrayList<>());
+                    }
+                    julkaisu.getPerusteet().addAll(perusteetTutkinnonosanKoodilla.get(julkaisu.getTutkinnonosa().getKoodiUri()));
                 }
-                julkaisu.getPerusteet().addAll(perusteetTutkinnonosanKoodilla.get(julkaisu.getTutkinnonosa().getKoodiUri()));
-            }
-            return julkaisu;
-        });
+                return julkaisu;
+            });
+        }
 
         return julkaisut;
     }