oppiainevlk update fix

Opetushallitus · Jan 10, 2025 · bf2e983 · bf2e983
1 parent 83ab71b
commit bf2e983
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 15 deletions.
diff --git a/...teet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java b/...teet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java
@@ -403,18 +403,18 @@ private OppiaineenVuosiluokkaKokonaisuus doUpdateOppiaineenVuosiluokkaKokonaisuu
         }
 
         //hibernate deletoi ja updatee väärässä järjestyksessä jos @ordercolumn ja @jointable
-        List<Long> poistuneetTavoitteet = ovk.getTavoitteet().stream()
-                .map(OpetuksenTavoite::getId)
-                .filter(id -> !dto.getTavoitteet().stream().map(OpetuksenTavoiteDto::getId).collect(Collectors.toList()).contains(id))
-                        .collect(Collectors.toList());
-        ovk.setTavoitteet(
-                Stream.concat(
-                        ovk.getTavoitteet().stream().filter(tavoite -> !poistuneetTavoitteet.contains(tavoite.getId())),
-                        ovk.getTavoitteet().stream().filter(tavoite -> poistuneetTavoitteet.contains(tavoite.getId()))
-                        ).collect(Collectors.toList()));
-
-        ovk = vuosiluokkakokonaisuusRepository.saveAndFlush(ovk);
-        mapper.map(dto, ovk);
+//        List<Long> poistuneetTavoitteet = ovk.getTavoitteet().stream()
+//                .map(OpetuksenTavoite::getId)
+//                .filter(id -> !dto.getTavoitteet().stream().map(OpetuksenTavoiteDto::getId).collect(Collectors.toList()).contains(id))
+//                        .collect(Collectors.toList());
+//        ovk.setTavoitteet(
+//                Stream.concat(
+//                        ovk.getTavoitteet().stream().filter(tavoite -> !poistuneetTavoitteet.contains(tavoite.getId())),
+//                        ovk.getTavoitteet().stream().filter(tavoite -> poistuneetTavoitteet.contains(tavoite.getId()))
+//                        ).collect(Collectors.toList()));
+//
+//        ovk = vuosiluokkakokonaisuusRepository.saveAndFlush(ovk);
+//        mapper.map(dto, ovk);
 
         Set<Long> sisaltoalueetIdt = ovk.getSisaltoalueet().stream().map(KeskeinenSisaltoalue::getId).collect(Collectors.toSet());
         ovk.setTavoitteet(ovk.getTavoitteet().stream().peek(tavoite -> tavoite.setSisaltoalueet(tavoite.getSisaltoalueet().stream()

diff --git a/...erusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/DtoMapperConfig.java b/...erusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/DtoMapperConfig.java
@@ -683,21 +683,18 @@ public void mapAtoB(Koodi a, OsaamisalaDto b, MappingContext context) {
 
         factory.classMap(OppiaineDto.class, Oppiaine.class)
                 .mapNulls(true)
-                .fieldBToA("vuosiluokkakokonaisuudet", "vuosiluokkakokonaisuudet")
                 .customize(OppiaineCustomMapper.create(koodistoClient))
                 .byDefault()
                 .register();
 
         factory.classMap(OppiaineLaajaDto.class, Oppiaine.class)
                 .mapNulls(true)
-                .fieldBToA("vuosiluokkakokonaisuudet", "vuosiluokkakokonaisuudet")
                 .customize(OppiaineCustomMapper.create(koodistoClient))
                 .byDefault()
                 .register();
 
         factory.classMap(OppiaineSuppeaDto.class, Oppiaine.class)
                 .mapNulls(true)
-                .fieldBToA("vuosiluokkakokonaisuudet", "vuosiluokkakokonaisuudet")
                 .customize(OppiaineCustomMapper.create(koodistoClient))
                 .byDefault()
                 .register();