Merge branch 'OK-472-osaamismerkit' into OK-471-osaamismerkki-vst-tot…

…eutuksille
Opetushallitus · Oct 3, 2024 · 1cb5255 · 1cb5255
2 parents e2e9029 + a09f772
commit 1cb5255
Show file tree

Hide file tree

Showing 12 changed files with 288 additions and 100 deletions.
diff --git a/.../createKoulutus/should-be-able-to-create-vapaa-sivistystyo---osaamismerkki--koulutus.json b/.../createKoulutus/should-be-able-to-create-vapaa-sivistystyo---osaamismerkki--koulutus.json
@@ -16,7 +16,6 @@
   "julkinen": false,
   "esikatselu": true,
   "ePerusteId": null,
-  "hakutuloslistauksenKuvake": null,
   "sorakuvausId": null,
   "metadata": {
     "tyyppi": "vapaa-sivistystyo-osaamismerkki",

diff --git a/src/main/app/src/components/HakutuloslistauksenKuvakeSection.tsx b/src/main/app/src/components/HakutuloslistauksenKuvakeSection.tsx
diff --git a/src/main/app/src/components/NumberInput/index.tsx b/src/main/app/src/components/NumberInput/index.tsx
@@ -1,6 +1,6 @@
 import React from 'react';
 
-import _fp from 'lodash/fp';
+import { identity, isNaN, isNil, isNumber, noop, toString } from 'lodash';
 
 import { Input, InputProps } from '#/src/components/virkailija';
 import { parseFloatComma } from '#/src/utils';
@@ -14,25 +14,25 @@ type NumberInputProps = {
 } & InputProps;
 
 const floatToCommaStr = (value?: number | null) =>
-  _fp.isNil(value) ? '' : _fp.toString(value).replace('.', ',');
+  isNil(value) ? '' : toString(value).replace('.', ',');
 
 const NumberInput = ({
-  onBlur = _fp.noop,
-  min = 0,
+  onBlur = noop,
+  min,
   max,
   fallbackValue = null,
-  parseValue = _fp.identity,
+  parseValue = identity,
   ...props
 }: NumberInputProps) => {
   const usedOnBlur = e => {
     const value: string = e?.target?.value;
     const floatValue = parseValue(value);
-    if (_fp.isNaN(floatValue) || _fp.isNil(floatValue)) {
+    if (isNaN(floatValue) || isNil(floatValue)) {
       e.target.value = fallbackValue;
     } else {
-      if (_fp.isNumber(max) && floatValue > max) {
+      if (isNumber(max) && floatValue > max) {
         e.target.value = max;
-      } else if (_fp.isNumber(min) && floatValue < min) {
+      } else if (isNumber(min) && floatValue < min) {
         e.target.value = min;
       } else {
         e.target.value = floatToCommaStr(floatValue);

diff --git a/src/main/app/src/pages/koulutus/KoulutusForm/KoulutusForm.tsx b/src/main/app/src/pages/koulutus/KoulutusForm/KoulutusForm.tsx
@@ -7,7 +7,6 @@ import { match } from 'ts-pattern';
 import { FormButton } from '#/src/components/FormButton';
 import FormCollapse from '#/src/components/FormCollapse';
 import FormCollapseGroup from '#/src/components/FormCollapseGroup';
-import HakutuloslistauksenKuvakeSection from '#/src/components/HakutuloslistauksenKuvakeSection';
 import { JulkaisutilaField } from '#/src/components/JulkaisutilaField';
 import JulkisuusSection from '#/src/components/JulkisuusSection';
 import KieliversiotFields from '#/src/components/KieliversiotFields';
@@ -372,17 +371,6 @@ export const KoulutusForm = ({
               koulutustyyppi={koulutustyyppi}
             />
 
-            {koulutustyyppi ===
-              KOULUTUSTYYPPI.VAPAA_SIVISTYSTYO_OSAAMISMERKKI && (
-              <FormCollapse
-                section="hakutuloslistauksenKuvake"
-                header={t('koulutuslomake.hakutuloslistauksenKuvake')}
-                Component={HakutuloslistauksenKuvakeSection}
-                disabled={onlyTarjoajaRights}
-                koulutustyyppi={koulutustyyppi}
-              />
-            )}
-
             {!isNewOphKoulutus && (
               <FormCollapse
                 section="tarjoajat"

diff --git a/...tusForm/HakeutumisTaiIlmoittautumistapaSection/HakeutumisTaiIlmoittautumistapaSection.tsx b/...tusForm/HakeutumisTaiIlmoittautumistapaSection/HakeutumisTaiIlmoittautumistapaSection.tsx
@@ -124,7 +124,7 @@ export const HakeutumisTaiIlmoittautumistapaSection = ({
                 <Field
                   name={`${name}.aloituspaikat`}
                   component={FormFieldIntegerInput}
-                  min={0}
+                  fallbackValue={null}
                   label={t('toteutuslomake.aloituspaikat')}
                   type="number"
                 />

diff --git a/src/main/app/src/translations/fi.json b/src/main/app/src/translations/fi.json
@@ -520,8 +520,7 @@
     "epayhteensopiviaToteutuksia": "Koulutuksen arvot eivät ole yhteensopivia kaikkien siihen liitettyjen toteutusten kanssa",
     "haeJarjestajanNimella": "Hae järjestäjän nimellä",
     "naytaVainValitut": "Näytä vain valitut",
-    "valitseErikoistumiskoulutus": "Valitse erikoistumiskoulutus",
-    "hakutuloslistauksenKuvake": "Koulutuksen hakutuloslistauksen kuvake"
+    "valitseErikoistumiskoulutus": "Valitse erikoistumiskoulutus"
   },
   "toteutuslomake": {
     "toteutuksenTiedot": "Toteutuksen tiedot",

diff --git a/src/main/app/src/types/koulutusTypes.ts b/src/main/app/src/types/koulutusTypes.ts
@@ -52,7 +52,6 @@ export type KoulutusFormValues = {
   tarjoajat: { tarjoajat: Array<string>; kaytaPohjanJarjestajaa?: boolean };
   julkinen?: boolean;
   teemakuva?: string;
-  hakutuloslistauksenKuvake?: string;
   esikatselu?: boolean;
   osaamisala?: {
     osaamisala: SelectOption;