Merge pull request #1077 from navikt/fiksVirksomhet

Endret næringstyper (Array) til næringstype (ikke array)
navikt · Aug 21, 2023 · bb75f02 · bb75f02
2 parents 6669d48 + 68bf313
commit bb75f02
Show file tree

Hide file tree

Showing 8 changed files with 22 additions and 22 deletions.
diff --git a/.changeset/fifty-rabbits-juggle.md b/.changeset/fifty-rabbits-juggle.md
@@ -0,0 +1,5 @@
+---
+'@navikt/sif-common-forms-ds': minor
+---
+
+Endret næringstyper (Array) til næringstype (ikke array) i Virksomhet
diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetInfo.tsx b/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetInfo.tsx
@@ -9,8 +9,8 @@ import JaNeiSvar from '../../components/summary/JaNeiSvar';
 import Sitat from '../../components/summary/Sitat';
 import SummaryBlock from '../../components/summary/SummaryBlock';
 import TallSvar from '../../components/summary/TallSvar';
-import { VirksomhetApiData } from './types';
-import { erVirksomhetRegnetSomNyoppstartet, harFiskerNæringstype } from './virksomhetUtils';
+import { Næringstype, VirksomhetApiData } from './types';
+import { erVirksomhetRegnetSomNyoppstartet } from './virksomhetUtils';
 import ExpandableInfo from '@navikt/sif-common-core-ds/lib/components/expandable-info/ExpandableInfo';
 import Block from '@navikt/sif-common-core-ds/lib/atoms/block/Block';
 
