Merge pull request #3245 from Opetushallitus/tor-2240-nimetaan-kentta…

…-uusiksi TOR-2240: Vaihdetaan kentän nimi siirtynytTutkinnonUusiinPerusteisiin -> siirtynytUusiinTutkinnonPerusteisiin
Opetushallitus · Dec 11, 2024 · d88c589 · d88c589
2 parents cc639af + b2f0ff9
commit d88c589
Show file tree

Hide file tree

Showing 7 changed files with 19 additions and 19 deletions.
diff --git a/src/main/resources/localization/koski-default-texts.json b/src/main/resources/localization/koski-default-texts.json
@@ -2760,6 +2760,6 @@
   "raportti-excel-muks-title": "Muu kuin säännelty koulutus",
   "raportti-excel-must-tiedoston-etuliite": "muks_suoritustiedot",
   "raportti-excel-kolumni-yhteislaajuusKaikkiTuntia": "Yhteislaajuus (tuntia)",
-  "Siirtynyt tutkinnon uusiin perusteisiin" : "Siirtynyt tutkinnon uusiin perusteisiin",
-  "description:Onko opiskeluoikeuden päättymisen syy siirtyminen..." : "Onko opiskeluoikeuden päättymisen syy siirtyminen tutkinnon uusiin perusteisiin."
+  "Siirtynyt uusiin tutkinnon perusteisiin": "Siirtynyt uusiin tutkinnon perusteisiin",
+  "description:Onko opiskeluoikeuden päättymisen syy siirtyminen...": "Onko opiskeluoikeuden päättymisen syy siirtyminen tutkinnon uusiin perusteisiin."
 }
diff --git a/src/main/scala/fi/oph/koski/schema/Ammatillinen.scala b/src/main/scala/fi/oph/koski/schema/Ammatillinen.scala
@@ -116,7 +116,7 @@ case class AmmatillisenOpiskeluoikeudenLisätiedot(
   oikeuttaMaksuttomuuteenPidennetty: Option[List[OikeuttaMaksuttomuuteenPidennetty]] = None,
   jotpaAsianumero: Option[Koodistokoodiviite] = None,
   @Description("Onko opiskeluoikeuden päättymisen syy siirtyminen tutkinnon uusiin perusteisiin.")
-  siirtynytTutkinnonUusiinPerusteisiin: Option[Boolean] = None,
+  siirtynytUusiinTutkinnonPerusteisiin: Option[Boolean] = None,
 ) extends OpiskeluoikeudenLisätiedot
   with Ulkomaanjaksollinen
   with SisäoppilaitosmainenMajoitus

diff --git a/src/main/scala/fi/oph/koski/validation/AmmatillinenValidation.scala b/src/main/scala/fi/oph/koski/validation/AmmatillinenValidation.scala
@@ -401,11 +401,11 @@ object AmmatillinenValidation {
     })
 
   private def validateTutkinnonUusiinPerusteisiinSiirtyminen(oo: AmmatillinenOpiskeluoikeus): HttpStatus = {
-    val siirtynytTutkinnonUusiinPerusteisiin = oo.lisätiedot.exists(_.siirtynytTutkinnonUusiinPerusteisiin.contains(true))
+    val siirtynytUusiinTutkinnonPerusteisiin = oo.lisätiedot.exists(_.siirtynytUusiinTutkinnonPerusteisiin.contains(true))
     val katsotaanEronneeksi = oo.tila.opiskeluoikeusjaksot.exists(_.tila.koodiarvo == "katsotaaneronneeksi")
 
-    HttpStatus.validate(!siirtynytTutkinnonUusiinPerusteisiin || katsotaanEronneeksi) {
-      KoskiErrorCategory.badRequest.validation.tila.eiPäättävääTilaa("Opiskeluoikeudella, jonka lisätiedoissa on merkintä 'siirtynyt tutkinnon uusiin perusteisiin', pitää päättyä tilaan 'katsotaan eronneeksi'.")
+    HttpStatus.validate(!siirtynytUusiinTutkinnonPerusteisiin || katsotaanEronneeksi) {
+      KoskiErrorCategory.badRequest.validation.tila.eiPäättävääTilaa("Opiskeluoikeudella, jonka lisätiedoissa on merkintä 'siirtynyt uusiin tutkinnon perusteisiin', pitää päättyä tilaan 'katsotaan eronneeksi'.")
     }
   }
 }
