Merge pull request #2785 from Opetushallitus/tor-2101-eb-aloitus-paiv…

…amaaran-vaihto-estaa-opiskeluoikeuden-luomisen TOR-2101: EB-tutkinnon luonti epäonnistuu käyttöliittymässä, jos päivämäärää vaihtaa
Opetushallitus · Nov 24, 2023 · 52e9522 · 52e9522
2 parents 4c322f0 + c2bce56
commit 52e9522
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 16 deletions.
diff --git a/web/app/esh/europeanschoolofhelsinkiSuoritus.js b/web/app/esh/europeanschoolofhelsinkiSuoritus.js
@@ -19,7 +19,11 @@ export const makeSuoritus = (
   }
 }
 
-export const makeEBSuoritus = (oppilaitos, curriculum) => {
+export const makeEBSuoritus = (
+  oppilaitos,
+  date, // date mukana vain bacon/React-yhteistoiminnan "korjaamiseksi": päivämäärän vaihto dialogilla sotkee muuten suoritus-atomin väärän tyyppiseksi
+  curriculum
+) => {
   if (!oppilaitos || !curriculum) return null
 
   return {

diff --git a/web/app/uusioppija/UusiEBTutkinnonSuoritus.jsx b/web/app/uusioppija/UusiEBTutkinnonSuoritus.jsx
@@ -2,16 +2,14 @@ import React from 'baret'
 import Bacon from 'baconjs'
 import Atom from 'bacon.atom'
 import { koodistoValues } from './koodisto'
-import {
-  makeEBSuoritus,
-  makeSuoritus
-} from '../esh/europeanschoolofhelsinkiSuoritus'
+import { makeEBSuoritus } from '../esh/europeanschoolofhelsinkiSuoritus'
 import KoodistoDropdown from '../koodisto/KoodistoDropdown'
-import Text from '../i18n/Text'
-import DateInput from '../date/DateInput'
-import { ift } from '../util/util'
 
-export default ({ suoritusAtom, oppilaitosAtom }) => {
+export default ({
+  suoritusAtom,
+  dateAtom, // dateAtom mukana vain bacon/React-yhteistoiminnan "korjaamiseksi": päivämäärän vaihto dialogilla sotkee muuten suoritus-atomin väärän tyyppiseksi
+  oppilaitosAtom
+}) => {
   // EB-opiskeluoikeuden suorituksen Curriculum
   const curriculumAtom = Atom()
   const curriculumP = koodistoValues('europeanschoolofhelsinkicurriculum').map(
@@ -21,11 +19,14 @@ export default ({ suoritusAtom, oppilaitosAtom }) => {
     curriculumAtom.set(curriculums[0])
   })
 
-  Bacon.combineWith(oppilaitosAtom, curriculumAtom, makeEBSuoritus).onValue(
-    (suoritus) => {
-      suoritusAtom.set(suoritus)
-    }
-  )
+  Bacon.combineWith(
+    oppilaitosAtom,
+    dateAtom, // dateAtom mukana vain bacon/React-yhteistoiminnan "korjaamiseksi": päivämäärän vaihto dialogilla sotkee muuten suoritus-atomin väärän tyyppiseksi
+    curriculumAtom,
+    makeEBSuoritus
+  ).onValue((suoritus) => {
+    suoritusAtom.set(suoritus)
+  })
 
   return (
     <>

diff --git a/web/app/uusioppija/UusiOpiskeluoikeus.jsx b/web/app/uusioppija/UusiOpiskeluoikeus.jsx
@@ -233,7 +233,8 @@ export default ({ opiskeluoikeusAtom }) => {
             tyyppi &&
             tyyppi.koodiarvo !== 'internationalschool' &&
             tyyppi.koodiarvo !== 'europeanschoolofhelsinki' &&
-            tyyppi.koodiarvo !== 'taiteenperusopetus'
+            tyyppi.koodiarvo !== 'taiteenperusopetus' &&
+            tyyppi.koodiarvo !== 'ebtutkinto'
         ),
         <Suorituskieli
           suorituskieliAtom={suorituskieliAtom}
@@ -359,6 +360,7 @@ export default ({ opiskeluoikeusAtom }) => {
           return (
             <UusiEBTutkinnonSuoritus
               suoritusAtom={suoritusAtom}
+              dateAtom={dateAtom}
               oppilaitosAtom={oppilaitosAtom}
             />
           )

diff --git a/web/test/spec/ebTutkintoSpec.js b/web/test/spec/ebTutkintoSpec.js
@@ -32,7 +32,14 @@ describe('EB-tutkinto', function () {
         ),
         opinnot.opiskeluoikeudet.lisääOpiskeluoikeus,
         addOppija.selectOppilaitos('Helsingin eurooppalainen koulu'),
-        addOppija.selectOpiskeluoikeudenTyyppi('EB-tutkinto')
+        addOppija.selectOpiskeluoikeudenTyyppi('EB-tutkinto'),
+        addOppija.selectAloituspäivä(
+          new Date().getDate() +
+            '.' +
+            (1 + new Date().getMonth()) +
+            '.' +
+            (new Date().getFullYear() - 1)
+        )
       )
 
       it('Lisää-nappi on enabloitu', function () {