[ACS-6369] - ACA on multi select file/folder permisstion menu should …

…not be shown (#3571)
Alfresco · Dec 22, 2023 · 683138c · 683138c
1 parent 7465bbb
commit 683138c
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/projects/aca-playwright-shared/src/resources/test-data/test-data-files-folders.ts b/projects/aca-playwright-shared/src/resources/test-data/test-data-files-folders.ts
@@ -51,8 +51,8 @@ export const folderFavFile = {
 
 // ---- multiple selection ---
 
-const multipleSelContextMenu = ['Download', 'Favorite', 'Move', 'Copy', 'Delete', 'Permissions'];
-const multipleSelToolbarMore = ['Favorite', 'Move', 'Copy', 'Delete', 'Permissions'];
+const multipleSelContextMenu = ['Download', 'Favorite', 'Move', 'Copy', 'Delete'];
+const multipleSelToolbarMore = ['Favorite', 'Move', 'Copy', 'Delete'];
 
 export const multipleSelFile = {
   contextMenu: multipleSelContextMenu,

diff --git a/projects/aca-shared/rules/src/app.rules.spec.ts b/projects/aca-shared/rules/src/app.rules.spec.ts
@@ -849,12 +849,17 @@ describe('app.evaluators', () => {
       expect(app.canManagePermissions(context)).toBe(false);
     });
 
+    it('should return false if many nodes are selected', () => {
+      context.selection.count = 2;
+      expect(app.canManagePermissions(context)).toBe(false);
+    });
+
     it('should return false if the selected node is a smart folder', () => {
       context.selection.first = { entry: { aspectNames: ['smf:customConfigSmartFolder'], isFolder: true } } as NodeEntry;
       expect(app.canManagePermissions(context)).toBe(false);
     });
 
-    it('should return true if user can update the selected node and it is not a trashcan nor smart folder', () => {
+    it('should return true if user can update the selected node and it is not a trashcan nor smart folder nor multiselect', () => {
       expect(app.canManagePermissions(context)).toBe(true);
     });
   });

diff --git a/projects/aca-shared/rules/src/app.rules.ts b/projects/aca-shared/rules/src/app.rules.ts
@@ -505,7 +505,7 @@ export const canEditAspects = (context: RuleContext): boolean =>
  * @param context Rule execution context
  */
 export const canManagePermissions = (context: RuleContext): boolean =>
-  [canUpdateSelectedNode(context), navigation.isNotTrashcan(context), !isSmartFolder(context)].every(Boolean);
+  [canUpdateSelectedNode(context), navigation.isNotTrashcan(context), !isSmartFolder(context), !isMultiselection(context)].every(Boolean);
 
 /**
  * Checks if user can toggle **Edit Offline** mode for selected node.