@@ -20,11 +20,10 @@ interface Props {
 
 const renderVirksomhetSummary = (virksomhet: VirksomhetApiData, intl: IntlShape) => {
     const land = virksomhet.registrertIUtlandet ? virksomhet.registrertIUtlandet.landnavn : 'Norge';
-    const næringstyper = virksomhet.næringstyper
-        .map((næring) => intlHelper(intl, `sifForms.virksomhet.næringstype_${næring}`))
-        .join(', ');
+    const næringstype = intlHelper(intl, `sifForms.virksomhet.næringstype_${virksomhet.næringstype}`);
+
     const fiskerinfo =
-        harFiskerNæringstype(virksomhet.næringstyper) && virksomhet.fiskerErPåBladB !== undefined
+        virksomhet.næringstype === Næringstype.FISKE && virksomhet.fiskerErPåBladB !== undefined
             ? {
                   erPåBladB: virksomhet.fiskerErPåBladB !== undefined && virksomhet.fiskerErPåBladB === true,
               }
@@ -41,7 +40,7 @@ const renderVirksomhetSummary = (virksomhet: VirksomhetApiData, intl: IntlShape)
 
     return (
         <SummaryBlock header={virksomhet.navnPåVirksomheten} margin="none">
-            <IntlLabelValue labelKey="sifForms.virksomhet.summary.næringstype">{næringstyper}. </IntlLabelValue>
+            <IntlLabelValue labelKey="sifForms.virksomhet.summary.næringstype">{næringstype}. </IntlLabelValue>
             {fiskerinfo && (
                 <div>
                     {fiskerinfo.erPåBladB === false ? (

diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetSummary.tsx b/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetSummary.tsx
@@ -28,13 +28,10 @@ const getFiskerNæringTekst = (intl: IntlShape, erPåBladB: boolean) => {
 export const renderVirksomhetSummary = (virksomhet: VirksomhetApiData, intl: IntlShape) => {
     const land = virksomhet.registrertIUtlandet ? virksomhet.registrertIUtlandet.landnavn : 'Norge';
 
-    const næringstyper = virksomhet.næringstyper
-        .map((næring) =>
-            næring === Næringstype.FISKE && virksomhet.fiskerErPåBladB !== undefined
-                ? getFiskerNæringTekst(intl, virksomhet.fiskerErPåBladB)
-                : intlHelper(intl, `sifForms.virksomhet.næringstype_${næring}`)
-        )
-        .join(', ');
+    const næringstype =
+        virksomhet.næringstype === Næringstype.FISKE && virksomhet.fiskerErPåBladB !== undefined
+            ? getFiskerNæringTekst(intl, virksomhet.fiskerErPåBladB)
+            : intlHelper(intl, `sifForms.virksomhet.næringstype_${virksomhet.næringstype}`);
 
     const tidsinfo = virksomhet.tilOgMed
         ? intlHelper(intl, 'sifForms.virksomhet.summary.tidsinfo.avsluttet', {
@@ -50,7 +47,7 @@ export const renderVirksomhetSummary = (virksomhet: VirksomhetApiData, intl: Int
             <IntlLabelValue labelKey="sifForms.virksomhet.summary.navn">
                 {virksomhet.navnPåVirksomheten}.
             </IntlLabelValue>
-            <IntlLabelValue labelKey="sifForms.virksomhet.summary.næringstype">{næringstyper}. </IntlLabelValue>
+            <IntlLabelValue labelKey="sifForms.virksomhet.summary.næringstype">{næringstype}. </IntlLabelValue>
             <div>
                 <FormattedMessage id="sifForms.virksomhet.summary.registrertILand" values={{ land }} />
                 {virksomhet.registrertINorge && (

diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/__tests__/mapVirksomhetToApiData.test.ts b/packages/sif-common-forms-ds/src/forms/virksomhet/__tests__/mapVirksomhetToApiData.test.ts
@@ -26,7 +26,7 @@ const virksomhetFormData: Virksomhet = {
 };
 
 const virksomhetApiData: VirksomhetApiData = {
-    næringstyper: [Næringstype.ANNEN],
+    næringstype: Næringstype.ANNEN,
     navnPåVirksomheten: 'ABC',
     fraOgMed: dateToISODate(fom),
     tilOgMed: null,

diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/mapVirksomhetToApiData.ts b/packages/sif-common-forms-ds/src/forms/virksomhet/mapVirksomhetToApiData.ts
@@ -14,7 +14,7 @@ export const mapVirksomhetToVirksomhetApiData = (
     const erNyoppstartet = erVirksomhetRegnetSomNyoppstartet(virksomhet.fom);
 
     const data: VirksomhetApiData = {
-        næringstyper: [virksomhet.næringstype],
+        næringstype: virksomhet.næringstype,
         navnPåVirksomheten: virksomhet.navnPåVirksomheten,
         registrertINorge,
         ...(registrertINorge

diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/types.ts b/packages/sif-common-forms-ds/src/forms/virksomhet/types.ts
@@ -78,7 +78,7 @@ export const isVirksomhet = (virksomhet: Partial<Virksomhet>): virksomhet is Vir
 };
 
 export interface VirksomhetApiData {
-    næringstyper: Næringstype[];
+    næringstype: Næringstype;
     fiskerErPåBladB?: boolean;
     fraOgMed: ISODate;
     tilOgMed?: ISODate | null;

diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/virksomhetUtils.ts b/packages/sif-common-forms-ds/src/forms/virksomhet/virksomhetUtils.ts
@@ -6,8 +6,7 @@ import dayjs from 'dayjs';
 import { guid } from '@navikt/sif-common-utils';
 import { Næringstype, Virksomhet, VirksomhetFormValues } from './types';
 
-export const harFiskerNæringstype = (næringstyper: Næringstype[]): boolean =>
-    næringstyper.find((n) => n === Næringstype.FISKE) !== undefined;
+export const harFiskerNæringstype = (næringstype: Næringstype): boolean => næringstype === Næringstype.FISKE;
 
 export const erFiskerNæringstype = (næringstype?: Næringstype): boolean =>
     næringstype ? næringstype === Næringstype.FISKE : false;

diff --git a/packages/sif-common-forms-ds/storybook/stories/virksomhet/VirksomhetExample.tsx b/packages/sif-common-forms-ds/storybook/stories/virksomhet/VirksomhetExample.tsx
@@ -4,7 +4,7 @@ import { useState } from 'react';
 import { useIntl } from 'react-intl';
 import Block from '@navikt/sif-common-core-ds/lib/atoms/block/Block';
 import { TypedFormikForm, TypedFormikWrapper, YesOrNo } from '@navikt/sif-common-formik-ds/lib';
-import { getListValidator } from '@navikt/sif-common-formik-ds/lib/validation';
+import { getRequiredFieldValidator } from '@navikt/sif-common-formik-ds/lib/validation';
 import getFormErrorHandler from '@navikt/sif-common-formik-ds/lib/validation/intlFormErrorHandler';
 import { ValidationError } from '@navikt/sif-common-formik-ds/lib/validation/types';
 import flat from 'flat';
@@ -77,7 +77,7 @@ const VirksomhetExample = () => {
                                 <VirksomhetInfoAndDialog<FormField>
                                     name={FormField.virksomhet}
                                     harFlereVirksomheter={harFlereVirksomheter}
-                                    validate={getListValidator({ required: true })}
+                                    validate={getRequiredFieldValidator()}
                                     labels={{
                                         addLabel: harFlereVirksomheter ? 'Registrer virksomhet' : 'Legg til',
                                         deleteLabel: 'Fjern',