diff --git a/src/test/scala/fi/oph/koski/api/oppijavalidation/OppijaValidationAmmatillinenSpec.scala b/src/test/scala/fi/oph/koski/api/oppijavalidation/OppijaValidationAmmatillinenSpec.scala
@@ -1490,20 +1490,20 @@ class OppijaValidationAmmatillinenSpec extends TutkinnonPerusteetTest[Ammatillin
 
     "Siirtyminen uudempiin perusteisiin" - {
       "Tietoa ei voi siirtää, jos opiskeluoikeus ei ole terminaalitilassa" in {
-        val lisätiedot = AmmatillisenOpiskeluoikeudenLisätiedot(siirtynytTutkinnonUusiinPerusteisiin = Some(true))
+        val lisätiedot = AmmatillisenOpiskeluoikeudenLisätiedot(siirtynytUusiinTutkinnonPerusteisiin = Some(true))
         val opiskeluoikeus = defaultOpiskeluoikeus.copy(
           lisätiedot = Some(lisätiedot),
           tila = AmmatillinenOpiskeluoikeudenTila(List(
             AmmatillinenOpiskeluoikeusjakso(LocalDate.of(2000, 1, 2), ExampleData.opiskeluoikeusLäsnä, Some(ExampleData.valtionosuusRahoitteinen)),
           )),
         )
         setupOppijaWithOpiskeluoikeus(opiskeluoikeus) {
-          verifyResponseStatus(400, KoskiErrorCategory.badRequest.validation.tila.eiPäättävääTilaa("Opiskeluoikeudella, jonka lisätiedoissa on merkintä 'siirtynyt tutkinnon uusiin perusteisiin', pitää päättyä tilaan 'katsotaan eronneeksi'."))
+          verifyResponseStatus(400, KoskiErrorCategory.badRequest.validation.tila.eiPäättävääTilaa("Opiskeluoikeudella, jonka lisätiedoissa on merkintä 'siirtynyt uusiin tutkinnon perusteisiin', pitää päättyä tilaan 'katsotaan eronneeksi'."))
         }
       }
 
       "Tiedon voi siirtää, jos opiskeluoikeus on päättynyt 'katsotaan eronneeksi' -tilaan" in {
-        val lisätiedot = AmmatillisenOpiskeluoikeudenLisätiedot(siirtynytTutkinnonUusiinPerusteisiin = Some(true))
+        val lisätiedot = AmmatillisenOpiskeluoikeudenLisätiedot(siirtynytUusiinTutkinnonPerusteisiin = Some(true))
         val opiskeluoikeus = AmmatillinenOpiskeluoikeusTestData.katsotaanEronneeksiOpiskeluoikeus(
           oppilaitosId = MockOrganisaatiot.stadinAmmattiopisto,
         )

diff --git a/tiedonsiirtoprotokollan_muutoshistoria.md b/tiedonsiirtoprotokollan_muutoshistoria.md
@@ -1,10 +1,10 @@
-## xx.11.2024
+## 11.12.2024
 
-- Lisätty kaikille opiskeluoikeuksille uusi kenttä `lähdejärjestelmäkytkentäPurettu`, joka ilmaisee että opiskeluoikeudelta on poistettu lähdejärjestelmätunniste, jotta sitä voidaan muokata KOSKI-käyttöliittymällä.
+- Lisätty ammatillisen opiskeluoikeuden lisätietoihin uusi kenttä `siirtynytUusiinTutkinnonPerusteisiin`, joka ilmaisee opiskeluoikeuden päättymisen syyksi siirtymisen uudempiin perusteisiin.
 
-## 26.11.2024
+## xx.11.2024
 
-- Lisätty ammatillisen opiskeluoikeuden lisätietoihin uusi kenttä `siirtynytTutkinnonUusiinPerusteisiin`, joka ilmaisee opiskeluoikeuden päättymisen syyksi siirtymisen uudempiin perusteisiin.
+- Lisätty kaikille opiskeluoikeuksille uusi kenttä `lähdejärjestelmäkytkentäPurettu`, joka ilmaisee että opiskeluoikeudelta on poistettu lähdejärjestelmätunniste, jotta sitä voidaan muokata KOSKI-käyttöliittymällä.
 
 ## 6.11.2024
 

diff --git a/validaation_muutoshistoria.md b/validaation_muutoshistoria.md
@@ -1,14 +1,14 @@
 # Koskeen tallennettavien tietojen validaatiosäännöt
 
+## 11.12.2024
+
+- Estetään siirtynytUusiinTutkinnonPerusteisiin -kentän siirto ammatillisen opiskeluoikeuden lisätietoihin, jos opiskeluoikeus ei ole päättyvässä tilassa.
+
 ## xx.11.2024
 
 - Estetään uuden kaikille opikeluoikeuksille lisätyn kentän `lähdejärjestelmäkytkentäPurettu` lisääminen tai sen sisällön muokkaaminen tiedonsiirron kautta.
 - Estetään opiskeluoikeuden päivitys tiedonsiirron kautta, jos sen lähdejärjestelmäkytkentä on purettu.
 
-## 26.11.2024
-
-- Estetään siirtynytTutkinnonUusiinPerusteisiin -kentän siirto ammatillisen opiskeluoikeuden lisätietoihin, jos opiskeluoikeus ei ole päättyvässä tilassa.
-
 ## 14.11.2024
 
 - Salli ammatillisen koulutuksen korotukset toisessa oppilaitoksessa suoritettuun tutkintoon

diff --git a/web/app/types/fi/oph/koski/schema/AmmatillisenOpiskeluoikeudenLisatiedot.ts b/web/app/types/fi/oph/koski/schema/AmmatillisenOpiskeluoikeudenLisatiedot.ts
@@ -27,8 +27,8 @@ export type AmmatillisenOpiskeluoikeudenLisätiedot = {
   majoitus?: Array<Aikajakso>
   vankilaopetuksessa?: Array<Aikajakso>
   henkilöstökoulutus: boolean
+  siirtynytUusiinTutkinnonPerusteisiin?: boolean
   erityinenTuki?: Array<Aikajakso>
-  siirtynytTutkinnonUusiinPerusteisiin?: boolean
   koulutusvienti: boolean
   opiskeluvalmiuksiaTukevatOpinnot?: Array<OpiskeluvalmiuksiaTukevienOpintojenJakso>
   jotpaAsianumero?: Koodistokoodiviite<'jotpaasianumero', string>
@@ -50,8 +50,8 @@ export const AmmatillisenOpiskeluoikeudenLisätiedot = (
     majoitus?: Array<Aikajakso>
     vankilaopetuksessa?: Array<Aikajakso>
     henkilöstökoulutus?: boolean
+    siirtynytUusiinTutkinnonPerusteisiin?: boolean
     erityinenTuki?: Array<Aikajakso>
-    siirtynytTutkinnonUusiinPerusteisiin?: boolean
     koulutusvienti?: boolean
     opiskeluvalmiuksiaTukevatOpinnot?: Array<OpiskeluvalmiuksiaTukevienOpintojenJakso>
     jotpaAsianumero?: Koodistokoodiviite<'jotpaasianumero', string>