From 50236967e58b39e52c453b1e1c4877797528833b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Odd=20Andreas=20S=C3=B8rs=C3=A6ther?=
 <oddsor@users.noreply.github.com>
Date: Tue, 22 Oct 2024 18:41:22 +0200
Subject: [PATCH] Fiks datosjekk i for godkjenning av tilskuddsperiode
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Når backenden returnerte år -99999 for kanBesluttesFom,
som den alltid gjør for første tilskuddsperiode, oppstod det
en feil i sjekken for om godkjenn-knappen skulle enables
eller ikke.

Backenden har nå blitt endret til å returnere et nytt bool-
felt "kanBehandles", som gjør at frontenden ikke lenger
behøver å gjøre "datomatematikk" for å avgjøre om knappen
skal være enabled eller ikke.
---
 package-lock.json                             | 11 ------
 .../BeslutterTilskuddsperioder.tsx            | 36 ++++++++-----------
 src/mocking/arbeidstrening-avtale-mock.ts     |  1 +
 src/mocking/context-mock.ts                   |  1 +
 .../AvtaleStatuserVeileder.stories.tsx        | 34 ++++++++++++++++++
 src/types/avtale.ts                           |  1 +
 6 files changed, 51 insertions(+), 33 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index fbf18b19..75303a57 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -49,7 +49,6 @@
                 "@storybook/test": "8.3.5",
                 "@testing-library/react": "^14.2.1",
                 "@types/amplitude-js": "^8.16.5",
-                "@types/classnames": "^2.3.0",
                 "@types/lodash.debounce": "^4.0.9",
                 "@types/lodash.isequal": "^4.5.8",
                 "@types/lodash.omit": "^4.5.9",
@@ -3196,16 +3195,6 @@
                 "@types/node": "*"
             }
         },
