fix. fikset feil ved annullering av avtaler som er feilregistrerte (#1291)

Author: eirikv

src/AvtaleSide/steg/GodkjenningSteg/GodkjenningSteg.tsx

@@ -57,10 +57,7 @@ const GodkjenningSteg: React.FunctionComponent<Props> = (props) => {
                     <TilskuddsPerioderOppsummering />
                 </Innholdsboks>
             )}
-
-            <Suspense fallback={null}>
-                <VersjoneringKomponent avtale={avtale} rolle={innloggetBruker.rolle} />
-            </Suspense>
+            <VersjoneringKomponent avtale={avtale} />
         </div>
     );
 };

src/AvtaleSide/steg/GodkjenningSteg/Versjonering/TidligereVersjoner.tsx

@@ -1,4 +1,4 @@
-import React, { useState } from 'react';
+import React, { useMemo, useState } from 'react';
 import VersjonModal from '@/komponenter/modal/VersjonModal';
 import { innholdTypeTekst } from '@/messages';
 import { AvtaleVersjon, TiltaksType } from '@/types/avtale';
@@ -9,57 +9,59 @@ import { formaterDato, NORSK_DATO_FORMAT } from '@/utils/datoUtils';
 
 const cls = BEMHelper('tidligereVersjoner');
 
