Diverse kodeforbedring etter Sonarcloud-rapport

example/src/components/SprakVelger.tsx

@@ -10,13 +10,13 @@ export const SprakVelger = (props: Props) => {
     const { locales, valgtLocale, settValgtLocale } = props;
     return (
         <div className="example__velger">
-            {locales.map((locale, i) =>
+            {locales.map((locale) =>
                 valgtLocale === locale ? (
-                    <span key={i} className="example__sprak">
+                    <span key={locale} className="example__sprak">
                         <b>{locale}</b>
                     </span>
                 ) : (
-                    <button key={i} className="example__sprak" onClick={() => settValgtLocale(locale)}>
+                    <button key={locale} className="example__sprak" onClick={() => settValgtLocale(locale)}>
                         {locale}
                     </button>
                 )

src/clients/apiClient.ts

@@ -36,19 +36,19 @@ const hentJsonOgSjekkAuth = (url: string, headers?: object) =>
 
 export const hentListeMedArbeidsforhold = (customApiUrl?: string) => {
     const defaultApiUrl = `${Environment.apiUrl}/arbeidsforhold`;
-    return hentJsonOgSjekkAuth(customApiUrl || defaultApiUrl);
+    return hentJsonOgSjekkAuth(customApiUrl ?? defaultApiUrl);
 };
 
 // Henter fødselsnummer fra token
 export const hentDetaljertArbeidsforholdArbeidstaker = (id: number, customApiUrl?: string) => {
     const defaultApiUrl = `${Environment.apiUrl}/arbeidsforholdinnslag/arbeidstaker/{id}`;
-    const apiUrl = (customApiUrl || defaultApiUrl).replace('{id}', id.toString());
+    const apiUrl = (customApiUrl ?? defaultApiUrl).replace('{id}', id.toString());
     return hentJsonOgSjekkAuth(apiUrl);
 };
 
 // Bruker tokenet til arbeidsgiver
 export const hentDetaljertArbeidsforholdArbeidsgiver = (fnr: string, id: number, customApiUrl?: string) => {
     const defaultApiUrl = `${Environment.apiUrl}/arbeidsforholdinnslag/arbeidsgiver/{id}`;
-    const apiUrl = (customApiUrl || defaultApiUrl).replace('{id}', id.toString());
+    const apiUrl = (customApiUrl ?? defaultApiUrl).replace('{id}', id.toString());
     return hentJsonOgSjekkAuth(apiUrl, { 'Fnr-Arbeidstaker': `${fnr}` });
 };

src/components/arbeidsgiver/ArbeidsgiverTittel.tsx

@@ -4,7 +4,6 @@ import { fnr } from '@/utils/fnr';
 
 interface Props {
     arbeidsgiver: AFArbeidsgiver;
-    overskrift?: boolean;
 }
 
 export const ArbeidsgiverTittel = (props: Props) => {

src/components/check-date-and-print/CheckDateAndPrint.tsx

@@ -23,7 +23,7 @@ export const CheckDateAndPrint = (props: Props) => {
 
     const date = dayjs(props.data)
         .locale(locale)
-        .format(props.dateFormat || 'DD.MM.YYYY');
+        .format(props.dateFormat ?? 'DD.MM.YYYY');
 
     const formattedDate = props.format ? parse(props.format, date) : date;
 

src/modules/af-detaljert/print/pdf/GenerellOversiktPDF.tsx

@@ -12,11 +12,6 @@ import { useLocale } from '@/modules/common/useLocale';
 
 interface Props {
     arbeidsforhold: AFUtvidet;
-    printGenerellOversikt: boolean;
-    printTimerTimelonnet: boolean;
-    printPermisjon: boolean;
-    printUtenlandsopphold: boolean;
-    printHistorikk: boolean;
     printName: string;
     printSSO: string;
 }

src/modules/af-detaljert/print/pdf/tabs/PermisjonPDF.tsx

@@ -10,10 +10,14 @@ interface Props {
     permisjoner: AFPermisjonPermittering[];
 }
 
+const sortPermisjoner = (left: AFPermisjonPermittering, right: AFPermisjonPermittering) => {
+    return sortPeriodeFraDesc(left.periode, right.periode);
+};
+
 export const PermisjonPDF = (props: Props) => {
     const { locale } = useLocale();
 
-    const sortedPermisjoner = props.permisjoner.sort((left, right) => sortPeriodeFraDesc(left.periode, right.periode));
+    const sortedPermisjoner = props.permisjoner.sort(sortPermisjoner);
 
     return (
         <>

src/modules/common/useIsPdf.tsx

@@ -1,5 +1,4 @@
-import React from 'react';
-import { useContext } from 'react';
+import React, { useContext } from 'react';
 
 const IsPdfContext = React.createContext<boolean>(false);
 

src/modules/common/useLocale.tsx

@@ -1,6 +1,5 @@
-import React from 'react';
+import React, { useContext } from 'react';
 import { Locale } from '@/types/locale';
-import { useContext } from 'react';
 
 const LocaleContext = React.createContext<Locale>('nb');