From 23762b334e586fecf074df8759a5f4bc080a791d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ilkka=20H=C3=A4nninen?= <ilkka.hanninen@reaktor.com>
Date: Thu, 2 Jan 2025 14:15:26 +0200
Subject: [PATCH] =?UTF-8?q?N=C3=A4yt=C3=A4=20opintojen=20rahoitus=20ib-opi?=
 =?UTF-8?q?skeluoikeusjaksoille?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../opiskeluoikeus/OpiskeluoikeudenTila.tsx   | 24 ++-------------
 .../UusiOpiskeluoikeudenTilaModal.tsx         | 16 ++++------
 web/app/ib/IBEditor.tsx                       |  4 +--
 web/app/util/opiskeluoikeusjakso.ts           | 29 +++++++++++++++++++
 4 files changed, 38 insertions(+), 35 deletions(-)
 create mode 100644 web/app/util/opiskeluoikeusjakso.ts

diff --git a/web/app/components-v2/opiskeluoikeus/OpiskeluoikeudenTila.tsx b/web/app/components-v2/opiskeluoikeus/OpiskeluoikeudenTila.tsx
index 60e706abc6..d7b1c3f061 100644
--- a/web/app/components-v2/opiskeluoikeus/OpiskeluoikeudenTila.tsx
+++ b/web/app/components-v2/opiskeluoikeus/OpiskeluoikeudenTila.tsx
@@ -43,26 +43,7 @@ import {
   UusiOpiskeluoikeudenTilaModal,
   UusiOpiskeluoikeusjakso
 } from './UusiOpiskeluoikeudenTilaModal'
-
-type RahoituksellinenOpiskeluoikeusjakso = Extract<
-  Opiskeluoikeusjakso,
-  { opintojenRahoitus?: any }
->
-function isRahoituksellinenOpiskeluoikeusjakso(
-  x: Opiskeluoikeusjakso
-): x is RahoituksellinenOpiskeluoikeusjakso {
-  return (
-    isAikuistenPerusopetuksenOpiskeluoikeusjakso(x) ||
-    isAmmatillinenOpiskeluoikeusjakso(x) ||
-    isDIAOpiskeluoikeusjakso(x) ||
-    isEuropeanSchoolOfHelsinkiOpiskeluoikeusjakso(x) ||
-    isInternationalSchoolOpiskeluoikeusjakso(x) ||
-    isLukionOpiskeluoikeusjakso(x) ||
-    isMuunKuinSäännellynKoulutuksenOpiskeluoikeudenJakso(x) ||
-    isTutkintokoulutukseenValmentavanOpiskeluoikeusjakso(x) ||
-    isVapaanSivistystyönJotpaKoulutuksenOpiskeluoikeusjakso(x)
-  )
-}
+import { isRahoituksellinenOpiskeluoikeusjakso } from '../../util/opiskeluoikeusjakso'
 
 // Opiskeluoikeuden tila viewer
 
@@ -105,8 +86,7 @@ export const OpiskeluoikeudenTilaView = <T extends OpiskeluoikeudenTila>(
                     <TestIdText id="tila">
                       <span>{t(jakso.tila.nimi)}</span>
                       <span>
-                        {isVapaanSivistystyönOpiskeluoikeusjakso(jakso) &&
-                          isRahoituksellinenOpiskeluoikeusjakso(jakso) &&
+                        {isRahoituksellinenOpiskeluoikeusjakso(jakso) &&
                           ` (${t(jakso.opintojenRahoitus?.nimi)})`}
                       </span>
                     </TestIdText>
diff --git a/web/app/components-v2/opiskeluoikeus/UusiOpiskeluoikeudenTilaModal.tsx b/web/app/components-v2/opiskeluoikeus/UusiOpiskeluoikeudenTilaModal.tsx
index f76cc57ee0..970c75d91f 100644
--- a/web/app/components-v2/opiskeluoikeus/UusiOpiskeluoikeudenTilaModal.tsx
+++ b/web/app/components-v2/opiskeluoikeus/UusiOpiskeluoikeudenTilaModal.tsx
@@ -11,6 +11,7 @@ import {
 } from '../../types/fi/oph/koski/schema/Koodistokoodiviite'
 import { Opiskeluoikeusjakso } from '../../types/fi/oph/koski/schema/Opiskeluoikeusjakso'
 import { VapaanSivistystyönJotpaKoulutuksenOpiskeluoikeusjakso } from '../../types/fi/oph/koski/schema/VapaanSivistystyonJotpaKoulutuksenOpiskeluoikeusjakso'
+import { hasProp } from '../../util/constraints'
 import { KoodiarvotOf } from '../../util/koodisto'
 import { isValmistuvaTerminaalitila } from '../../util/opiskeluoikeus'
 import { ClassOf } from '../../util/types'
@@ -24,7 +25,6 @@ import { RaisedButton } from '../controls/RaisedButton'
 import { FormField, Nothing } from '../forms/FormField'
 import { FormModel, useForm } from '../forms/FormModel'
 import { ValidationError } from '../forms/validator'
-import { TestIdLayer } from '../../appstate/useTestId'
 
 export type UusiOpiskeluoikeudenTilaModalProps<T extends Opiskeluoikeusjakso> =
   CommonProps<{
@@ -87,15 +87,6 @@ const useInitialOpiskelujaksoForm = <T extends Opiskeluoikeusjakso>(
     }
   }, [opiskeluoikeusjaksoClass])
 
-function hasRahoitus(x: string) {
-  switch (x) {
-    case VapaanSivistystyönJotpaKoulutuksenOpiskeluoikeusjakso.className:
-      return true
-    default:
-      return false
-  }
-}
-
 type P<T extends Opiskeluoikeusjakso> = {
   form: FormModel<UusiOpiskeluoikeusjakso<T>>
   opiskeluoikeusjaksoClass: ClassOf<T>
@@ -198,6 +189,9 @@ export const UusiOpiskeluoikeudenTilaModal = <T extends Opiskeluoikeusjakso>(
     }
   }, [form.state, props])
 
