From 76d5e5f8057dc3fb71eba6372c615ee4cb231e4d Mon Sep 17 00:00:00 2001
From: VitoAlbano <vito.albano.123@gmail.com>
Date: Thu, 25 Jan 2018 11:24:57 +0000
Subject: [PATCH] [ADF-2163] start refactoring folder dulpication action

---
 .../app/components/files/files.component.html |  2 +-
 .../content-action.component.ts               | 24 ++++++++++---------
 2 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/demo-shell/src/app/components/files/files.component.html b/demo-shell/src/app/components/files/files.component.html
index 87a9b33fca..f41a9171cc 100644
--- a/demo-shell/src/app/components/files/files.component.html
+++ b/demo-shell/src/app/components/files/files.component.html
@@ -198,7 +198,7 @@
                     <!-- folder actions -->
                     <content-action
                         icon="content_copy"
-                        target="all"
+
                         title="{{'DOCUMENT_LIST.ACTIONS.FOLDER.COPY' | translate}}"
                         permission="update"
                         [disableWithNoPermission]="true"
diff --git a/lib/content-services/document-list/components/content-action/content-action.component.ts b/lib/content-services/document-list/components/content-action/content-action.component.ts
index fd7b1c54e8..000a9765d0 100644
--- a/lib/content-services/document-list/components/content-action/content-action.component.ts
+++ b/lib/content-services/document-list/components/content-action/content-action.component.ts
@@ -84,7 +84,7 @@ export class ContentActionComponent implements OnInit, OnChanges {
     success = new EventEmitter();
 
     model: ContentActionModel;
-    duplicateModel: ContentActionModel;
+    // duplicateModel: ContentActionModel;
 
     constructor(
         private list: ContentActionListComponent,
@@ -102,15 +102,13 @@ export class ContentActionComponent implements OnInit, OnChanges {
             target: this.target,
             disabled: this.disabled
         });
-        this.duplicateModel = null;
+
         if (this.handler) {
-            if(this.target === 'all') {
-                this.duplicateModel = Object.assign({}, this.model);
-                this.duplicateModel.handler = this.getSystemHandler('folder', this.handler);
-                this.duplicateModel.target = 'folder';
+            if (this.target === 'all') {
+                this.duplicateActionForFolder();
                 this.model.target = 'document';
-                this.model.handler =  this.getSystemHandler('document', this.handler);
-            }else {
+                this.model.handler = this.getSystemHandler('document', this.handler);
+            }else{
                 this.model.handler = this.getSystemHandler(this.target, this.handler);
             }
         }
@@ -126,9 +124,6 @@ export class ContentActionComponent implements OnInit, OnChanges {
 
     register(): boolean {
         if (this.list) {
-            if(this.target === 'all' ) {
-                this.list.registerAction(this.duplicateModel);
-            }
             return this.list.registerAction(this.model);
         }
         return false;
@@ -139,6 +134,13 @@ export class ContentActionComponent implements OnInit, OnChanges {
         this.model.title = this.title;
     }
 
+    private duplicateActionForFolder() {
+        let folderActionModel = Object.assign({}, this.model);
+        folderActionModel.handler = this.getSystemHandler('folder', this.handler);
+        folderActionModel.target = 'folder';
+        this.list.registerAction(folderActionModel);
+    }
+
     getSystemHandler(target: string, name: string): ContentActionHandler {
         if (target) {
             let ltarget = target.toLowerCase();