Merge pull request #1251 from Elorfin/master

add an option in CopyEvent to choose if we want to publish or not the copied resource

Event/CopyResourceEvent.php

@@ -26,6 +26,12 @@ class CopyResourceEvent extends Event implements DataConveyorEventInterface
     private $copy;
     private $isPopulated = false;
 
+    /**
+     * If true the copy will be published
+     * @var bool
+     */
+    private $publish = false;
+
     /**
      * Constructor.
      *
@@ -36,6 +42,11 @@ public function __construct(AbstractResource $resource, ResourceNode $parent)
     {
         $this->resource = $resource;
         $this->parent   = $parent;
+
+        // By default, use the same published state as the copied node
+        if ($this->resource->getResourceNode()) {
+            $this->publish = $this->resource->getResourceNode()->isPublished();
+        }
     }
 
     /**
@@ -82,4 +93,18 @@ public function isPopulated()
     {
         return $this->isPopulated;
     }
+
+    /**
+     * Is the copied resource need to be published or not ?
+     * @return bool
+     */
+    public function getPublish()
+    {
+        return $this->publish;
+    }
+
+    public function setPublish($publish)
+    {
+        $this->publish = $publish;
+    }
 }

Manager/ResourceManager.php

@@ -772,6 +772,10 @@ public function copy(
 
             $copy = $event->getCopy();
             $newNode = $this->copyNode($node, $parent, $user, $withRights, $rights, $index);
+
+            // Set the published state
+            $newNode->setPublished($event->getPublish());
+
             $copy->setResourceNode($newNode);
 
             if ($node->getResourceType()->getName() == 'directory' &&