[ADF-2358] Always enabling copy action (#3044)

demo-shell/src/app/components/files/files.component.html

@@ -251,7 +251,7 @@
                     <content-action
                         icon="content_copy"
                         title="{{'DOCUMENT_LIST.ACTIONS.FOLDER.COPY' | translate}}"
-                        permission="update"
+                        permission="copy"
                         [disableWithNoPermission]="true"
                         (error)="onContentActionError($event)"
                         (success)="onContentActionSuccess($event)"

lib/content-services/document-list/components/document-list.component.spec.ts

@@ -344,6 +344,27 @@ describe('DocumentList', () => {
 
     });
 
+    it('should not disable the action if there is copy permission', () => {
+        let documentMenu = new ContentActionModel({
+            disableWithNoPermission: true,
+            permission: 'copy',
+            target: 'document',
+            title: 'FileAction'
+        });
+
+        documentList.actions = [
+            documentMenu
+        ];
+
+        let nodeFile = { entry: { isFile: true, name: 'xyz', allowableOperations: ['create', 'update'] } };
+
+        let actions = documentList.getNodeActions(nodeFile);
+        expect(actions.length).toBe(1);
+        expect(actions[0].title).toEqual('FileAction');
+        expect(actions[0].disabled).toBeFalsy();
+
+    });
+
     it('should disable the action if there is no permission for the folder and disableWithNoPermission true', () => {
         let documentMenu = new ContentActionModel({
             disableWithNoPermission: true,

lib/content-services/document-list/components/document-list.component.ts

@@ -24,7 +24,8 @@ import {
     DisplayMode,
     ObjectDataColumn,
     PaginatedComponent,
-    PaginationQueryParams
+    PaginationQueryParams,
+    PermissionsEnum
 } from '@alfresco/adf-core';
 import { AlfrescoApiService, AppConfigService, DataColumnListComponent, UserPreferencesService } from '@alfresco/adf-core';
 import {
@@ -420,7 +421,7 @@ export class DocumentListComponent implements OnInit, OnChanges, OnDestroy, Afte
     }
 
     checkPermission(node: any, action: ContentActionModel): ContentActionModel {
-        if (action.permission) {
+        if (action.permission && action.permission !== PermissionsEnum.COPY) {
             if (this.hasPermissions(node)) {
                 let permissions = node.entry.allowableOperations;
                 let findPermission = permissions.find(permission => permission === action.permission);

lib/core/models/permissions.enum.ts

@@ -19,6 +19,7 @@ export class PermissionsEnum extends String {
     static DELETE: string = 'delete';
     static UPDATE: string = 'update';
     static CREATE: string = 'create';
+    static COPY: string = 'copy';
     static UPDATEPERMISSIONS: string = 'updatePermissions';
     static NOT_DELETE: string = '!delete';
     static NOT_UPDATE: string = '!update';

lib/core/services/content.service.spec.ts

@@ -151,6 +151,11 @@ describe('ContentService', () => {
         expect(contentService.hasPermission(permissionNode, null)).toBeFalsy();
     });
 
+    it('should havePermission return true if permission parameter is copy', () => {
+        let permissionNode = null;
+        expect(contentService.hasPermission(permissionNode, 'copy')).toBeTruthy();
+    });
+
     describe('Download blob', () => {
 
         it('Should use native msSaveOrOpenBlob if the browser is IE', (done) => {

lib/core/services/content.service.ts

@@ -213,6 +213,10 @@ export class ContentService {
             }
         }
 
+        if (permission === PermissionsEnum.COPY) {
+            hasPermission = true;
+        }
+
         return hasPermission;
     }