BUGFIX: Don’t copy removed nodes

With this change, removed nodes are not copied anymore
in the recursive copy actions. Also the removed state is kept
when a node is similarisied to prevent unpublished removed
nodes from popping up again inadvertendly.

Resolves: #5185

Neos.ContentRepository/Classes/Domain/Model/Node.php

@@ -1743,6 +1743,10 @@ protected function createRecursiveCopy(NodeInterface $referenceNode, string $nod
         }
         /** @var $childNode Node */
         foreach ($this->getChildNodes() as $childNode) {
+            // Don't copy removed nodes
+            if ($childNode->isRemoved()) {
+                continue;
+            }
             // Prevent recursive copy when copying into itself
             if ($childNode->getIdentifier() !== $copiedNode->getIdentifier()) {
                 $childNode->copyIntoInternal($copiedNode, $childNode->getName(), $detachedCopy);

Neos.ContentRepository/Classes/Domain/Model/NodeData.php

@@ -763,7 +763,8 @@ public function similarize(AbstractNodeData $sourceNode, $isCopy = false)
             'hiddenAfterDateTime',
             'hiddenBeforeDateTime',
             'hiddenInIndex',
-            'accessRoles'
+            'accessRoles',
+            'removed',
         ];
         if (!$isCopy) {
             $propertyNames[] = 'creationDateTime';