Mer fix

navikt · Nov 7, 2023 · 614f071 · 614f071
1 parent ef3109d
commit 614f071
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 35 deletions.
diff --git a/apps/dolly-frontend/src/main/js/src/components/fagsystem/inntektsmelding/visning/Visning.tsx b/apps/dolly-frontend/src/main/js/src/components/fagsystem/inntektsmelding/visning/Visning.tsx
@@ -55,22 +55,23 @@ export const InntektsmeldingVisning = ({
 
 	const forsteMiljo = data?.find((miljoData) => miljoData?.data)?.miljo
 
-	// const harTransaksjonsidData = data?.some((inntekt) => inntekt?.data?.request)
 	const harTransaksjonsidData = data?.every((inntekt) => inntekt?.data?.request)
-	// console.log('harTransaksjonsidData: ', harTransaksjonsidData) //TODO - SLETT MEG
-	console.log('data: ', data) //TODO - SLETT MEG
-	console.log('bestillinger: ', bestillinger) //TODO - SLETT MEG
 
 	const setTransaksjonsidData = () => {
 		return data.map((miljo) => {
+			const request = miljo?.data?.request
+			const dokument = miljo?.data?.dokument
 			return {
 				data: {
-					dokument: miljo.data,
-					request: {
-						//TODO: Hent inntektsmelding fra alle bestillinger, ikke bare den forste
-						inntekter: bestillinger?.[0]?.data.inntektsmelding.inntekter,
-						miljoe: miljo.miljo,
-					},
+					dokument: dokument ? dokument : miljo.data,
+					request: request
+						? request
+						: {
+								inntekter: bestillinger?.flatMap(
+									(bestilling) => bestilling?.data.inntektsmelding.inntekter,
+								),
+								miljoe: miljo.miljo,
+						  },
 				},
 				miljo: miljo.miljo,
 			}
@@ -101,9 +102,6 @@ export const InntektsmeldingVisning = ({
 	const filteredData =
 		tilgjengeligMiljoe && mergetData?.filter((item) => item.miljo === tilgjengeligMiljoe)
 
-	// console.log('filteredData: ', filteredData) //TODO - SLETT MEG
-	// console.log('mergetData: ', mergetData) //TODO - SLETT MEG
-
 	return (
 		<>
 			<SubOverskrift label="Inntektsmelding (fra Altinn)" iconKind="inntektsmelding" />

diff --git a/...ain/js/src/components/fagsystem/inntektsmelding/visning/partials/personVisningContent.tsx b/...ain/js/src/components/fagsystem/inntektsmelding/visning/partials/personVisningContent.tsx
@@ -5,6 +5,7 @@ import useBoolean from '@/utils/hooks/useBoolean'
 import { Journalpost } from '@/components/fagsystem/inntektsmelding/InntektsmeldingTypes'
 import styled from 'styled-components'
 import { useDokument } from '@/utils/hooks/useJoarkDokument'
+import { Alert } from '@navikt/ds-react'
 
 interface PersonVisningContentProps {
 	id: Journalpost
@@ -56,7 +57,14 @@ export const PersonVisningContent = ({ miljoe, dokumentInfo }: PersonVisningCont
 					{(viserSkjemainnhold ? 'SKJUL ' : 'VIS ') + 'SKJEMAINNHOLD'}
 				</Button>
 			</div>
-			{viserSkjemainnhold && <CodeView language="xml" code={dokument} />}
+			{viserSkjemainnhold &&
+				(dokument ? (
+					<CodeView language="xml" code={dokument} />
+				) : (
+					<Alert variant="error" size="small" inline>
+						Kunne ikke hente skjemainnhold
+					</Alert>
+				))}
 		</StyledJournalpost>
 	)
 }
diff --git a/apps/dolly-frontend/src/main/js/src/utils/hooks/useFagsystemer.tsx b/apps/dolly-frontend/src/main/js/src/utils/hooks/useFagsystemer.tsx
@@ -110,7 +110,7 @@ export const useTransaksjonIdData = (ident, system, harBestilling, fagsystemMilj
 		harBestilling ? `/dolly-backend/api/v1/transaksjonid?ident=${ident}&system=${system}` : null,
 		fetcher,
 	)
-	// console.log('data: ', data) //TODO - SLETT MEG
+
 	const getMiljoData = () => {
 		if (!harBestilling || !data) {
 			return null
@@ -119,10 +119,6 @@ export const useTransaksjonIdData = (ident, system, harBestilling, fagsystemMilj
 		if (fagsystemMiljoer && fagsystemMiljoer.length > 0) {
 			fagsystemMiljoer.map((miljo) => {
 				const fagsystemData = data?.find((d) => d?.miljoe === miljo)?.transaksjonId
-				// if (!fagsystemData?.dokument && !fagsystemData?.request) {
-				// 	miljoData.push({})
-				// }
-				//TODO: Fortsette her!! Vil legge journalpostId og dokumentInfoId inn i et dokument-object der de ikke er det
 				miljoData.push({ data: fagsystemData, miljo: miljo })
 			})
 		} else {
@@ -134,8 +130,6 @@ export const useTransaksjonIdData = (ident, system, harBestilling, fagsystemMilj
 	}
 	const miljoData = getMiljoData()
 
-	//TODO: Kanskje legge til alle relevante miljoer i miljodata? Slik at vi faar faner for alle miljoer, ogsaa de det ikke finnes data i
-
 	return {
 		data: miljoData?.sort((a, b) => a.miljo?.localeCompare(b.miljo)),
 		loading: isLoading,

diff --git a/apps/dolly-frontend/src/main/js/src/utils/hooks/useJoarkDokument.tsx b/apps/dolly-frontend/src/main/js/src/utils/hooks/useJoarkDokument.tsx
@@ -3,19 +3,6 @@ import { fetcher } from '@/api'
 
 const joarkUrl = '/testnav-joark-dokument-service/api/v2/journalpost/'
 
-//TODO: Trenger kanskje ikke denne?
-export const useJournalpost = (journalpostId: number, miljo: string) => {
-	const { data, isLoading, error } = useSWR<any, Error>(
-		[joarkUrl + journalpostId, { miljo: miljo }],
-		([url, headers]) => fetcher(url, headers),
-	)
-	return {
-		journalpost: data,
-		loading: isLoading,
-		error: error,
-	}
-}
-
 export const useDokument = (
 	journalpostId: number,
 	dokumentInfoId: number,
@@ -24,7 +11,9 @@ export const useDokument = (
 ) => {
 	const { data, isLoading, error } = useSWR<any, Error>(
 		[
-			`${joarkUrl}${journalpostId}/dokumenter/${dokumentInfoId}?dokumentType=${dokumentType}`,
+			journalpostId && dokumentInfoId && dokumentType
+				? `${joarkUrl}${journalpostId}/dokumenter/${dokumentInfoId}?dokumentType=${dokumentType}`
+				: null,
 			{ miljo: miljo },
 		],
 		([url, headers]) => fetcher(url, headers),