KJHH-2337 move yksilointi to henkilo controller from oppija controller

oppijanumerorekisteri-service/src/main/java/fi/vm/sade/oppijanumerorekisteri/controllers/HenkiloController.java

@@ -234,7 +234,7 @@ public HenkiloReadDto getByHetu(@PathVariable String hetu,
     @RequestMapping(value = "", method = RequestMethod.POST)
     public String createHenkiloFromHenkiloCreateDto(
             @ApiParam("Henkilön sukupuolen kelvolliset arvot löytyvät sukupuoli koodistosta.") @RequestBody @Validated HenkiloCreateDto henkilo) {
-        return this.henkiloModificationService.createHenkilo(henkilo).getOidHenkilo();
+        return this.henkiloModificationService.createAndYksiloiHenkilo(henkilo).getOidHenkilo();
     }
 
     @ApiOperation(value = "Henkilön olemassaolon tarkistus", authorizations = @Authorization("onr"), notes = "Tarkistaa henkilön olemassa olon annetun syötteen pohjalta.")

oppijanumerorekisteri-service/src/main/java/fi/vm/sade/oppijanumerorekisteri/services/HenkiloModificationService.java

@@ -24,6 +24,10 @@ public interface HenkiloModificationService {
 
     HenkiloDto createHenkilo(HenkiloCreateDto henkiloDto);
 
+    Henkilo createAndYksiloiHenkilo(HenkiloCreateDto henkiloDto);
+
+    boolean isHenkiloValidForHetuttomanYksilointi(Henkilo henkilo);
+
     Henkilo createHenkilo(HuoltajaCreateDto huoltajaCreateDto, String kasittelijaOid);
 
     /**

oppijanumerorekisteri-service/src/main/java/fi/vm/sade/oppijanumerorekisteri/services/YksilointiService.java

@@ -28,8 +28,6 @@ public interface YksilointiService {
      */
     Henkilo yksiloiManuaalisesti(final String henkiloOid);
 
-    boolean isHenkiloValidForHetuttomanYksilointi(Henkilo henkilo);
-
     /**
      * Virkailijan käynnistämä hetuttoman henkilön yksilöinti.
      * @param henkiloOid Henkilön oid

oppijanumerorekisteri-service/src/main/java/fi/vm/sade/oppijanumerorekisteri/services/impl/HenkiloModificationServiceImpl.java

@@ -122,7 +122,7 @@ public HenkiloUpdateDto updateHenkilo(HenkiloUpdateDto henkiloUpdateDto) {
 
         this.mapper.map(henkiloUpdateDto, henkiloSaved);
         // varmistetaan että tyhjä hetu tallentuu nullina
-        if (StringUtils.isEmpty(henkiloSaved.getHetu())) {
+        if (!StringUtils.hasLength(henkiloSaved.getHetu())) {
             henkiloUpdateDto.setHetu(null);
             henkiloSaved.setHetu(null);
         }
@@ -295,7 +295,7 @@ public Henkilo update(Henkilo henkilo) {
         setSyntymaaikaAndSukupuoliFromHetu(henkilo);
 
         // varmistetaan että tyhjä hetu tallentuu nullina
-        if (StringUtils.isEmpty(henkilo.getHetu())) {
+        if (!StringUtils.hasLength(henkilo.getHetu())) {
             henkilo.setHetu(null);
         }
 
@@ -405,6 +405,38 @@ public List<HenkiloPerustietoDto> findOrCreateHenkiloFromPerustietoDto(List<Henk
                 .collect(toList());
     }
 
+    @Override
+    @Transactional
+    public Henkilo createAndYksiloiHenkilo(HenkiloCreateDto henkiloDto) {
+        Henkilo henkilo = this.mapper.map(henkiloDto, Henkilo.class);
+        Henkilo created = this.createHenkilo(henkilo);
+        if (isHenkiloValidForHetuttomanYksilointi(created)) {
+            HenkiloDuplikaattiCriteria criteria = new HenkiloDuplikaattiCriteria(created.getEtunimet(),
+                created.getKutsumanimi(), created.getSukunimi(), created.getSyntymaaika());
+            int duplikaatit = this.henkiloDataRepository.findDuplikaatitCount(criteria, created.getOidHenkilo());
+            if (duplikaatit < 1) {
+                created.setYksiloity(true);
+                this.henkiloDataRepository.save(created);
+            }
+        }
+        return created;
+    }
+
+    @Override
+    public boolean isHenkiloValidForHetuttomanYksilointi(Henkilo henkilo) {
+        return henkilo.getHetu() == null
+            && henkilo.isYksiloityVTJ() == false
+            && henkilo.isYksiloity() == false
+            && henkilo.isDuplicate() == false
+            && StringUtils.hasLength(henkilo.getEtunimet())
+            && StringUtils.hasLength(henkilo.getSukunimi())
+            && StringUtils.hasLength(henkilo.getKutsumanimi())
+            && StringUtils.hasLength(henkilo.getSukupuoli())
+            && henkilo.getSyntymaaika() != null
+            && henkilo.getAidinkieli() != null
+            && henkilo.getKansalaisuus() != null && henkilo.getKansalaisuus().size() > 0;
+    }
+
     @Override
     @Transactional
     public HenkiloDto createHenkilo(HenkiloCreateDto henkiloDto) {
@@ -421,7 +453,7 @@ public Henkilo createHenkilo(HuoltajaCreateDto huoltajaCreateDto, String kasitte
             throw new UnprocessableEntityException(errors);
         }
         Henkilo henkilo = this.mapper.map(huoltajaCreateDto, Henkilo.class);
-        return this.createHenkilo(henkilo, kasittelijaOid, StringUtils.isEmpty(henkilo.getHetu()));
+        return this.createHenkilo(henkilo, kasittelijaOid, !StringUtils.hasLength(henkilo.getHetu()));
     }
 
     @Override
@@ -448,7 +480,7 @@ public Henkilo createHenkilo(Henkilo henkiloCreate, String kasittelijaOid, boole
         }
 
         // varmistetaan että tyhjä hetu tallentuu nullina
-        if (StringUtils.isEmpty(henkiloCreate.getHetu())) {
+        if (!StringUtils.hasLength(henkiloCreate.getHetu())) {
             henkiloCreate.setHetu(null);
         }
         setSyntymaaikaAndSukupuoliFromHetu(henkiloCreate);

oppijanumerorekisteri-service/src/main/java/fi/vm/sade/oppijanumerorekisteri/services/impl/OppijaServiceImpl.java

@@ -51,7 +51,6 @@ public class OppijaServiceImpl implements OppijaService {
     private final UserDetailsHelper userDetailsHelper;
     private final PermissionChecker permissionChecker;
     private final ObjectMapper objectMapper;
-    private final YksilointiService yksilointiService;
 
     @Override
     public String create(OppijaCreateDto dto) {
@@ -63,15 +62,6 @@ public String create(OppijaCreateDto dto) {
         organisaatiot.forEach(entity::addOrganisaatio);
 
         entity = henkiloModificationService.createHenkilo(entity, kayttajaOid, true);
-        if (yksilointiService.isHenkiloValidForHetuttomanYksilointi(entity)) {
-            HenkiloDuplikaattiCriteria criteria = new HenkiloDuplikaattiCriteria(entity.getEtunimet(),
-                entity.getKutsumanimi(), entity.getSukunimi(), entity.getSyntymaaika());
-            int duplikaatit = henkiloRepository.findDuplikaatitCount(criteria, entity.getOidHenkilo());
-            if (duplikaatit < 1) {
-                entity.setYksiloity(true);
-                henkiloRepository.save(entity);
-            }
-        }
 
         return entity.getOidHenkilo();
     }

oppijanumerorekisteri-service/src/main/java/fi/vm/sade/oppijanumerorekisteri/services/impl/YksilointiServiceImpl.java

@@ -149,21 +149,6 @@ public Henkilo yksiloiManuaalisesti(final String henkiloOid) {
         return yksiloiHenkilo(henkilo);
     }
 
-    @Override
-    public boolean isHenkiloValidForHetuttomanYksilointi(Henkilo henkilo) {
-        return henkilo.getHetu() == null
-            && henkilo.isYksiloityVTJ() == false
-            && henkilo.isYksiloity() == false
-            && henkilo.isDuplicate() == false
-            && StringUtils.hasLength(henkilo.getEtunimet())
-            && StringUtils.hasLength(henkilo.getSukunimi())
-            && StringUtils.hasLength(henkilo.getKutsumanimi())
-            && StringUtils.hasLength(henkilo.getSukupuoli())
-            && henkilo.getSyntymaaika() != null
-            && henkilo.getAidinkieli() != null
-            && henkilo.getKansalaisuus() != null && henkilo.getKansalaisuus().size() > 0;
-    }
-
     @Override
     @Transactional
     public Henkilo hetuttomanYksilointi(String henkiloOid) {

oppijanumerorekisteri-service/src/test/java/fi/vm/sade/oppijanumerorekisteri/controllers/HenkiloControllerTest.java

@@ -1,6 +1,7 @@
 package fi.vm.sade.oppijanumerorekisteri.controllers;
 
 import com.fasterxml.jackson.databind.ObjectMapper;
+
 import fi.vm.sade.auditlog.Target;
 import fi.vm.sade.oppijanumerorekisteri.FilesystemHelper;
 import fi.vm.sade.oppijanumerorekisteri.OppijanumerorekisteriServiceApplication;
@@ -11,7 +12,10 @@
 import fi.vm.sade.oppijanumerorekisteri.dto.*;
 import fi.vm.sade.oppijanumerorekisteri.exceptions.ConflictException;
 import fi.vm.sade.oppijanumerorekisteri.exceptions.NotFoundException;
+import fi.vm.sade.oppijanumerorekisteri.mappers.EntityUtils;
 import fi.vm.sade.oppijanumerorekisteri.models.Henkilo;
+import fi.vm.sade.oppijanumerorekisteri.models.Kansalaisuus;
+import fi.vm.sade.oppijanumerorekisteri.models.Kielisyys;
 import fi.vm.sade.oppijanumerorekisteri.repositories.OrganisaatioRepository;
 import fi.vm.sade.oppijanumerorekisteri.services.*;
 import fi.vm.sade.oppijanumerorekisteri.services.impl.PermissionCheckerImpl;
@@ -335,9 +339,19 @@ public void createHenkiloFromHenkiloCreateDto() throws Exception {
                         .yhteystietoArvo("[email protected]")
                         .build())
                 .build()));
-        HenkiloDto henkiloDtoOutput = DtoUtils.createHenkiloDto("arpa", "arpa", "kuutio", "081296-967T", "1.2.3.4.5",
-                false, "fi", "suomi", "246", "1.2.3.4.1", "[email protected]");
-        given(this.henkiloModificationService.createHenkilo(any(HenkiloCreateDto.class))).willReturn(henkiloDtoOutput);
+        Henkilo henkiloDtoOutput = Henkilo.builder()
+                .oidHenkilo("1.2.3.4.5")
+                .hetu("081296-967T")
+                .etunimet("arpa")
+                .kutsumanimi("arpa")
+                .sukunimi("kuutio")
+                .aidinkieli(new Kielisyys("fi", "suomi"))
+                .asiointiKieli(new Kielisyys("fi", "suomi"))
+                .passivoitu(false)
+                .kansalaisuus(Set.of(new Kansalaisuus("246")))
+                .yhteystiedotRyhma(Set.of(EntityUtils.createYhteystiedotRyhma("[email protected]")))
+                .build();
+        given(this.henkiloModificationService.createAndYksiloiHenkilo(any(HenkiloCreateDto.class))).willReturn(henkiloDtoOutput);
         this.mvc.perform(post("/henkilo").content(this.objectMapper.writeValueAsString(henkiloDtoInput))
                         .with(csrf())
                         .contentType(MediaType.APPLICATION_JSON))

oppijanumerorekisteri-service/src/test/java/fi/vm/sade/oppijanumerorekisteri/services/impl/OppijaServiceImplTest.java

@@ -89,7 +89,7 @@ public void setup() {
                 organisaatioServiceMock, mapperMock, henkiloRepositoryMock,
                 tuontiRepositoryMock,
                 organisaatioRepositoryMock, henkiloViiteRepositoryMock, userDetailsHelperMock,
-                permissionCheckerMock, objectMapperMock, yksilointiService);
+                permissionCheckerMock, objectMapperMock);
     }
 
     @Test