From 56d1a0278613b5d8bad0431b8943d06833547fd3 Mon Sep 17 00:00:00 2001
From: Nicolas N <Watercolours@users.noreply.github.com>
Date: Fri, 19 Apr 2024 10:31:14 +0200
Subject: [PATCH] Gi saksbehandler valg ved ferdigstilt oppgave (#4562)

---
 .../ferdigstilloppgave/FerdigstillOppgave.tsx |  9 +++--
 .../FerdigstillOppgaveModal.tsx               | 29 ++++++++++-----
 .../nybehandling/FullfoerOppgaveModal.tsx     | 37 +++++++++++--------
 .../nybehandling/OpprettNyBehandling.tsx      |  3 +-
 .../OppsummeringOppgavebehandling.tsx         | 15 ++++----
 5 files changed, 55 insertions(+), 38 deletions(-)

diff --git a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/ferdigstilloppgave/FerdigstillOppgave.tsx b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/ferdigstilloppgave/FerdigstillOppgave.tsx
index 5a63f93e0c9..d8d37edb61a 100644
--- a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/ferdigstilloppgave/FerdigstillOppgave.tsx
+++ b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/ferdigstilloppgave/FerdigstillOppgave.tsx
@@ -2,12 +2,14 @@ import { Alert, BodyLong, Button, Heading, Link } from '@navikt/ds-react'
 import { useJournalfoeringOppgave } from '~components/person/journalfoeringsoppgave/useJournalfoeringOppgave'
 import { FlexRow } from '~shared/styled'
 import AvbrytBehandleJournalfoeringOppgave from '~components/person/journalfoeringsoppgave/AvbrytBehandleJournalfoeringOppgave'
-import { useNavigate } from 'react-router-dom'
+import { Navigate, useNavigate } from 'react-router-dom'
 import { FormWrapper } from '~components/person/journalfoeringsoppgave/BehandleJournalfoeringOppgave'
 import FerdigstillOppgaveModal from '~components/person/journalfoeringsoppgave/ferdigstilloppgave/FerdigstillOppgaveModal'
 import { Journalstatus } from '~shared/types/Journalpost'
 import { PersonOversiktFane } from '~components/person/Person'
 import { ExternalLinkIcon } from '@navikt/aksel-icons'
+import { erOppgaveRedigerbar } from '~shared/types/oppgave'
+import React from 'react'
 
 export default function FerdigstillOppgave() {
   const { journalpost, oppgave } = useJournalfoeringOppgave()
@@ -16,9 +18,8 @@ export default function FerdigstillOppgave() {
 
   const tilbake = () => navigate('../', { relative: 'path' })
 
-  if (!oppgave) {
-    tilbake()
-    return null
+  if (!oppgave || !erOppgaveRedigerbar(oppgave.status)) {
+    return <Navigate to="../" relative="path" />
   }
 
   const journalpostErFerdigstilt =
diff --git a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/ferdigstilloppgave/FerdigstillOppgaveModal.tsx b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/ferdigstilloppgave/FerdigstillOppgaveModal.tsx
index 345d7a213d0..9f684a7e966 100644
--- a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/ferdigstilloppgave/FerdigstillOppgaveModal.tsx
+++ b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/ferdigstilloppgave/FerdigstillOppgaveModal.tsx
@@ -15,14 +15,6 @@ export default function FerdigstillOppgaveModal({ oppgave }: { oppgave: OppgaveD
 
   const [ferdigstillOppgaveStatus, apiFerdigstillOppgave] = useApiCall(ferdigstillOppgave)
 
-  const ferdigstill = () => {
-    apiFerdigstillOppgave(oppgave.id, () => {
-      setTimeout(() => {
-        navigate(`/`)
-      }, 5000)
-    })
-  }
-
   return (
     <>
       <Button variant="primary" onClick={() => setOpen(true)}>
@@ -41,13 +33,30 @@ export default function FerdigstillOppgaveModal({ oppgave }: { oppgave: OppgaveD
           </BodyLong>
 
           {isSuccess(ferdigstillOppgaveStatus) ? (
-            <Alert variant="success">Oppgaven er nå ferdigstilt. Du blir straks sendt tilbake til oppgavelisten.</Alert>
+            <>
+              <Alert variant="success">Oppgaven er nå ferdigstilt!</Alert>
+
+              <br />
+
+              <FlexRow justify="center">
+                <Button variant="secondary" onClick={() => navigate('/')}>
+                  Gå til oppgavelisten
+                </Button>
+                <Button variant="primary" onClick={() => navigate(`/person/${oppgave.fnr}`)}>
+                  Gå til sakoversikten
+                </Button>
+              </FlexRow>
+            </>
           ) : (
             <FlexRow justify="center">
               <Button variant="secondary" onClick={() => setOpen(false)} disabled={isPending(ferdigstillOppgaveStatus)}>
                 Nei, avbryt
               </Button>
-              <Button variant="primary" onClick={ferdigstill} loading={isPending(ferdigstillOppgaveStatus)}>
+              <Button
+                variant="primary"
+                onClick={() => apiFerdigstillOppgave(oppgave.id)}
+                loading={isPending(ferdigstillOppgaveStatus)}
+              >
                 Ja, ferdigstill
               </Button>
             </FlexRow>
diff --git a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/FullfoerOppgaveModal.tsx b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/FullfoerOppgaveModal.tsx
index c960bdde514..5c5b0cab16b 100644
--- a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/FullfoerOppgaveModal.tsx
+++ b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/FullfoerOppgaveModal.tsx
@@ -8,7 +8,7 @@ import { NyBehandlingRequest } from '~shared/types/IDetaljertBehandling'
 import { useNavigate } from 'react-router-dom'
 import { FlexRow } from '~shared/styled'
 
-import { isFailure, isPending, isSuccess } from '~shared/api/apiUtils'
+import { isPending, isSuccess, mapFailure } from '~shared/api/apiUtils'
 import { OppgaveDTO } from '~shared/types/oppgave'
 
 interface ModalProps {
@@ -30,11 +30,7 @@ export default function FullfoerOppgaveModal({ oppgave, behandlingBehov }: Modal
         mottattDato: behandlingBehov!!.mottattDato!!.replace('Z', ''),
       },
       () => {
-        apiFerdigstillOppgave(oppgave.id, () => {
-          setTimeout(() => {
-            navigate('/')
-          }, 5000)
-        })
+        apiFerdigstillOppgave(oppgave.id)
       }
     )
   }
@@ -60,9 +56,20 @@ export default function FullfoerOppgaveModal({ oppgave, behandlingBehov }: Modal
           </BodyLong>
 
           {isSuccess(opprettBehandlingStatus) && isSuccess(ferdigstillOppgaveStatus) ? (
-            <Alert variant="success">
-              Behandling opprettet for bruker med fødselsnummer {oppgave.fnr}. Du blir straks sendt til oppgavebenken.
-            </Alert>
+            <>
+              <Alert variant="success">Behandling opprettet for bruker med fødselsnummer {oppgave.fnr}</Alert>
+
+              <br />
+
+              <FlexRow justify="center">
+                <Button variant="secondary" onClick={() => navigate('/')}>
+                  Gå til oppgavelisten
+                </Button>
+                <Button variant="primary" onClick={() => navigate(`/person/${oppgave.fnr}`)}>
+                  Gå til sakoversikten
+                </Button>
+              </FlexRow>
+            </>
           ) : (
             <FlexRow justify="center">
               <Button
@@ -81,16 +88,16 @@ export default function FullfoerOppgaveModal({ oppgave, behandlingBehov }: Modal
               </Button>
             </FlexRow>
           )}
-          {isFailure(opprettBehandlingStatus) && (
+          {mapFailure(opprettBehandlingStatus, (error) => (
             <Modal.Footer>
-              <Alert variant="error">Det oppsto en feil ved oppretting av behandlingen.</Alert>
+              <Alert variant="error">{error.detail || 'Det oppsto en feil ved oppretting av behandlingen.'}</Alert>
             </Modal.Footer>
-          )}
-          {isFailure(ferdigstillOppgaveStatus) && (
+          ))}
+          {mapFailure(ferdigstillOppgaveStatus, (error) => (
             <Modal.Footer>
-              <Alert variant="error">Det oppsto en feil ved lukking av oppgaven.</Alert>
+              <Alert variant="error">{error.detail || 'Det oppsto en feil ved lukking av oppgaven'}</Alert>
             </Modal.Footer>
-          )}
+          ))}
         </Modal.Body>
       </Modal>
     </>
diff --git a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/OpprettNyBehandling.tsx b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/OpprettNyBehandling.tsx
index ae65390c20e..ca9b87a31fb 100644
--- a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/OpprettNyBehandling.tsx
+++ b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/OpprettNyBehandling.tsx
@@ -15,6 +15,7 @@ import { FormProvider, useForm } from 'react-hook-form'
 import { ControlledDatoVelger } from '~shared/components/datoVelger/ControlledDatoVelger'
 import { settNyBehandlingRequest } from '~store/reducers/JournalfoeringOppgaveReducer'
 import { useAppDispatch } from '~store/Store'
+import { erOppgaveRedigerbar } from '~shared/types/oppgave'
 
 export interface NyBehandlingSkjema {
   spraak: Spraak | null
@@ -33,7 +34,7 @@ export default function OpprettNyBehandling() {
   const dispatch = useAppDispatch()
   const navigate = useNavigate()
 
-  if (!oppgave) {
+  if (!oppgave || !erOppgaveRedigerbar(oppgave.status)) {
     return <Navigate to="../" relative="path" />
   }
 
diff --git a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/OppsummeringOppgavebehandling.tsx b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/OppsummeringOppgavebehandling.tsx
index 6e594c2b72c..e9e0732c5ec 100644
--- a/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/OppsummeringOppgavebehandling.tsx
+++ b/apps/etterlatte-saksbehandling-ui/client/src/components/person/journalfoeringsoppgave/nybehandling/OppsummeringOppgavebehandling.tsx
@@ -1,7 +1,7 @@
 import { Alert, Button, Detail, Heading, Tag } from '@navikt/ds-react'
 import { useJournalfoeringOppgave } from '~components/person/journalfoeringsoppgave/useJournalfoeringOppgave'
 import AvbrytBehandleJournalfoeringOppgave from '~components/person/journalfoeringsoppgave/AvbrytBehandleJournalfoeringOppgave'
-import { useNavigate } from 'react-router-dom'
+import { Navigate, useNavigate } from 'react-router-dom'
 import { Info } from '~components/behandling/soeknadsoversikt/Info'
 import { SakType } from '~shared/types/sak'
 import { formaterSakstype, formaterSpraak, formaterStringDato } from '~utils/formattering'
@@ -11,6 +11,7 @@ import FullfoerOppgaveModal from '~components/person/journalfoeringsoppgave/nybe
 import { FlexRow } from '~shared/styled'
 import { gyldigBehandlingRequest } from '~components/person/journalfoeringsoppgave/nybehandling/validator'
 import React from 'react'
+import { erOppgaveRedigerbar } from '~shared/types/oppgave'
 
 export default function OppsummeringOppgavebehandling() {
   const { journalpost, oppgave, nyBehandlingRequest, sakMedBehandlinger } = useJournalfoeringOppgave()
@@ -19,13 +20,11 @@ export default function OppsummeringOppgavebehandling() {
 
   const tilbake = () => navigate('../', { relative: 'path' })
 
-  if (
-    !journalpost ||
-    !nyBehandlingRequest ||
-    !oppgave ||
-    !sakMedBehandlinger ||
-    !gyldigBehandlingRequest(nyBehandlingRequest)
-  ) {
+  if (!oppgave || !erOppgaveRedigerbar(oppgave?.status)) {
+    return <Navigate to="../" relative="path" />
+  }
+
+  if (!journalpost || !nyBehandlingRequest || !sakMedBehandlinger || !gyldigBehandlingRequest(nyBehandlingRequest)) {
     return <Alert variant="error">Noe data i journalføringen eller behandlingen er feil</Alert>
   }