From bf2e98395f0ed67f73adc3a3e637a6528ba121e2 Mon Sep 17 00:00:00 2001 From: Olli-Pekka Lehtokallio Date: Fri, 10 Jan 2025 16:51:16 +0200 Subject: [PATCH] oppiainevlk update fix --- .../service/impl/yl/OppiaineServiceImpl.java | 24 +++++++++---------- .../service/mapping/DtoMapperConfig.java | 3 --- 2 files changed, 12 insertions(+), 15 deletions(-) diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java index ea6c37887..a885c0be4 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java +++ b/eperusteet/eperusteet-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 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 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 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/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/DtoMapperConfig.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/DtoMapperConfig.java index 2d4812cd1..9a2082357 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/mapping/DtoMapperConfig.java +++ b/eperusteet/eperusteet-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();