fix: Korjaa uusien henkilöiden lisäämisestä aiheutuva ongelma henkilö…

…t sivulla (#451)
finnishtransportagency · Oct 20, 2022 · bda7aea · bda7aea
1 parent be868d3
commit bda7aea
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/src/components/form/TextFieldWithController.tsx b/src/components/form/TextFieldWithController.tsx
@@ -2,7 +2,9 @@ import { TextField, TextFieldProps } from "@mui/material";
 import React from "react";
 import { FieldValues, UseControllerProps, useController } from "react-hook-form";
 
-type TextFieldWithControllerProps<TFieldValues> = TextFieldProps & { controllerProps: UseControllerProps<TFieldValues> };
+type TextFieldWithControllerProps<TFieldValues extends FieldValues> = TextFieldProps & {
+  controllerProps: UseControllerProps<TFieldValues>;
+};
 
 export const TextFieldWithController = <TFieldValues extends FieldValues>({
   controllerProps,

diff --git a/src/components/projekti/KayttoOikeusHallinta.tsx b/src/components/projekti/KayttoOikeusHallinta.tsx
@@ -84,11 +84,11 @@ function KayttoOikeusHallintaFormElements({
         projektiPaallikot: FieldArrayWithId<RequiredInputValues, "kayttoOikeudet", "id">[];
         muutHenkilot: FieldArrayWithId<RequiredInputValues, "kayttoOikeudet", "id">[];
       }>(
-        (acc, kayttoOikeus) => {
+        (acc, kayttoOikeus, index) => {
           if (kayttoOikeus.tyyppi === KayttajaTyyppi.PROJEKTIPAALLIKKO) {
-            acc.projektiPaallikot.push(kayttoOikeus);
+            acc.projektiPaallikot[index] = kayttoOikeus;
           } else {
-            acc.muutHenkilot.push(kayttoOikeus);
+            acc.muutHenkilot[index] = kayttoOikeus;
           }
           return acc;
         },
@@ -109,11 +109,10 @@ function KayttoOikeusHallintaFormElements({
         <SectionContent>
           <h5 className="vayla-paragraph">Projektipäällikkö</h5>
           <p>Projektipäällikö on haettu Projektivelhosta. Jos haluat vaihtaa projektipäällikön, muutos pitää tehdä Projektivelhoon.</p>
-          {projektiPaallikot.map((paallikko) => {
+          {projektiPaallikot.map((paallikko, index) => {
             const initialKayttaja = initialKayttajat?.find(({ uid }) => uid === paallikko.kayttajatunnus) || null;
             const kayttajaFromApi = projektiKayttajatFromApi.find(({ kayttajatunnus }) => kayttajatunnus === paallikko.kayttajatunnus);
             const muokattavissa = kayttajaFromApi?.muokattavissa === false ? false : true;
-            const index = kayttoOikeudet.findIndex((oikeus) => oikeus.kayttajatunnus === paallikko.kayttajatunnus);
             return (
               <UserFields
                 disableFields={disableFields}
@@ -131,11 +130,10 @@ function KayttoOikeusHallintaFormElements({
       <SectionContent>
         <h5 className="vayla-paragraph">Muut henkilöt</h5>
         <SectionContent largeGaps>
-          {muutHenkilot.map((user) => {
+          {muutHenkilot.map((user, index) => {
             const initialKayttaja = initialKayttajat?.find(({ uid }) => uid === user.kayttajatunnus) || null;
             const kayttajaFromApi = projektiKayttajatFromApi.find(({ kayttajatunnus }) => kayttajatunnus === user.kayttajatunnus);
             const muokattavissa = kayttajaFromApi?.muokattavissa === false ? false : true;
-            const index = kayttoOikeudet.findIndex((oikeus) => oikeus.kayttajatunnus === user.kayttajatunnus);
             return (
               <UserFields
                 disableFields={disableFields}