Merge pull request #1480 from Tampere/fix/project-object-editing

Fix a bug preventing project object editing for non-owner with editin…
Tampere · Jan 21, 2025 · 4fcbee3 · 4fcbee3
2 parents 370b58e + 43479cb
commit 4fcbee3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 17 deletions.
diff --git a/frontend/src/views/Project/ProjectViewWrapper/ModifyButton.tsx b/frontend/src/views/Project/ProjectViewWrapper/ModifyButton.tsx
@@ -65,25 +65,20 @@ export function ModifyButton({
     );
   }
 
-  const splitButtonDisabled = forProjectObject
-    ? popupOpen ||
-      !isOwner ||
-      isSubmitting ||
-      dirtyAndValidViews.finances.isDirtyAndValid ||
-      dirtyAndValidViews.permissions.isDirtyAndValid
-    : popupOpen ||
-      (!isOwner && !canWrite) ||
-      isSubmitting ||
-      dirtyAndValidViews.finances.isDirtyAndValid ||
-      dirtyAndValidViews.permissions.isDirtyAndValid;
+  const editingDisabled =
+    popupOpen ||
+    (!isOwner && !canWrite) ||
+    isSubmitting ||
+    dirtyAndValidViews.finances.isDirtyAndValid ||
+    dirtyAndValidViews.permissions.isDirtyAndValid;
 
   if (forProjectObject) {
     return (
       <>
         <SplitButton
           hideSelectedOptionFromList
-          hideButtonSelection={editing}
-          disableButtonSelection={splitButtonDisabled}
+          hideButtonSelection={editing || !isOwner}
+          disableButtonSelection={editingDisabled}
           cssProp={css`
             margin-left: auto;
           `}
@@ -108,7 +103,7 @@ export function ModifyButton({
                   setEditing(true);
                 }
               }}
-              disabled={splitButtonDisabled}
+              disabled={editingDisabled}
               variant={editing ? 'outlined' : 'contained'}
               size="small"
               endIcon={editing ? <Undo /> : <Create />}
@@ -135,7 +130,7 @@ export function ModifyButton({
       <SplitButton
         hideSelectedOptionFromList
         hideButtonSelection={editing}
-        disableButtonSelection={splitButtonDisabled}
+        disableButtonSelection={editingDisabled}
         cssProp={css`
           margin-left: auto;
         `}
@@ -158,7 +153,7 @@ export function ModifyButton({
                 setEditing(true);
               }
             }}
-            disabled={splitButtonDisabled}
+            disabled={editingDisabled}
             variant={editing ? 'outlined' : 'contained'}
             size="small"
             endIcon={editing ? <Undo /> : <Create />}

diff --git a/shared/src/language/fi.ts b/shared/src/language/fi.ts
@@ -63,7 +63,7 @@ export const fi = {
   'projectObjectView.moveToProject': 'Siirretään kohde toiselle hankkeelle',
   'projectObjectView.moveToProject.label': 'Kohteen uusi hanke',
   'projectObjectView.moveToProject.contentText':
-    'Kohteen voi siirtää investointihankkeelle, jolla on sopiva toteutusväli sekä oikea lautakunta.',
+    'Kohteen voi siirtää investointihankkeelle, jolla on sopiva toteutusväli sekä oikea lautakunta ja jonka omistaja käyttäjä on. Omistajuusrajoite ei koske pääkäyttäjää.',
   'projectObjectView.moveToProject.contentTextSap':
     'Kohteen mahdollinen SAP-rakenneosan tunniste poistetaan siirron yhteydessä.',
   'projectObjectView.moveToProject.noProjects': 'Sopivia hankkeita ei ole.',