From 87e3c59da8b2594cdc40a784c3a038db4b3d9eee Mon Sep 17 00:00:00 2001
From: niestrow <wojciech.niestroj@contractors.roche.com>
Date: Fri, 29 Nov 2024 17:24:40 +0100
Subject: [PATCH] FIx sharing project not saving

---
 frontend-v2/src/features/projects/Project.tsx | 35 ++++++++++++-------
 .../src/features/projects/UserAccess.tsx      | 10 +++++-
 2 files changed, 31 insertions(+), 14 deletions(-)

diff --git a/frontend-v2/src/features/projects/Project.tsx b/frontend-v2/src/features/projects/Project.tsx
index 7ac934c8..7db9a7fe 100644
--- a/frontend-v2/src/features/projects/Project.tsx
+++ b/frontend-v2/src/features/projects/Project.tsx
@@ -160,7 +160,6 @@ const ProjectRow: FC<Props> = ({
               () => dispatch(decrementDirtyCount()),
             );
           }
-          console.log('project', project, data.project)
           if (JSON.stringify(project) !== JSON.stringify(data.project)) {
             dispatch(incrementDirtyCount());
             updateProject({ id: project.id, project: data.project }).then(() =>
@@ -173,9 +172,15 @@ const ProjectRow: FC<Props> = ({
   );
 
   const onCancel = () => {
-    setValue('project.description', project.description);
+    setValue("project.description", project.description);
   };
 
+  const onShareCanel = () => {
+    setValue("project.user_access", project.user_access);
+    setUserAccessOpen(false);
+    setIsEditMode(false);
+  }
+
   if (isLoading) {
     return <div>Loading...</div>;
   }
@@ -341,7 +346,10 @@ const ProjectRow: FC<Props> = ({
 
               <Tooltip title="Share Project">
                 <IconButton
-                  onClick={() => setUserAccessOpen(true)}
+                  onClick={() => {
+                    setIsEditMode(true);
+                    setUserAccessOpen(true);
+                  }}
                   disabled={isSharedWithMe}
                 >
                   <PersonAdd />
@@ -352,15 +360,6 @@ const ProjectRow: FC<Props> = ({
                   <Delete />
                 </IconButton>
               </Tooltip>
-              <UserAccess
-                open={userAccessOpen}
-                onClose={userAccessClose}
-                control={control}
-                userAccess={userAccess as ProjectAccess[]}
-                append={append}
-                remove={remove}
-                project={project}
-              />
               {isSharedWithMe && (
                 <Tooltip title="This project is shared with me as view-only">
                   <div>
@@ -402,6 +401,16 @@ const ProjectRow: FC<Props> = ({
           )}
         </TableCell>
       </TableRow>
+      <UserAccess
+        open={userAccessOpen}
+        onClose={userAccessClose}
+        onCancel={onShareCanel}
+        control={control}
+        userAccess={userAccess as ProjectAccess[]}
+        append={append}
+        remove={remove}
+        project={project}
+      />
       {isDescriptionModalOpen && (
         <DescriptionModal
           isOpen={isDescriptionModalOpen}
@@ -412,7 +421,7 @@ const ProjectRow: FC<Props> = ({
           project={project}
         >
           <MaterialTextField
-            {...register('project.description')}
+            {...register("project.description")}
             sx={{ width: "50vw" }}
             disabled={!isEditMode}
             multiline
diff --git a/frontend-v2/src/features/projects/UserAccess.tsx b/frontend-v2/src/features/projects/UserAccess.tsx
index 02fca709..bbded8b7 100644
--- a/frontend-v2/src/features/projects/UserAccess.tsx
+++ b/frontend-v2/src/features/projects/UserAccess.tsx
@@ -32,6 +32,7 @@ interface Props {
   userAccess: ProjectAccess[];
   append: (value: ProjectAccess) => void;
   remove: (index: number) => void;
+  onCancel: () => void;
   control: Control<FormData>;
   onClose: () => void;
 }
@@ -41,7 +42,7 @@ const UserAccess: FC<Props> = ({
   userAccess,
   append,
   remove,
-  control,
+  onCancel,
   onClose,
 }) => {
   const { data: users } = useUserListQuery();
@@ -122,6 +123,13 @@ const UserAccess: FC<Props> = ({
               margin: ".5rem",
             }}
           >
+            <Button
+              variant="outlined"
+              sx={{ marginLeft: "1rem" }}
+              onClick={onCancel}
+            >
+              Cancel
+            </Button>
             <Button
               variant="contained"
               sx={{ marginLeft: "1rem" }}