-        "node_modules/@types/classnames": {
-            "version": "2.3.1",
-            "resolved": "https://registry.npmjs.org/@types/classnames/-/classnames-2.3.1.tgz",
-            "integrity": "sha512-zeOWb0JGBoVmlQoznvqXbE0tEC/HONsnoUNH19Hc96NFsTAwTXbTqb8FMYkru1F/iqp7a18Ws3nWJvtA1sHD1A==",
-            "deprecated": "This is a stub types definition. classnames provides its own type definitions, so you do not need this installed.",
-            "dev": true,
-            "dependencies": {
-                "classnames": "*"
-            }
-        },
         "node_modules/@types/connect": {
             "version": "3.4.38",
             "resolved": "https://registry.npmjs.org/@types/connect/-/connect-3.4.38.tgz",
diff --git a/src/BeslutterSide/beslutterTilskuddsperioder/BeslutterTilskuddsperioder.tsx b/src/BeslutterSide/beslutterTilskuddsperioder/BeslutterTilskuddsperioder.tsx
index f0f9bf80..8ebba603 100644
--- a/src/BeslutterSide/beslutterTilskuddsperioder/BeslutterTilskuddsperioder.tsx
+++ b/src/BeslutterSide/beslutterTilskuddsperioder/BeslutterTilskuddsperioder.tsx
@@ -13,7 +13,6 @@ import { Periode, TilskuddsperiodeContext } from '@/BeslutterSide/BeslutterSide'
 import { Returårsaker, TilskuddsPeriode } from '@/types/avtale';
 import { tilskuddsperiodeReturÅrsakTekst } from '@/messages';
 import TilskuddsperiodeReturModal from '@/BeslutterSide/beslutterPanel/TilskuddsperiodeVisAvslag';
-import moment from 'moment';
 
 interface Props {
     startAnimering: () => void;
@@ -51,22 +50,17 @@ const BeslutterTilskuddsPerioder: FunctionComponent<Props> = (props) => {
 
     const hentAvslattInfoTilskuddsperiode = (periode: TilskuddsPeriode): JSX.Element => {
         return (
-            <>
-                <BodyShort size="small">
-                    Tilskuddsperioden ble returnert av
-                    <span className={cls.element('bold')}>{' ' + periode.avslåttAvNavIdent + ' '}</span> den
-                    <span className={cls.element('bold')}>
-                        {' ' + formatterDato(periode.avslåttTidspunkt ?? '', NORSK_DATO_FORMAT) + ' '}
-                    </span>
-                    med følgende årsak(er):
-                    <span className={cls.element('bold')}>
-                        {' ' + hentReturÅrsaker(periode) + ' '}
-                        {''}
-                    </span>
-                    med forklaringen:
-                    <span className={cls.element('bold')}>{' ' + periode.avslagsforklaring}</span>
-                </BodyShort>
-            </>
+            <BodyShort size="small">
+                Tilskuddsperioden ble returnert av
+                <span className={cls.element('bold')}>{' ' + periode.avslåttAvNavIdent + ' '}</span> den
+                <span className={cls.element('bold')}>
+                    {' ' + formatterDato(periode.avslåttTidspunkt ?? '', NORSK_DATO_FORMAT) + ' '}
+                </span>
+                med følgende årsak(er):
+                <span className={cls.element('bold')}>{' ' + hentReturÅrsaker(periode) + ' '}</span>
+                med forklaringen:
+                <span className={cls.element('bold')}>{' ' + periode.avslagsforklaring}</span>
+            </BodyShort>
         );
     };
 
@@ -89,13 +83,11 @@ const BeslutterTilskuddsPerioder: FunctionComponent<Props> = (props) => {
                         </tr>
                     </thead>
                     <tbody>
-                        {avtale.tilskuddPeriode.map((periode, index) => {
+                        {avtale.tilskuddPeriode.map((periode) => {
                             const gjeldende = periode.løpenummer === gjeldendeTilskuddsperiode?.løpenummer;
-                            const kanBesluttes = moment().isSameOrAfter(moment(periode.kanBesluttesFom));
                             return (
-                                <React.Fragment key={index}>
+                                <React.Fragment key={periode.id}>
                                     <tr
-                                        key={index}
                                         className={cls.element(
                                             'tilskuddsperiode-rad',
                                             settStylingForTabellrad(periode),
@@ -138,7 +130,7 @@ const BeslutterTilskuddsPerioder: FunctionComponent<Props> = (props) => {
                                                     {periode.status === 'UBEHANDLET' && (
                                                         <>
                                                             <Button
-                                                                disabled={!enhet || !kanBesluttes}
+                                                                disabled={!enhet || !periode.kanBehandles}
                                                                 onClick={() => {
                                                                     if (enhet) {
                                                                         setGodkjennModalÅpen(true);
diff --git a/src/mocking/arbeidstrening-avtale-mock.ts b/src/mocking/arbeidstrening-avtale-mock.ts
index 68b8a76d..d788fbc8 100644
--- a/src/mocking/arbeidstrening-avtale-mock.ts
+++ b/src/mocking/arbeidstrening-avtale-mock.ts
@@ -27,6 +27,7 @@ const tilskuddsPeriode: TilskuddsPeriode[] = [
         avslagsårsaker: new Set<Returårsaker>(),
         lonnstilskuddProsent: 60,
         kanBesluttesFom: '2021-01-01',
+        kanBehandles: true,
         aktiv: true,
     },
 ];
diff --git a/src/mocking/context-mock.ts b/src/mocking/context-mock.ts
index 8ae909c5..6de70c5d 100644
--- a/src/mocking/context-mock.ts
+++ b/src/mocking/context-mock.ts
@@ -46,6 +46,7 @@ const avtaleInnhold: Avtale | Avtaleinnhold = {
             avslagsårsaker: new Set<Returårsaker>(),
             lonnstilskuddProsent: 60,
             kanBesluttesFom: '2021-01-01',
+            kanBehandles: true,
             aktiv: true,
         },
     ],
diff --git a/src/stories/AvtaleStatuserVeileder.stories.tsx b/src/stories/AvtaleStatuserVeileder.stories.tsx
index 5d6d495a..8c982ded 100644
--- a/src/stories/AvtaleStatuserVeileder.stories.tsx
+++ b/src/stories/AvtaleStatuserVeileder.stories.tsx
@@ -42,6 +42,7 @@ const erUfordelt: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -66,6 +67,7 @@ const erUfordelt: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -109,6 +111,7 @@ const annullert: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -133,6 +136,7 @@ const annullert: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -176,6 +180,7 @@ const avbrutt: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -200,6 +205,7 @@ const avbrutt: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -243,6 +249,7 @@ const påbegynt: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -267,6 +274,7 @@ const påbegynt: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -310,6 +318,7 @@ const manglerGodkjenningVeilederHarGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -334,6 +343,7 @@ const manglerGodkjenningVeilederHarGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -377,6 +387,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerHarGodkjentMenIkkeMentor: Avtale =
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2024-05-10',
@@ -402,6 +413,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerHarGodkjentMenIkkeMentor: Avtale =
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -445,6 +457,7 @@ const manglerGodkjenningDeltakerOgArbeidsgiverOgMentorHarGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -470,6 +483,7 @@ const manglerGodkjenningDeltakerOgArbeidsgiverOgMentorHarGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -513,6 +527,7 @@ const manglerGodkjenningDeltakerHarGodkjentMenIkkeArbeidsgiverOgMentor: Avtale =
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -538,6 +553,7 @@ const manglerGodkjenningDeltakerHarGodkjentMenIkkeArbeidsgiverOgMentor: Avtale =
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -581,6 +597,7 @@ const manglerGodkjenningDeltakerHarGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -605,6 +622,7 @@ const manglerGodkjenningDeltakerHarGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -648,6 +666,7 @@ const manglerGodkjenningArbeidsgiverHarGodkjentMenIkkeDeltakerogMentor: Avtale =
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '',
@@ -673,6 +692,7 @@ const manglerGodkjenningArbeidsgiverHarGodkjentMenIkkeDeltakerogMentor: Avtale =
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -716,6 +736,7 @@ const manglerGodkjenningArbeidsgiverHarGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '',
@@ -740,6 +761,7 @@ const manglerGodkjenningArbeidsgiverHarGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -783,6 +805,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerOgMentorHarIkkeGodkjent: Avtale =
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '',
@@ -808,6 +831,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerOgMentorHarIkkeGodkjent: Avtale =
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -851,6 +875,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerHarIkkeGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '',
@@ -875,6 +900,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerHarIkkeGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -918,6 +944,7 @@ const klarForOppstartUteforArena: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -942,6 +969,7 @@ const klarForOppstartUteforArena: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -985,6 +1013,7 @@ const klarForOppstartArena: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -1009,6 +1038,7 @@ const klarForOppstartArena: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -1052,6 +1082,7 @@ const gjennomføres: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -1076,6 +1107,7 @@ const gjennomføres: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -1119,6 +1151,7 @@ const avsluttet: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -1143,6 +1176,7 @@ const avsluttet: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
diff --git a/src/types/avtale.ts b/src/types/avtale.ts
index dea272ba..bc689736 100644
--- a/src/types/avtale.ts
+++ b/src/types/avtale.ts
@@ -295,6 +295,7 @@ export interface TilskuddsPeriode {
     status: TilskuddPeriodeStatus;
     lonnstilskuddProsent: number;
     kanBesluttesFom: string;
+    kanBehandles: boolean;
     aktiv: boolean;
     enhet?: string;
     enhetsnavn?: string;