Ikkje ta i mot undefined som tekst i kolonne

(tving oss til å sende inn '-' utanfrå om data er undefined)
navikt · Oct 28, 2024 · a349058 · a349058
1 parent a14e850
commit a349058
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 15 deletions.
diff --git a/src/components/tabell/headerceller/Bosted.tsx b/src/components/tabell/headerceller/Bosted.tsx
@@ -3,6 +3,7 @@ import {Sorteringsfelt} from '../../../model-interfaces';
 import {Kolonne} from '../../../ducks/ui/listevisning';
 import SorteringHeader from '../sortering-header';
 
+/** Denne viser kommune, "Utland eller "Ukjent". */
 export const Bosted = ({gjeldendeSorteringsfelt, valgteKolonner, rekkefolge, onClick}: HeadercelleProps) => (
     <SorteringHeader
         skalVises={valgteKolonner.includes(Kolonne.BOSTED_KOMMUNE)}

diff --git a/src/components/tabell/kolonner/tekstkolonne.tsx b/src/components/tabell/kolonner/tekstkolonne.tsx
@@ -3,13 +3,14 @@ import {BodyShort} from '@navikt/ds-react';
 import classNames from 'classnames';
 
 interface TekstKolonneProps {
-    tekst?: string;
+    /** Send inn "-" om det ikkje er noko tekst å vise */
+    tekst: string;
     skalVises: boolean;
     className?: string;
 }
 
 export function TekstKolonne({tekst, skalVises, className}: TekstKolonneProps) {
-    if (!skalVises || !tekst) {
+    if (!skalVises) {
         return null;
     }
 

diff --git a/src/enhetsportefolje/enhet-kolonner.tsx b/src/enhetsportefolje/enhet-kolonner.tsx
@@ -21,8 +21,9 @@ import {BarnUnder18Aar, BrukerModell, FiltervalgModell, VeilederModell} from '..
 import {
     aapRettighetsperiode,
     aapVurderingsfrist,
-    bostedKommune,
+    bostedKommuneUtlandEllerUkjent,
     capitalize,
+    bostedBydelEllerUkjent,
     mapOmAktivitetsPlikt,
     nesteUtlopsdatoEllerNull,
     oppfolingsdatoEnsligeForsorgere,
@@ -157,12 +158,12 @@ function EnhetKolonner({className, bruker, enhetId, filtervalg, valgteKolonner,
             <TekstKolonne
                 className="col col-xs-2"
                 skalVises={valgteKolonner.includes(Kolonne.BOSTED_KOMMUNE)}
-                tekst={bostedKommune(bruker, geografiskbostedData)}
+                tekst={bostedKommuneUtlandEllerUkjent(bruker, geografiskbostedData)}
             />
             <TekstKolonne
                 className="col col-xs-2"
                 skalVises={valgteKolonner.includes(Kolonne.BOSTED_BYDEL)}
-                tekst={bruker.bostedBydel ? geografiskbostedData.get(bruker.bostedBydel) : '-'}
+                tekst={bruker.bostedBydel ? bostedBydelEllerUkjent(bruker.bostedBydel, geografiskbostedData) : '-'}
             />
             <TekstKolonne
                 className="col col-xs-2"
@@ -325,7 +326,7 @@ function EnhetKolonner({className, bruker, enhetId, filtervalg, valgteKolonner,
                 skalVises={!!ferdigfilterListe?.includes(MOTER_IDAG) && valgteKolonner.includes(Kolonne.MOTE_ER_AVTALT)}
             />
             <TekstKolonne
-                tekst={bruker.utkast14aStatus}
+                tekst={bruker.utkast14aStatus ?? '-'}
                 skalVises={
                     !!ferdigfilterListe?.includes(UNDER_VURDERING) && valgteKolonner.includes(Kolonne.VEDTAKSTATUS)
                 }
@@ -380,7 +381,7 @@ function EnhetKolonner({className, bruker, enhetId, filtervalg, valgteKolonner,
                 className="col col-xs-2"
             />
             <TekstKolonne
-                tekst={bruker.ensligeForsorgereOvergangsstonad?.vedtaksPeriodetype}
+                tekst={bruker.ensligeForsorgereOvergangsstonad?.vedtaksPeriodetype ?? '-'}
                 skalVises={valgteKolonner.includes(Kolonne.ENSLIGE_FORSORGERE_VEDTAKSPERIODE)}
                 className="col col-xs-2"
             />

diff --git a/src/minoversikt/minoversikt-kolonner.tsx b/src/minoversikt/minoversikt-kolonner.tsx
@@ -3,8 +3,9 @@ import moment from 'moment';
 import {
     aapRettighetsperiode,
     aapVurderingsfrist,
-    bostedKommune,
+    bostedKommuneUtlandEllerUkjent,
     capitalize,
+    bostedBydelEllerUkjent,
     mapOmAktivitetsPlikt,
     nesteUtlopsdatoEllerNull,
     oppfolingsdatoEnsligeForsorgere,
@@ -176,12 +177,12 @@ function MinoversiktDatokolonner({bruker, enhetId, filtervalg, valgteKolonner}:
             <TekstKolonne
                 className="col col-xs-2"
                 skalVises={valgteKolonner.includes(Kolonne.BOSTED_KOMMUNE)}
-                tekst={bostedKommune(bruker, geografiskbostedData)}
+                tekst={bostedKommuneUtlandEllerUkjent(bruker, geografiskbostedData)}
             />
             <TekstKolonne
                 className="col col-xs-2"
                 skalVises={valgteKolonner.includes(Kolonne.BOSTED_BYDEL)}
-                tekst={bruker.bostedBydel ? geografiskbostedData.get(bruker.bostedBydel) : '-'}
+                tekst={bruker.bostedBydel ? bostedBydelEllerUkjent(bruker.bostedBydel, geografiskbostedData) : '-'}
             />
             <TekstKolonne
                 className="col col-xs-2"
@@ -353,7 +354,7 @@ function MinoversiktDatokolonner({bruker, enhetId, filtervalg, valgteKolonner}:
                 }
             />
             <TekstKolonne
-                tekst={bruker.utkast14aStatus}
+                tekst={bruker.utkast14aStatus ?? '-'}
                 skalVises={
                     !!ferdigfilterListe?.includes(UNDER_VURDERING) && valgteKolonner.includes(Kolonne.VEDTAKSTATUS)
                 }
@@ -408,7 +409,7 @@ function MinoversiktDatokolonner({bruker, enhetId, filtervalg, valgteKolonner}:
                 className="col col-xs-2"
             />
             <TekstKolonne
-                tekst={bruker.ensligeForsorgereOvergangsstonad?.vedtaksPeriodetype}
+                tekst={bruker.ensligeForsorgereOvergangsstonad?.vedtaksPeriodetype ?? '-'}
                 skalVises={valgteKolonner.includes(Kolonne.ENSLIGE_FORSORGERE_VEDTAKSPERIODE)}
                 className="col col-xs-2"
             />

diff --git a/src/utils/utils.ts b/src/utils/utils.ts
@@ -75,7 +75,7 @@ export function ytelsestypetekst(brukerytelse: string) {
         return 'Ordinær';
     } else if (brukerytelse === 'AAP_UNNTAK') {
         return 'Unntak';
-    }
+    } else return '-';
 }
 
 export function aapVurderingsfrist(
@@ -186,7 +186,7 @@ export function tolkBehovSpraak(
     filtervalg: FiltervalgModell,
     bruker: BrukerModell,
     tolkbehovSpraakData: Map<string, string>
-) {
+): string {
     const behovSpraak: string[] = [];
     let leggTilSpraak = leggTilSpraakInfo(filtervalg);
 
@@ -262,7 +262,7 @@ export function capitalize(str: string) {
         .replace(/(^|[^a-z\u00C0-\u017F\u0400-\u04FF'])([a-z\u00C0-\u017F\u0400-\u04FF])/g, s => s.toUpperCase());
 }
 
-export function bostedKommune(bruker: BrukerModell, geografiskbostedData) {
+export function bostedKommuneUtlandEllerUkjent(bruker: BrukerModell, geografiskbostedData) {
     if (bruker.bostedKommune) {
         return geografiskbostedData.get(bruker.bostedKommune);
     }
@@ -275,6 +275,10 @@ export function bostedKommune(bruker: BrukerModell, geografiskbostedData) {
     return '-';
 }
 
+export const bostedBydelEllerUkjent = (bostedBydel: string, geografiskbostedData: Map<string, string>): string => {
+    return geografiskbostedData.get(bostedBydel) || 'Ukjent';
+};
+
 export const mapOmAktivitetsPlikt = (aktivitetsplikt?: boolean): string => {
     if (aktivitetsplikt === undefined) {
         return 'Ukjent';