Fix params to getDialogLenke

navikt · Aug 15, 2023 · 0b2125a · 0b2125a
1 parent 36f8d88
commit 0b2125a
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 12 deletions.
diff --git a/src/moduler/aktivitet/visning/hjelpekomponenter/SendEnMeldingKnapp.tsx b/src/moduler/aktivitet/visning/hjelpekomponenter/SendEnMeldingKnapp.tsx
@@ -30,7 +30,7 @@ const SendEnMeldingKnapp = (props: Props) => {
     const veilederOnClick = (event: React.MouseEvent) => {
         if (erVeileder) {
             navigate(hovedsideRoute(), { replace: true });
-            byttTilDialogFlate(event, aktivitet.id, dialog?.id);
+            byttTilDialogFlate({ event, fnr, aktivitetId: aktivitet?.id, dialogId: dialog?.id });
         }
     };
 
@@ -39,7 +39,7 @@ const SendEnMeldingKnapp = (props: Props) => {
             <Button
                 variant="secondary"
                 as="a"
-                href={getDialogLenke(erVeileder, fnr, aktivitet.id, dialog?.id)}
+                href={getDialogLenke({ erVeileder, fnr, aktivitetId: aktivitet.id, dialogId: dialog?.id })}
                 icon={<ChatElipsisIcon aria-hidden fontSize="1.5rem" />}
                 onClick={veilederOnClick}
             >

diff --git a/src/moduler/dialog/DialogFlateUtils.ts b/src/moduler/dialog/DialogFlateUtils.ts
@@ -7,34 +7,54 @@ interface DialogEventDetails {
     aktivitetId?: string;
 }
 
-export const byttTilDialogFlate = (event: MouseEvent, fnr?: string, aktiviteId?: string, dialogId?: string) => {
+export const byttTilDialogFlate = ({
+    event,
+    aktivitetId,
+    dialogId,
+    fnr,
+}: {
+    event: MouseEvent;
+    fnr?: string;
+    aktivitetId?: string;
+    dialogId?: string;
+}) => {
     event.preventDefault();
-    window.history.pushState('', 'Dialog', getDialogLenke(true, fnr, aktiviteId, dialogId));
+    window.history.pushState('', 'Dialog', getDialogLenke({ erVeileder: true, fnr, aktivitetId, dialogId }));
     window.dispatchEvent(
         new CustomEvent<DialogEventDetails>('visDialog', {
             detail: {
                 dialogId: dialogId,
-                aktivitetId: aktiviteId,
+                aktivitetId: aktivitetId,
             },
         })
     );
 };
 
-export const getDialogLenke = (erVeileder: boolean, fnr?: string, aktiviteId?: string, dialogId?: string) => {
+export const getDialogLenke = ({
+    dialogId,
+    fnr,
+    aktivitetId,
+    erVeileder,
+}: {
+    erVeileder: boolean;
+    fnr?: string;
+    aktivitetId?: string;
+    dialogId?: string;
+}) => {
     if (erVeileder && fnr) {
         if (dialogId) {
             return `/${fnr}/${dialogId}`;
         }
-        if (aktiviteId) {
-            return `/${fnr}/ny?aktivitetId=${aktiviteId}`;
+        if (aktivitetId) {
+            return `/${fnr}/ny?aktivitetId=${aktivitetId}`;
         }
         return `/${fnr}`;
     } else {
         if (dialogId) {
             return `${ARBEIDSRETTET_DIALOG_URL}/${dialogId}`;
         }
-        if (aktiviteId) {
-            return `${ARBEIDSRETTET_DIALOG_URL}/ny?aktivitetId=${aktiviteId}`;
+        if (aktivitetId) {
+            return `${ARBEIDSRETTET_DIALOG_URL}/ny?aktivitetId=${aktivitetId}`;
         }
         return `${ARBEIDSRETTET_DIALOG_URL}`;
     }

diff --git a/src/moduler/dialog/DialogLink.tsx b/src/moduler/dialog/DialogLink.tsx
@@ -24,7 +24,7 @@ const LenkeTilDialog = (props: Props) => {
     const internalOnClick = (event: MouseEvent) => {
         if (erVeileder) {
             navigate(hovedsideRoute(), { replace: true });
-            byttTilDialogFlate(event, fnr, aktivitetId, dialogId);
+            byttTilDialogFlate({ event, fnr, aktivitetId, dialogId });
         }
     };
 
@@ -34,7 +34,7 @@ const LenkeTilDialog = (props: Props) => {
 
     return (
         <a
-            href={getDialogLenke(erVeileder, fnr, aktivitetId, dialogId)}
+            href={getDialogLenke({ erVeileder, fnr, aktivitetId, dialogId })}
             onClick={internalOnClick}
             className={className}
         >