♻️ Rydd opp i Form Fields-typen og bruk hele typen til/fra atom'et

navelgen · trinelinderud · navelgen · commit 91991b2e1751 · 2025-03-10T13:56:41.000+01:00
Co-authored-by: Trine Linderud &lt;trine.linderud@nav.no&gt;
diff --git a/src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/arbeidsgivere/ArbeidsgiverRad.tsx b/src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/arbeidsgivere/ArbeidsgiverRad.tsx
@@ -12,7 +12,7 @@ interface ArbeidsgiverRadProps {
     arbeidsgiverNavn?: string;
     organisasjonsnummer: string;
     årsinntekt: number;
-    type?: Skjønnsfastsettingstype;
+    type: Skjønnsfastsettingstype | null;
     årligField: UseFormRegisterReturn;
     orgnummerField: UseFormRegisterReturn;
     antallArbeidsgivere: number;
diff --git a/src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/skjønnsfastsettingForm/SkjønnsfastsettingForm.tsx b/src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/skjønnsfastsettingForm/SkjønnsfastsettingForm.tsx
@@ -23,7 +23,6 @@ import { SkjønnsfastsettingType } from '@saksbilde/sykepengegrunnlag/skjønnsfa
 import { SkjønnsfastsettingÅrsak } from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/SkjønnsfastsettingÅrsak';
 import { SkjønnsfastsettingArbeidsgivere } from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/arbeidsgivere/SkjønnsfastsettingArbeidsgivere';
 import {
-    ArbeidsgiverForm,
     Skjønnsfastsettingstype,
     usePostSkjønnsfastsattSykepengegrunnlag,
 } from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/skjønnsfastsetting';
@@ -36,30 +35,6 @@ import styles from './SkjønnsfastsettingForm.module.css';
 
 const skjemaFamily = atomFamily((_skjæringstidspunkt: string) => atom<Maybe<SkjønnsfastsettingFormFields>>(null));
 
-export const useSkjønnsfastsettelseFormState = (skjæringstidspunkt: string) =>
-    useAtomValue(skjemaFamily(skjæringstidspunkt));
-
-export const useSetSkjønnsfastsettelseFormState = (skjæringstidspunkt: string) => {
-    const setState = useSetAtom(skjemaFamily(skjæringstidspunkt));
-    return (årsak: string, begrunnelseFritekst: string, type?: Skjønnsfastsettingstype) => {
-        setState((prevState) => {
-            return {
-                ...(prevState ?? defaultState),
-                type: type,
-                årsak: årsak,
-                begrunnelseFritekst: begrunnelseFritekst,
-            } as SkjønnsfastsettingFormFields;
-        });
-    };
-};
-
-const defaultState: SkjønnsfastsettingFormFields = {
-    begrunnelseFritekst: '',
-    type: undefined,
-    årsak: '',
-    arbeidsgivere: [],
-};
-
 export const useAktiveArbeidsgivere = (
     person: PersonFragment,
     period: BeregnetPeriodeFragment | GhostPeriodeFragment,
@@ -86,23 +61,15 @@ function useFormDefaults(
     aktiveArbeidsgivereInntekter: Arbeidsgiverinntekt[],
     forrigeSkjønnsfastsettelse: Sykepengegrunnlagskjonnsfastsetting | null,
 ): SkjønnsfastsettingFormFields {
-    const skjønnsfastsettelseFormState = useSkjønnsfastsettelseFormState(period.skjaeringstidspunkt);
+    const skjønnsfastsettelseFormState = useAtomValue(skjemaFamily(period.skjaeringstidspunkt));
     if (skjønnsfastsettelseFormState) {
-        return {
-            begrunnelseFritekst: skjønnsfastsettelseFormState.begrunnelseFritekst,
-            type: skjønnsfastsettelseFormState.type,
-            årsak: skjønnsfastsettelseFormState.årsak,
-            arbeidsgivere: aktiveArbeidsgivereInntekter.map((inntekt) => ({
-                organisasjonsnummer: inntekt.arbeidsgiver,
-                årlig: 0,
-            })),
-        };
+        return skjønnsfastsettelseFormState;
     } else {
         if (forrigeSkjønnsfastsettelse && !forrigeSkjønnsfastsettelse.ferdigstilt) {
             return {
                 begrunnelseFritekst: forrigeSkjønnsfastsettelse.skjonnsfastsatt.begrunnelseFritekst ?? '',
-                type: mapType(forrigeSkjønnsfastsettelse.skjonnsfastsatt.type) ?? '',
-                årsak: forrigeSkjønnsfastsettelse.skjonnsfastsatt.arsak ?? '',
+                type: mapType(forrigeSkjønnsfastsettelse.skjonnsfastsatt.type),
+                årsak: forrigeSkjønnsfastsettelse.skjonnsfastsatt.arsak,
                 arbeidsgivere: aktiveArbeidsgivereInntekter.map((inntekt) => ({
                     organisasjonsnummer: inntekt.arbeidsgiver,
                     årlig: 0,
@@ -111,7 +78,7 @@ function useFormDefaults(
         } else {
             return {
                 begrunnelseFritekst: '',
-                type: undefined,
+                type: null,
                 årsak: '',
                 arbeidsgivere: aktiveArbeidsgivereInntekter.map((inntekt) => ({
                     organisasjonsnummer: inntekt.arbeidsgiver,
@@ -135,12 +102,17 @@ const mapType = (type: Maybe<Skjonnsfastsettingstype> = Skjonnsfastsettingstype.
 };
 
 export interface SkjønnsfastsettingFormFields {
-    arbeidsgivere: ArbeidsgiverForm[];
+    arbeidsgivere: SkjønnsfastsettingFormFieldsArbeidsgiver[];
     årsak: string;
-    type?: Skjønnsfastsettingstype;
+    type: Skjønnsfastsettingstype | null;
     begrunnelseFritekst: string;
 }
 
+export interface SkjønnsfastsettingFormFieldsArbeidsgiver {
+    organisasjonsnummer: string;
+    årlig: number;
+}
+
 interface SkjønnsfastsettingFormProps {
     person: PersonFragment;
     periode: BeregnetPeriodeFragment | GhostPeriodeFragment;
@@ -177,7 +149,6 @@ export const SkjønnsfastsettingForm = ({
     );
     const erBeslutteroppgave = isBeregnetPeriode(periode) && (periode.totrinnsvurdering?.erBeslutteroppgave ?? false);
     const feiloppsummeringRef = useRef<HTMLDivElement>(null);
-    const setSkjønnsfastsettelseFormState = useSetSkjønnsfastsettelseFormState(periode.skjaeringstidspunkt);
     const avrundetSammenligningsgrunnlag = avrundetToDesimaler(sammenligningsgrunnlag);
     const cancelEditing = () => {
         setEditing(false);
@@ -203,24 +174,24 @@ export const SkjønnsfastsettingForm = ({
         control: control,
     });
 
-    const watchedValues = watch(['årsak', 'begrunnelseFritekst', 'type']);
-
-    const prevValues = useRef(watchedValues);
+    const watchedFormFields = watch();
+    const prevFormFields = useRef(watchedFormFields);
 
+    const setFormFields = useSetAtom(skjemaFamily(periode.skjaeringstidspunkt));
     useEffect(() => {
-        if (JSON.stringify(prevValues.current) !== JSON.stringify(watchedValues)) {
-            setSkjønnsfastsettelseFormState(watchedValues[0], watchedValues[1], watchedValues[2]);
-            prevValues.current = watchedValues;
+        if (JSON.stringify(prevFormFields.current) !== JSON.stringify(watchedFormFields)) {
+            setFormFields(watchedFormFields);
+            prevFormFields.current = watchedFormFields;
         }
-    }, [watchedValues, setSkjønnsfastsettelseFormState]);
+    }, [watchedFormFields, setFormFields]);
 
     const valgtMal = maler.find((it) => it.arsak === valgtÅrsak);
 
     const harFeil = !formState.isValid && formState.isSubmitted;
 
     const visFeilOppsummering = harFeil && Object.entries(formState.errors).length > 0;
     const sykepengegrunnlagEndring = watch('arbeidsgivere')?.reduce(
-        (a: number, b: ArbeidsgiverForm) => +a + +b.årlig,
+        (a: number, b: SkjønnsfastsettingFormFieldsArbeidsgiver) => +a + +b.årlig,
         0.0,
     );
 
@@ -324,7 +295,7 @@ const valgtInntekt = (
     inntekt: Arbeidsgiverinntekt,
     antallAktiveArbeidsgivere: number,
     totaltSammenligningsgrunnlag: number,
-    type?: Skjønnsfastsettingstype,
+    type: Skjønnsfastsettingstype | null,
 ): number => {
     switch (type) {
         case Skjønnsfastsettingstype.OMREGNET_ÅRSINNTEKT:
diff --git a/src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/skjønnsfastsettingForm/skjønnsfastsettingFormToDto.tsx b/src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/skjønnsfastsettingForm/skjønnsfastsettingFormToDto.tsx
@@ -1,16 +1,13 @@
 import { SkjønnsfastsettingMal } from '@external/sanity';
 import { Arbeidsgiverinntekt, Maybe, PersonFragment } from '@io/graphql';
-import {
-    ArbeidsgiverForm,
-    Skjønnsfastsettingstype,
-} from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/skjønnsfastsetting';
+import { Skjønnsfastsettingstype } from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/skjønnsfastsetting';
 import { SkjønnsfastsattSykepengegrunnlagDTO, SkjønnsfastsettingstypeDTO } from '@typer/overstyring';
 import { ActivePeriod } from '@typer/shared';
 import { toKronerOgØre } from '@utils/locale';
 import { finnFørsteVedtaksperiodeIdPåSkjæringstidspunkt } from '@utils/sykefraværstilfelle';
 import { isBeregnetPeriode } from '@utils/typeguards';
 
-import { SkjønnsfastsettingFormFields } from './SkjønnsfastsettingForm';
+import { SkjønnsfastsettingFormFields, SkjønnsfastsettingFormFieldsArbeidsgiver } from './SkjønnsfastsettingForm';
 
 interface InitierendeVedtaksperiodeForArbeidsgiver {
     arbeidsgiver: string;
@@ -58,34 +55,37 @@ export const skjønnsfastsettingFormToDto = (
         fødselsnummer: person.fodselsnummer,
         aktørId: person.aktorId,
         skjæringstidspunkt: period.skjaeringstidspunkt,
-        arbeidsgivere: form.arbeidsgivere.map(({ årlig, organisasjonsnummer }: ArbeidsgiverForm) => ({
-            organisasjonsnummer: organisasjonsnummer,
-            årlig: årlig,
-            fraÅrlig: inntekter.find((it) => it.arbeidsgiver === organisasjonsnummer)?.omregnetArsinntekt?.belop ?? 0,
-            årsak: form.årsak,
-            type:
-                form.type === undefined
-                    ? SkjønnsfastsettingstypeDTO.ANNET
-                    : form.type === Skjønnsfastsettingstype.RAPPORTERT_ÅRSINNTEKT
-                      ? SkjønnsfastsettingstypeDTO.RAPPORTERT_ÅRSINNTEKT
-                      : form.type === Skjønnsfastsettingstype.OMREGNET_ÅRSINNTEKT
-                        ? SkjønnsfastsettingstypeDTO.OMREGNET_ÅRSINNTEKT
-                        : SkjønnsfastsettingstypeDTO.ANNET,
-            begrunnelseMal: malFraSanity?.begrunnelse
-                .replace('${omregnetÅrsinntekt}', toKronerOgØre(omregnetÅrsinntekt))
-                .replace('${omregnetMånedsinntekt}', toKronerOgØre(omregnetÅrsinntekt / 12))
-                .replace('${sammenligningsgrunnlag}', toKronerOgØre(sammenligningsgrunnlag)),
-            begrunnelseFritekst: form.begrunnelseFritekst,
-            lovhjemmel: { ...malFraSanity.lovhjemmel },
-            begrunnelseKonklusjon: malFraSanity?.konklusjon.replace(
-                '${skjønnsfastsattÅrsinntekt}',
-                toKronerOgØre(skjønnsfastsatt),
-            ),
-            initierendeVedtaksperiodeId:
-                førsteVilkårsprøvdePeriodePåSkjæringstidspunkt.filter(
-                    (it) => it.arbeidsgiver === organisasjonsnummer,
-                )[0]?.initierendeVedtaksperiodeId ?? null,
-        })),
+        arbeidsgivere: form.arbeidsgivere.map(
+            ({ årlig, organisasjonsnummer }: SkjønnsfastsettingFormFieldsArbeidsgiver) => ({
+                organisasjonsnummer: organisasjonsnummer,
+                årlig: årlig,
+                fraÅrlig:
+                    inntekter.find((it) => it.arbeidsgiver === organisasjonsnummer)?.omregnetArsinntekt?.belop ?? 0,
+                årsak: form.årsak,
+                type:
+                    form.type === undefined
+                        ? SkjønnsfastsettingstypeDTO.ANNET
+                        : form.type === Skjønnsfastsettingstype.RAPPORTERT_ÅRSINNTEKT
+                          ? SkjønnsfastsettingstypeDTO.RAPPORTERT_ÅRSINNTEKT
+                          : form.type === Skjønnsfastsettingstype.OMREGNET_ÅRSINNTEKT
+                            ? SkjønnsfastsettingstypeDTO.OMREGNET_ÅRSINNTEKT
+                            : SkjønnsfastsettingstypeDTO.ANNET,
+                begrunnelseMal: malFraSanity?.begrunnelse
+                    .replace('${omregnetÅrsinntekt}', toKronerOgØre(omregnetÅrsinntekt))
+                    .replace('${omregnetMånedsinntekt}', toKronerOgØre(omregnetÅrsinntekt / 12))
+                    .replace('${sammenligningsgrunnlag}', toKronerOgØre(sammenligningsgrunnlag)),
+                begrunnelseFritekst: form.begrunnelseFritekst,
+                lovhjemmel: { ...malFraSanity.lovhjemmel },
+                begrunnelseKonklusjon: malFraSanity?.konklusjon.replace(
+                    '${skjønnsfastsattÅrsinntekt}',
+                    toKronerOgØre(skjønnsfastsatt),
+                ),
+                initierendeVedtaksperiodeId:
+                    førsteVilkårsprøvdePeriodePåSkjæringstidspunkt.filter(
+                        (it) => it.arbeidsgiver === organisasjonsnummer,
+                    )[0]?.initierendeVedtaksperiodeId ?? null,
+            }),
+        ),
         vedtaksperiodeId: finnFørsteVedtaksperiodeIdPåSkjæringstidspunkt(person.arbeidsgivere, period),
     };
 };
diff --git a/src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/skjønnsfastsetting.ts b/src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/skjønnsfastsetting.ts
@@ -25,11 +25,6 @@ export enum Skjønnsfastsettingstype {
     ANNET = 'ANNET',
 }
 
-export interface ArbeidsgiverForm {
-    organisasjonsnummer: string;
-    årlig: number;
-}
-
 export const usePostSkjønnsfastsattSykepengegrunnlag = (onFerdigKalkulert: () => void) => {
     const addToast = useAddToast();
     const removeToast = useRemoveToast();