-const TidligereVersjoner: React.FunctionComponent<{ versjoner: AvtaleVersjon[]; tiltakstype: TiltaksType }> = (
-    props,
-) => {
+interface Props {
+    versjoner: AvtaleVersjon[];
+    tiltakstype: TiltaksType;
+}
+
+const TidligereVersjoner = (props: Props) => {
+    const { versjoner } = props;
+
     const [isOpen, setOpen] = useState<boolean>(false);
     const [currentVersjon, setCurrentVersjon] = useState<number>(0);
 
-    const sorterteVersjoner = Array.from(props.versjoner);
-    sorterteVersjoner.sort((a, b) => a.versjon - b.versjon);
-    const versjonLenker = sorterteVersjoner.map((avtaleVersjon: AvtaleVersjon, index: number) => {
-        return (
-            <LinkPanel
-                className={cls.className}
-                key={index}
-                href={'#'}
-                onClick={() => {
-                    setCurrentVersjon(avtaleVersjon.versjon);
-                    setOpen(true);
-                }}
-                border={true}
-            >
-                <LinkPanel.Title className={cls.element('linke-container')}>
-                    <div className={cls.element('rad')}>
-                        <Label>
-                            Versjon {avtaleVersjon.versjon}
-                            {avtaleVersjon.innholdType && <> - {innholdTypeTekst(avtaleVersjon)}</>}
-                        </Label>
-                        <BodyShort size="small">
-                            {avtaleVersjon.ikrafttredelsestidspunkt &&
-                                formaterDato(avtaleVersjon.ikrafttredelsestidspunkt, NORSK_DATO_FORMAT)}
-                        </BodyShort>
-                    </div>
-                </LinkPanel.Title>
-            </LinkPanel>
-        );
-    });
+    const sorterteVersjoner = useMemo(() => Array.from(versjoner).sort((a, b) => a.versjon - b.versjon), [versjoner]);
+
     return (
         <>
-            {versjonLenker.length > 0 && (
-                <>
-                    <Heading size="small" className={cls.element('alle_versjoner_tittel')}>
-                        Alle versjoner av avtalen
-                    </Heading>
-                    <div>{versjonLenker}</div>
-                    <VersjonModal
-                        isOpen={isOpen}
-                        lukkModal={() => setOpen(false)}
-                        avtaleInnhold={sorterteVersjoner[currentVersjon > 0 ? currentVersjon - 1 : 0]}
-                        tiltakstype={props.tiltakstype}
-                    />
-                </>
-            )}
+            <Heading size="small" className={cls.element('alle_versjoner_tittel')}>
+                Alle versjoner av avtalen
+            </Heading>
+            <div>
+                {sorterteVersjoner.map((avtaleVersjon: AvtaleVersjon, index: number) => {
+                    return (
+                        <LinkPanel
+                            className={cls.className}
+                            key={index}
+                            href={'#'}
+                            onClick={() => {
+                                setCurrentVersjon(avtaleVersjon.versjon);
+                                setOpen(true);
+                            }}
+                            border={true}
+                        >
+                            <LinkPanel.Title className={cls.element('linke-container')}>
+                                <div className={cls.element('rad')}>
+                                    <Label>
+                                        Versjon {avtaleVersjon.versjon}
+                                        {avtaleVersjon.innholdType && <> - {innholdTypeTekst(avtaleVersjon)}</>}
+                                    </Label>
+                                    <BodyShort size="small">
+                                        {avtaleVersjon.ikrafttredelsestidspunkt &&
+                                            formaterDato(avtaleVersjon.ikrafttredelsestidspunkt, NORSK_DATO_FORMAT)}
+                                    </BodyShort>
+                                </div>
+                            </LinkPanel.Title>
+                        </LinkPanel>
+                    );
+                })}
+            </div>
+            <VersjonModal
+                isOpen={isOpen}
+                lukkModal={() => setOpen(false)}
+                avtaleInnhold={sorterteVersjoner[currentVersjon > 0 ? currentVersjon - 1 : 0]}
+                tiltakstype={props.tiltakstype}
+            />
         </>
     );
 };

src/AvtaleSide/steg/GodkjenningSteg/Versjonering/VersjoneringKomponent.tsx

@@ -1,28 +1,26 @@
 import TidligereVersjoner from '@/AvtaleSide/steg/GodkjenningSteg/Versjonering/TidligereVersjoner';
 import Innholdsboks from '@/komponenter/Innholdsboks/Innholdsboks';
 import { Avtale } from '@/types/avtale';
-import { Rolle } from '@/types/innlogget-bruker';
 import React from 'react';
 import MediaQuery from 'react-responsive';
 import { useHentVersjoner } from '@/services/use-rest';
 
 interface Props {
-    rolle: Rolle;
     avtale: Avtale;
 }
 
-const VersjoneringKomponent: React.FunctionComponent<Props> = (props) => {
-    const versjoner = useHentVersjoner(props.avtale.id);
+const VersjoneringKomponent = (props: Props) => {
+    const { avtale } = props;
+    const versjoner = useHentVersjoner(avtale);
 
-    const finnesKunEnVersjon = versjoner.length === 1;
-    if (finnesKunEnVersjon) {
+    if (versjoner.length <= 1) {
         return null;
     }
 
     return (
         <MediaQuery print={false}>
             <Innholdsboks>
-                <TidligereVersjoner versjoner={versjoner} tiltakstype={props.avtale.tiltakstype} />
+                <TidligereVersjoner versjoner={versjoner} tiltakstype={avtale.tiltakstype} />
             </Innholdsboks>
         </MediaQuery>
     );

src/BeslutterSide/BeslutterSide.tsx

@@ -104,9 +104,7 @@ const BeslutterSide: FunctionComponent = () => {
                     </div>
                     {visVersjon && (
                         <div className={cls.element('avtale-versjon-wrapper')}>
-                            <Suspense fallback={null}>
-                                <VersjoneringKomponent avtale={avtale} rolle={innloggetBruker.rolle} />
-                            </Suspense>
+                            <VersjoneringKomponent avtale={avtale} />
                         </div>
                     )}
                 </div>

src/mocking/arbeidstrening-avtale-mock.ts

@@ -102,6 +102,7 @@ const arbeidstreningAvtaleMock: Avtale = {
     erAvtaleInngått: false,
     erRyddeAvtale: false,
     opphav: 'VEILEDER',
+    feilregistrert: false,
 };
 
 export default arbeidstreningAvtaleMock;

src/mocking/lonnstilskudd-avtale-mock.ts

@@ -84,6 +84,7 @@ const lonnstilskuddAvtaleMock: Avtale = {
     erAvtaleInngått: false,
     erRyddeAvtale: false,
     opphav: 'VEILEDER',
+    feilregistrert: false,
 };
 
 export default lonnstilskuddAvtaleMock;

src/services/use-rest.ts

@@ -2,8 +2,8 @@ import axios from 'axios';
 import axiosRetry from 'axios-retry';
 import useSWR from 'swr';
 
-import { ApiError, AutentiseringError, FeilkodeError, IkkeTilgangError } from '@/types/errors';
-import { AvtaleVersjon } from '@/types/avtale';
+import { ApiError, AutentiseringError, FeilkodeError, IkkeFunnetError, IkkeTilgangError } from '@/types/errors';
+import { Avtale, AvtaleVersjon } from '@/types/avtale';
 import { Enhet } from '@/types/enhet';
 import { SIDE_FOER_INNLOGGING } from '@/RedirectEtterLogin';
 import { basename } from '@/Router';
@@ -30,6 +30,9 @@ api.interceptors.response.use(
         if (error.response?.status === 400 && error.response?.headers.feilkode) {
             throw new FeilkodeError(error.response?.headers.feilkode);
         }
+        if (error.response?.status === 404) {
+            throw new IkkeFunnetError('Fant ikke ressursen.');
+        }
         throw new ApiError('Feil ved kontakt mot baksystem.');
     },
 );
@@ -43,9 +46,12 @@ const swrConfig = {
     suspense: true,
 };
 
-export const useHentVersjoner = (avtaleId: string) => {
-    const { data } = useSWR<AvtaleVersjon[]>(`/avtaler/${avtaleId}/versjoner`, swrConfig);
-    return data!; // nosonar
+export const useHentVersjoner = (avtale: Avtale) => {
+    const { data } = useSWR<AvtaleVersjon[]>(
+        !avtale.feilregistrert ? `/avtaler/${avtale.id}/versjoner` : null,
+        swrConfig,
+    );
+    return data || [];
 };
 
 export const useHentEnhet = (enhetsnummer?: string) => {

src/types/avtale.ts

@@ -194,6 +194,7 @@ export interface AvtaleMetadata {
     opphav: Avtaleopphav;
     kreverOppfolgingFom?: string;
     kreverOppfolgingFrist?: string;
+    feilregistrert: boolean;
 }
 
 export interface RefusjonKontaktperson {