Fix scenario editor (#128)

* Fix ScenarioEditor error OPENHEXA-C4

* Sentry: add user information to sentry payload

Author: qgerome

src/features/analysis/AccessibilityAnalysisForm.tsx

@@ -147,7 +147,9 @@ const AccessibilityAnalysisForm = (props: Props) => {
       }
       const { success, errors } = data.updateAccessmodAccessibilityAnalysis;
 
-      if (success) return;
+      if (success) {
+        return;
+      }
 
       if (
         errors.includes(UpdateAccessmodAccessibilityAnalysisError.NameDuplicate)
@@ -158,7 +160,7 @@ const AccessibilityAnalysisForm = (props: Props) => {
   });
 
   useEffect(() => {
-    if (form.isDirty) {
+    if (form.isDirty && form.previousFormData !== form.formData) {
       form.handleSubmit();
     }
   }, [form]);

src/features/dataset/ScenarioEditor.tsx

@@ -27,6 +27,7 @@ const ScenarioEditor = (props: ScenarioEditorProps) => {
     },
     [data, onChange]
   );
+
   const handleRowRemove = useCallback(
     (index: number) => {
       const newData = data.filter((_, i) => i !== index);
@@ -35,11 +36,13 @@ const ScenarioEditor = (props: ScenarioEditorProps) => {
     },
     [data, onChange]
   );
+
   const handleRowAdd = useCallback(() => {
     const newData = data.concat({ kls: `class ${data.length + 1}`, speed: 0 });
     setData(newData);
     onChange(newData);
   }, [data, onChange]);
+
   const handleUpload = useCallback(
     (event: ChangeEvent<HTMLInputElement>) => {
       if (event.target.files && event.target.files.length === 1) {
@@ -126,13 +129,13 @@ const ScenarioEditor = (props: ScenarioEditorProps) => {
             </tr>
           </thead>
           <tbody>
-            {scenario.map((row, i) => {
+            {data.map((row, i) => {
               return (
                 <tr key={i}>
                   <td className="whitespace-nowrap px-3 py-1 text-sm font-medium text-gray-800 md:py-2">
                     <Input
                       className="w-full"
-                      value={data[i].kls}
+                      value={row.kls}
                       onChange={(e) =>
                         handleRowChange(i, "kls", e.target.value)
                       }
@@ -142,7 +145,7 @@ const ScenarioEditor = (props: ScenarioEditorProps) => {
                   <td className="whitespace-nowrap px-3 py-1 text-sm font-medium text-gray-800 md:py-2">
                     <Input
                       className="w-full"
-                      value={data[i].speed}
+                      value={row.speed}
                       onChange={(e) =>
                         handleRowChange(i, "speed", e.target.value)
                       }

src/pages/_app.tsx

@@ -7,6 +7,9 @@ import { Settings } from "luxon";
 import { appWithTranslation } from "next-i18next";
 import Head from "next/head";
 import NavigationProgress from "nextjs-progressbar";
+import { useEffect } from "react";
+import * as Sentry from "@sentry/nextjs";
+
 import "../styles/globals.css";
 
 Settings.defaultLocale = "en";
@@ -19,6 +22,14 @@ function CustomApp({ Component, pageProps }: AppPropsWithLayout) {
     Component.getLayout ??
     ((page) => <Layout pageProps={pageProps}>{page}</Layout>);
 
+  useEffect(() => {
+    Sentry.setUser(
+      pageProps.user
+        ? { email: pageProps.user.email, id: pageProps.user.id }
+        : null
+    );
+  }, [pageProps.user]);
+
   return (
     <>
       <NavigationProgress color="#002C5F" height={3} />