Fjerner unødvendig kode fra oppgave (#6582)

navikt · Dec 6, 2024 · a35ed0a · a35ed0a
1 parent 82f374c
commit a35ed0a
Show file tree

Hide file tree

Showing 7 changed files with 36 additions and 60 deletions.
diff --git a/apps/etterlatte-behandling/src/main/kotlin/oppgave/OppgaveRoutes.kt b/apps/etterlatte-behandling/src/main/kotlin/oppgave/OppgaveRoutes.kt
@@ -287,7 +287,3 @@ internal fun Route.oppgaveRoutes(service: OppgaveService) {
         }
     }
 }
-
-internal data class GosysOppgaveversjon(
-    val versjon: Long,
-)
diff --git a/apps/etterlatte-behandling/src/main/kotlin/oppgaveGosys/GosysOppgaveRoute.kt b/apps/etterlatte-behandling/src/main/kotlin/oppgaveGosys/GosysOppgaveRoute.kt
@@ -17,7 +17,6 @@ import no.nav.etterlatte.libs.ktor.route.OPPGAVEID_GOSYS_CALL_PARAMETER
 import no.nav.etterlatte.libs.ktor.route.gosysOppgaveId
 import no.nav.etterlatte.libs.ktor.route.kunSaksbehandler
 import no.nav.etterlatte.libs.ktor.token.brukerTokenInfo
