Skip to content

Commit 5d27cd8

Browse files
♻️ Bruker useAtom-funksjon i stedet for useSetAtom og useAtomValue
Co-authored-by: Svein Elgstøen <[email protected]>
1 parent cc6c2a0 commit 5d27cd8

File tree

2 files changed

+12
-17
lines changed

2 files changed

+12
-17
lines changed

src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/atoms.ts

+1-9
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { atom, useAtom, useAtomValue, useSetAtom } from 'jotai/index';
1+
import { atom, useAtom } from 'jotai/index';
22
import { atomFamily } from 'jotai/utils';
33

44
import { Maybe } from '@io/graphql';
@@ -12,14 +12,6 @@ export const useAtomSkjemaForPersonOgSkjæringstidspunkt = (skjaeringstidspunkt:
1212
store: usePersonStore(),
1313
});
1414

15-
export const useAtomValueSkjemaForPersonOgSkjæringstidspunkt = (skjaeringstidspunkt: string) =>
16-
useAtomValue(skjemaFamily(skjaeringstidspunkt), {
17-
store: usePersonStore(),
18-
});
19-
20-
export const useSetAtomSkjemaForPersonOgSkjæringstidspunkt = (skjaeringstidspunkt: string) =>
21-
useSetAtom(skjemaFamily(skjaeringstidspunkt), { store: usePersonStore() });
22-
2315
const editingFamily = atomFamily((_skjæringstidspunkt: string) => atom<boolean>(false));
2416

2517
export const useAtomEditingForPersonOgSkjæringstidspunkt = (skjaeringstidspunkt: string) =>

src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/skjønnsfastsettingForm/SkjønnsfastsettingForm.tsx

+11-8
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,7 @@ import {
1616
Sykepengegrunnlagsgrense,
1717
Sykepengegrunnlagskjonnsfastsetting,
1818
} from '@io/graphql';
19-
import {
20-
useAtomValueSkjemaForPersonOgSkjæringstidspunkt,
21-
useSetAtomSkjemaForPersonOgSkjæringstidspunkt,
22-
} from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/atoms';
19+
import { useAtomSkjemaForPersonOgSkjæringstidspunkt } from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/atoms';
2320
import { SkjønnsfastsettingBegrunnelse } from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/SkjønnsfastsettingBegrunnelse';
2421
import { SkjønnsfastsettingType } from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/SkjønnsfastsettingType';
2522
import { SkjønnsfastsettingÅrsak } from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/SkjønnsfastsettingÅrsak';
@@ -57,11 +54,10 @@ export const useAktiveArbeidsgivere = (
5754
);
5855

5956
function useFormDefaults(
60-
period: BeregnetPeriodeFragment | GhostPeriodeFragment,
57+
skjønnsfastsettelseFormState: Maybe<SkjønnsfastsettingFormFields>,
6158
aktiveArbeidsgivereInntekter: Arbeidsgiverinntekt[],
6259
forrigeSkjønnsfastsettelse: Sykepengegrunnlagskjonnsfastsetting | null,
6360
): SkjønnsfastsettingFormFields {
64-
const skjønnsfastsettelseFormState = useAtomValueSkjemaForPersonOgSkjæringstidspunkt(period.skjaeringstidspunkt);
6561
if (skjønnsfastsettelseFormState) {
6662
return skjønnsfastsettelseFormState;
6763
} else {
@@ -156,10 +152,18 @@ export const SkjønnsfastsettingForm = ({
156152
const { isLoading, error, postSkjønnsfastsetting, timedOut, setTimedOut } =
157153
usePostSkjønnsfastsattSykepengegrunnlag(cancelEditing);
158154

155+
const [skjønnsfastsettelseFormState, setFormFields] = useAtomSkjemaForPersonOgSkjæringstidspunkt(
156+
periode.skjaeringstidspunkt,
157+
);
158+
159159
const form = useForm<SkjønnsfastsettingFormFields>({
160160
shouldFocusError: false,
161161
mode: 'onBlur',
162-
defaultValues: useFormDefaults(periode, aktiveArbeidsgivereInntekter, sisteSkjønnsfastsettelse),
162+
defaultValues: useFormDefaults(
163+
skjønnsfastsettelseFormState,
164+
aktiveArbeidsgivereInntekter,
165+
sisteSkjønnsfastsettelse,
166+
),
163167
});
164168

165169
const { control, formState, setValue, getValues, handleSubmit, watch } = form;
@@ -177,7 +181,6 @@ export const SkjønnsfastsettingForm = ({
177181
const watchedFormFields = watch();
178182
const prevFormFields = useRef(watchedFormFields);
179183

180-
const setFormFields = useSetAtomSkjemaForPersonOgSkjæringstidspunkt(periode.skjaeringstidspunkt);
181184
useEffect(() => {
182185
if (JSON.stringify(prevFormFields.current) !== JSON.stringify(watchedFormFields)) {
183186
setFormFields(watchedFormFields);

0 commit comments

Comments
 (0)