Skip to content

Commit

Permalink
Korjaa EB-tutkinnon luonti vaihdetulla päivämäärällä;
Browse files Browse the repository at this point in the history
Bacon/React-yhteistoiminta aiheuttaa ilman turhaa dateAtom:in käyttöä oudon bugin, jossa käytetään ESH-haaran luomaa koodia, vaikka onkin vaihdettu EB-haaraan jo aiemmin.
  • Loading branch information
AleksiAhtiainen committed Nov 24, 2023
1 parent ae52e16 commit c2bce56
Show file tree
Hide file tree
Showing 4 changed files with 28 additions and 15 deletions.
6 changes: 5 additions & 1 deletion web/app/esh/europeanschoolofhelsinkiSuoritus.js
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down
27 changes: 14 additions & 13 deletions web/app/uusioppija/UusiEBTutkinnonSuoritus.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand All @@ -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 (
<>
Expand Down
1 change: 1 addition & 0 deletions web/app/uusioppija/UusiOpiskeluoikeus.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -360,6 +360,7 @@ export default ({ opiskeluoikeusAtom }) => {
return (
<UusiEBTutkinnonSuoritus
suoritusAtom={suoritusAtom}
dateAtom={dateAtom}
oppilaitosAtom={oppilaitosAtom}
/>
)
Expand Down
9 changes: 8 additions & 1 deletion web/test/spec/ebTutkintoSpec.js
Original file line number Diff line number Diff line change
Expand Up @@ -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 () {
Expand Down

0 comments on commit c2bce56

Please sign in to comment.