+  const schema = useSchema(props.opiskeluoikeusjaksoClass)
+  const hasRahoitus = hasProp(schema, 'opintojenRahoitus')
+
   return (
     <Modal
       {...common(props, ['UusiOpiskeluoikeudenTilaModal'])}
@@ -234,7 +228,7 @@ export const UusiOpiskeluoikeudenTilaModal = <T extends Opiskeluoikeusjakso>(
           />
         </Label>
 
-        {hasRahoitus(props.opiskeluoikeusjaksoClass) && (
+        {hasRahoitus && (
           <OpiskeluoikeudenTilanRahoitusField
             form={form}
             enableValmistuminen={props.enableValmistuminen}
diff --git a/web/app/ib/IBEditor.tsx b/web/app/ib/IBEditor.tsx
index 7a2fae3614..92206b9bb0 100644
--- a/web/app/ib/IBEditor.tsx
+++ b/web/app/ib/IBEditor.tsx
@@ -35,11 +35,11 @@ import { match } from '../util/patternmatch'
 import { OsasuoritusOf } from '../util/schema'
 import { useBooleanState } from '../util/useBooleanState'
 import {
-import { UusiIBTutkintoOppiaineDialog } from './dialogs/UusiIBTutkintoOppiaineDialog'
-import { UusiIBTutkintoOsasuoritusDialog } from './dialogs/UusiIBTutkintoOsasuoritusDialog'
   ibKoulutusNimi,
   IBPäätasonSuoritusTiedot
 } from './IBPaatasonSuoritusTiedot'
+import { UusiIBTutkintoOppiaineDialog } from './dialogs/UusiIBTutkintoOppiaineDialog'
+import { UusiIBTutkintoOsasuoritusDialog } from './dialogs/UusiIBTutkintoOsasuoritusDialog'
 import { UusiPreIB2015OppiaineDialog } from './dialogs/UusiPreIB2015OppiaineDialog'
 import { UusiPreIB2015OsasuoritusDialog } from './dialogs/UusiPreIB2015OsasuoritusDialog'
 import { UusiPreIB2019OppiaineDialog } from './dialogs/UusiPreIB2019OppiaineDialog'
diff --git a/web/app/util/opiskeluoikeusjakso.ts b/web/app/util/opiskeluoikeusjakso.ts
new file mode 100644
index 0000000000..dbdec9b0b0
--- /dev/null
+++ b/web/app/util/opiskeluoikeusjakso.ts
@@ -0,0 +1,29 @@
+import { isAikuistenPerusopetuksenOpiskeluoikeusjakso } from '../types/fi/oph/koski/schema/AikuistenPerusopetuksenOpiskeluoikeusjakso'
+import { isAmmatillinenOpiskeluoikeusjakso } from '../types/fi/oph/koski/schema/AmmatillinenOpiskeluoikeusjakso'
+import { isDIAOpiskeluoikeusjakso } from '../types/fi/oph/koski/schema/DIAOpiskeluoikeusjakso'
+import { isEuropeanSchoolOfHelsinkiOpiskeluoikeusjakso } from '../types/fi/oph/koski/schema/EuropeanSchoolOfHelsinkiOpiskeluoikeusjakso'
+import { isInternationalSchoolOpiskeluoikeusjakso } from '../types/fi/oph/koski/schema/InternationalSchoolOpiskeluoikeusjakso'
+import { isLukionOpiskeluoikeusjakso } from '../types/fi/oph/koski/schema/LukionOpiskeluoikeusjakso'
+import { isMuunKuinSäännellynKoulutuksenOpiskeluoikeudenJakso } from '../types/fi/oph/koski/schema/MuunKuinSaannellynKoulutuksenOpiskeluoikeudenJakso'
+import { Opiskeluoikeusjakso } from '../types/fi/oph/koski/schema/Opiskeluoikeusjakso'
+import { isTutkintokoulutukseenValmentavanOpiskeluoikeusjakso } from '../types/fi/oph/koski/schema/TutkintokoulutukseenValmentavanOpiskeluoikeusjakso'
+import { isVapaanSivistystyönJotpaKoulutuksenOpiskeluoikeusjakso } from '../types/fi/oph/koski/schema/VapaanSivistystyonJotpaKoulutuksenOpiskeluoikeusjakso'
+
+export type RahoituksellinenOpiskeluoikeusjakso = Extract<
+  Opiskeluoikeusjakso,
+  { opintojenRahoitus?: any }
+>
+
+export const isRahoituksellinenOpiskeluoikeusjakso = (
+  x: Opiskeluoikeusjakso
+): x is RahoituksellinenOpiskeluoikeusjakso =>
+  isAikuistenPerusopetuksenOpiskeluoikeusjakso(x) ||
+  isAmmatillinenOpiskeluoikeusjakso(x) ||
+  isDIAOpiskeluoikeusjakso(x) ||
+  isEuropeanSchoolOfHelsinkiOpiskeluoikeusjakso(x) ||
+  isInternationalSchoolOpiskeluoikeusjakso(x) ||
+  isLukionOpiskeluoikeusjakso(x) ||
+  isMuunKuinSäännellynKoulutuksenOpiskeluoikeudenJakso(x) ||
+  isTutkintokoulutukseenValmentavanOpiskeluoikeusjakso(x) ||
+  isVapaanSivistystyönJotpaKoulutuksenOpiskeluoikeusjakso(x) ||
+  isLukionOpiskeluoikeusjakso(x)