-import no.nav.etterlatte.oppgave.GosysOppgaveversjon
 
 internal fun Route.gosysOppgaveRoute(gosysService: GosysOppgaveService) {
     route("/api/oppgaver/gosys") {
@@ -118,3 +117,7 @@ data class FeilregistrerOppgaveRequest(
     val beskrivelse: String,
     val versjon: Long,
 )
+
+internal data class GosysOppgaveversjon(
+    val versjon: Long,
+)
diff --git a/...terlatte-saksbehandling-ui/client/src/components/behandling/sidemeny/SettPaaVentModal.tsx b/...terlatte-saksbehandling-ui/client/src/components/behandling/sidemeny/SettPaaVentModal.tsx
@@ -56,7 +56,7 @@ export const SettPaaVentModal = ({ oppgave }: { oppgave: OppgaveDTO }) => {
         if (oppgave.status === Oppgavestatus.PAA_VENT) {
           redigerFristFunc({
             oppgaveId: oppgave.id,
-            redigerFristRequest: { frist: new Date(data.nyFrist), versjon: null },
+            frist: new Date(data.nyFrist),
           })
         }
         dispatch(settOppgave(oppgave))

diff --git a/...tte-saksbehandling-ui/client/src/components/oppgavebenk/frist/SettNyOppgaveFristModal.tsx b/...tte-saksbehandling-ui/client/src/components/oppgavebenk/frist/SettNyOppgaveFristModal.tsx
@@ -23,7 +23,7 @@ export const SettNyOppgaveFristModal = ({ oppgave, oppdaterFrist }: Props) => {
 
   const settNyFristForOppgave = () => {
     if (nyFrist) {
-      redigerFristFunc({ oppgaveId: oppgave.id, redigerFristRequest: { frist: nyFrist, versjon: null } }, () => {
+      redigerFristFunc({ oppgaveId: oppgave.id, frist: nyFrist }, () => {
         oppdaterFrist(oppgave.id, nyFrist.toISOString())
         setFristFeilmelding('')
         setAapen(false)

diff --git a/...latte-saksbehandling-ui/client/src/components/oppgavebenk/tildeling/VelgSaksbehandler.tsx b/...latte-saksbehandling-ui/client/src/components/oppgavebenk/tildeling/VelgSaksbehandler.tsx
@@ -25,7 +25,6 @@ const mapSaksbehandler = (oppgave: OppgaveDTO): Saksbehandler | undefined =>
     : undefined
 
 export const VelgSaksbehandler = ({ saksbehandlereIEnhet, oppdaterTildeling, oppgave }: Props): ReactNode => {
-  const versjon = null
   const { sakId, id: oppgaveId, status } = oppgave
   const innloggetSaksbehandler = useInnloggetSaksbehandler()
   const erRedigerbar =
@@ -39,42 +38,32 @@ export const VelgSaksbehandler = ({ saksbehandlereIEnhet, oppdaterTildeling, opp
   const [fjernSaksbehandlerResult, fjernSaksbehandler] = useApiCall(fjernSaksbehandlerApi)
   const [byttSaksbehandlerResult, byttSaksbehandler] = useApiCall(tildelSaksbehandlerApi)
 
+  const tildel = (saksbehandler: Saksbehandler) =>
+    byttSaksbehandler(
+      { oppgaveId, saksbehandler: saksbehandler.ident },
+      () => {
+        oppdaterTildeling(oppgave, saksbehandler)
+        setValgtSaksbehandler(saksbehandler)
+        setOpenDropdown(false)
+      },
+      (error) => {
+        console.log(error)
+      }
+    )
+
   const onSaksbehandlerSelect = (saksbehandlerNavn: string, erValgt: boolean) => {
     if (erValgt) {
       const selectedSaksbehandler: Saksbehandler | undefined = saksbehandlereIEnhet.find(
         (behandler) => behandler.navn === saksbehandlerNavn
       )
 
       if (selectedSaksbehandler) {
-        byttSaksbehandler(
-          { oppgaveId, nysaksbehandler: { saksbehandler: selectedSaksbehandler.ident!, versjon } },
-          () => {
-            oppdaterTildeling(oppgave, selectedSaksbehandler)
-            setValgtSaksbehandler(selectedSaksbehandler)
-            setOpenDropdown(false)
-          },
-          (error) => console.log(error)
-        )
+        tildel(selectedSaksbehandler)
       }
     }
   }
 
-  const onTildelTilMeg = () => {
-    byttSaksbehandler(
-      { oppgaveId, nysaksbehandler: { saksbehandler: innloggetSaksbehandler.ident, versjon } },
-      () => {
-        oppdaterTildeling(oppgave, innloggetSaksbehandler)
-        setValgtSaksbehandler({
-          ident: innloggetSaksbehandler.ident,
-          navn: innloggetSaksbehandler.navn,
-        })
-        setOpenDropdown(false)
-      },
-      (error) => {
-        console.log(error)
-      }
-    )
-  }
+  const onTildelTilMeg = () => tildel(innloggetSaksbehandler)
 
   const onFjernTildeling = () => {
     fjernSaksbehandler(

diff --git a/apps/etterlatte-saksbehandling-ui/client/src/shared/api/gosys.ts b/apps/etterlatte-saksbehandling-ui/client/src/shared/api/gosys.ts
@@ -1,5 +1,4 @@
 import { apiClient, ApiResponse } from '~shared/api/apiClient'
-import { OppdatertOppgaveversjonResponseDto, SaksbehandlerEndringDto } from '~shared/api/oppgaver'
 import { OppgaveDTO } from '~shared/types/oppgave'
 import { GosysFilter } from '~components/oppgavebenk/filtreringAvOppgaver/typer'
 import { GosysOppgave } from '~shared/types/Gosys'
@@ -38,3 +37,12 @@ export const ferdigstilleGosysOppgave = async (args: {
   versjon: number
 }): Promise<ApiResponse<OppgaveDTO>> =>
   apiClient.post(`/oppgaver/gosys/${args.oppgaveId}/ferdigstill?versjon=${args.versjon}`, {})
+
+export interface OppdatertOppgaveversjonResponseDto {
+  versjon: number | null
+}
+
+export interface SaksbehandlerEndringDto {
+  saksbehandler: string
+  versjon: number | null
+}
diff --git a/apps/etterlatte-saksbehandling-ui/client/src/shared/api/oppgaver.ts b/apps/etterlatte-saksbehandling-ui/client/src/shared/api/oppgaver.ts
@@ -57,15 +57,6 @@ export const ferdigstillOppgaveMedMerknad = async (args: {
   merknad?: string | null
 }): Promise<ApiResponse<OppgaveDTO>> => apiClient.put(`/oppgaver/${args.id}/ferdigstill`, { merknad: args.merknad })
 
-export interface OppdatertOppgaveversjonResponseDto {
-  versjon: number | null
-}
-
-export interface SaksbehandlerEndringDto {
-  saksbehandler: string
-  versjon: number | null
-}
-
 export const saksbehandlereIEnhetApi = async (args: {
   enheter: string[]
 }): Promise<ApiResponse<Array<Saksbehandler>>> => {
@@ -74,28 +65,17 @@ export const saksbehandlereIEnhetApi = async (args: {
 
 export const tildelSaksbehandlerApi = async (args: {
   oppgaveId: string
-  nysaksbehandler: SaksbehandlerEndringDto
-}): Promise<ApiResponse<OppdatertOppgaveversjonResponseDto>> => {
-  return apiClient.post(`/oppgaver/${args.oppgaveId}/tildel-saksbehandler`, { ...args.nysaksbehandler })
+  saksbehandler: string
+}): Promise<ApiResponse<void>> => {
+  return apiClient.post(`/oppgaver/${args.oppgaveId}/tildel-saksbehandler`, { saksbehandler: args.saksbehandler })
 }
 
-export const fjernSaksbehandlerApi = async (args: {
-  oppgaveId: string
-  sakId: number
-}): Promise<ApiResponse<OppdatertOppgaveversjonResponseDto>> => {
+export const fjernSaksbehandlerApi = async (args: { oppgaveId: string; sakId: number }): Promise<ApiResponse<void>> => {
   return apiClient.delete(`/oppgaver/${args.oppgaveId}/saksbehandler`)
 }
 
-export interface RedigerFristRequest {
-  frist: Date
-  versjon: number | null
-}
-
-export const redigerFristApi = async (args: {
-  oppgaveId: string
-  redigerFristRequest: RedigerFristRequest
-}): Promise<ApiResponse<void>> => {
-  return apiClient.put(`/oppgaver/${args.oppgaveId}/frist`, { ...args.redigerFristRequest })
+export const redigerFristApi = async (args: { oppgaveId: string; frist: Date }): Promise<ApiResponse<void>> => {
+  return apiClient.put(`/oppgaver/${args.oppgaveId}/frist`, { frist: args.frist })
 }
 
 export interface EndrePaaVentRequest {
-Original file line number
+Diff line change
@@ Expand Up / @@ -287,7 +287,3 @@ internal fun Route.oppgaveRoutes(service: OppgaveService) { @@
             }
         }
     }
-    internal data class GosysOppgaveversjon(
-        val versjon: Long,
-    )