TASK: Make E2E tests work for Neos 9.0

.circleci/config.yml

@@ -65,7 +65,7 @@ jobs:
     environment:
       COMPOSER_CACHE_DIR: /home/circleci/composer/cache-dir
     docker:
-      - image: cimg/php:8.0-node
+      - image: cimg/php:8.2-node
     steps:
       - attach_workspace: *attach_workspace
       - run: chmod -R 777 /home/circleci
@@ -81,8 +81,9 @@ jobs:
   e2e:
     environment:
       FLOW_CONTEXT: Production
+      DB_HOST: 127.0.0.1
     docker:
-      - image: cimg/php:8.0-node
+      - image: cimg/php:8.2-node
       - image: cimg/mariadb:10.6
         environment:
           MYSQL_DATABASE: neos
@@ -113,6 +114,8 @@ jobs:
           cd /home/circleci/app/Packages/Application/Neos.Neos.Ui
           nvm install
           nvm use
+          echo 127.0.0.1 onedimension.localhost | sudo tee -a /etc/hosts
+          echo 127.0.0.1 twodimensions.localhost | sudo tee -a /etc/hosts
           make test-e2e-saucelabs
       - store_artifacts:
           path: /home/circleci/app/Data/Logs
@@ -121,7 +124,7 @@ jobs:
     environment:
       FLOW_CONTEXT: Production
     docker:
-      - image: cimg/php:8.0-node
+      - image: cimg/php:8.2-node
       - image: cimg/mariadb:10.6
         environment:
           MYSQL_DATABASE: neos

Classes/ContentRepository/Service/WorkspaceService.php

@@ -13,7 +13,7 @@
 
 use Neos\ContentRepository\Core\ContentRepository;
 use Neos\ContentRepository\Core\Factory\ContentRepositoryId;
-use Neos\ContentRepository\Core\Projection\ContentGraph\ContentSubgraphIdentity;
+use Neos\ContentRepository\Core\Feature\WorkspacePublication\Command\DiscardIndividualNodesFromWorkspace;
 use Neos\ContentRepository\Core\Projection\ContentGraph\Node;
 use Neos\ContentRepository\Core\Projection\Workspace\Workspace;
 use Neos\Neos\FrontendRouting\NodeAddress;
@@ -24,15 +24,14 @@
 use Neos\Flow\Annotations as Flow;
 use Neos\Neos\Domain\Service\UserService as DomainUserService;
 use Neos\Neos\PendingChangesProjection\ChangeFinder;
-use Neos\Neos\PendingChangesProjection\ChangeProjection;
 use Neos\Neos\Service\UserService;
+use Neos\Neos\Ui\Domain\Model\Feedback\Operations\RemoveNode;
 
 /**
  * @Flow\Scope("singleton")
  */
 class WorkspaceService
 {
-
     /**
      * @Flow\Inject
      * @var UserService
@@ -156,6 +155,53 @@ public function getAllowedTargetWorkspaces(ContentRepository $contentRepository)
         return $workspacesArray;
     }
 
+    public function predictRemoveNodeFeedbackFromDiscardIndividualNodesFromWorkspaceCommand(
+        DiscardIndividualNodesFromWorkspace $command,
+        ContentRepository $contentRepository
+    ): array {
+        $workspace = $contentRepository->getWorkspaceFinder()->findOneByName($command->workspaceName);
+        if (is_null($workspace)) {
+            return Nodes::createEmpty();
+        }
+
+        $changeFinder = $contentRepository->projectionState(ChangeFinder::class);
+        $changes = $changeFinder->findByContentStreamId($workspace->currentContentStreamId);
+
+        $handledNodes = [];
+        $result = [];
+        foreach ($changes as $change) {
+            if ($change->created) {
+                foreach ($command->nodesToDiscard as $nodeToDiscard) {
+                    if (in_array($nodeToDiscard, $handledNodes)) {
+                        continue;
+                    }
+
+                    if (
+                        $nodeToDiscard->contentStreamId->equals($change->contentStreamId)
+                        && $nodeToDiscard->nodeAggregateId->equals($change->nodeAggregateId)
+                        && $nodeToDiscard->dimensionSpacePoint->equals($change->originDimensionSpacePoint)
+                    ) {
+                        $subgraph = $contentRepository->getContentGraph()
+                            ->getSubgraph(
+                                $nodeToDiscard->contentStreamId,
+                                $nodeToDiscard->dimensionSpacePoint,
+                                VisibilityConstraints::withoutRestrictions()
+                            );
+
+                        $childNode = $subgraph->findNodeById($nodeToDiscard->nodeAggregateId);
+                        $parentNode = $subgraph->findParentNode($nodeToDiscard->nodeAggregateId);
+                        if ($parentNode) {
+                            $result[] = new RemoveNode($childNode, $parentNode);
+                            $handledNodes[] = $nodeToDiscard;
+                        }
+                    }
+                }
+            }
+        }
+
+        return $result;
+    }
+
     private function getClosestDocumentNode(Node $node): ?Node
     {
         $subgraph = $this->contentRepositoryRegistry->subgraphForNode($node);

Classes/Controller/BackendServiceController.php

@@ -294,18 +294,27 @@ public function discardAction(array $nodeContextPaths): void
                     $nodeAddress->dimensionSpacePoint
                 );
             }
-            $contentRepository->handle(
-                DiscardIndividualNodesFromWorkspace::create(
-                    $workspaceName,
-                    NodeIdsToPublishOrDiscard::create(...$nodeIdentifiersToDiscard)
-                )
-            )->block();
+
+            $command = DiscardIndividualNodesFromWorkspace::create(
+                $workspaceName,
+                NodeIdsToPublishOrDiscard::create(...$nodeIdentifiersToDiscard)
+            );
+            $removeNodeFeedback = $this->workspaceService
+                ->predictRemoveNodeFeedbackFromDiscardIndividualNodesFromWorkspaceCommand(
+                    $command,
+                    $contentRepository
+                );
+
+            $contentRepository->handle($command)->block();
 
             $success = new Success();
             $success->setMessage(sprintf('Discarded %d node(s).', count($nodeContextPaths)));
 
             $updateWorkspaceInfo = new UpdateWorkspaceInfo($contentRepositoryId, $workspaceName);
             $this->feedbackCollection->add($success);
+            foreach ($removeNodeFeedback as $removeNode) {
+                $this->feedbackCollection->add($removeNode);
+            }
             $this->feedbackCollection->add($updateWorkspaceInfo);
         } catch (\Exception $e) {
             $error = new Error();

Classes/Domain/Model/ChangeInterface.php

@@ -13,7 +13,6 @@
  * source code.
  */
 
-
 use Neos\ContentRepository\Core\Projection\ContentGraph\Node;
 
 /**

Classes/Domain/Model/Changes/MoveAfter.php

@@ -88,9 +88,6 @@ public function apply(): void
             $updateParentNodeInfo->setNode($parentNodeOfPreviousSibling);
             $this->feedbackCollection->add($updateParentNodeInfo);
 
-            $removeNode = new RemoveNode($subject, $parentNodeOfPreviousSibling);
-            $this->feedbackCollection->add($removeNode);
-
             $this->finish($subject);
         }
     }

Classes/Domain/Model/Changes/MoveBefore.php

@@ -87,9 +87,6 @@ public function apply(): void
 
             $this->feedbackCollection->add($updateParentNodeInfo);
 
-            $removeNode = new RemoveNode($subject, $succeedingSiblingParent);
-            $this->feedbackCollection->add($removeNode);
-
             $this->finish($subject);
         }
     }

Classes/Domain/Model/Changes/MoveInto.php

@@ -95,9 +95,6 @@ public function apply(): void
             $updateParentNodeInfo->setNode($parentNode);
             $this->feedbackCollection->add($updateParentNodeInfo);
 
-            $removeNode = new RemoveNode($subject, $parentNode);
-            $this->feedbackCollection->add($removeNode);
-
             $this->finish($subject);
         }
     }

Classes/Domain/Model/Feedback/Operations/ReloadContentOutOfBand.php

@@ -132,7 +132,20 @@ protected function renderContent(ControllerContext $controllerContext): string|R
 
             if ($this->nodeDomAddress) {
                 $fusionView = new FusionView();
-                $fusionView->setControllerContext($controllerContext);
+
+                $fakeActionRequest = clone $controllerContext->getRequest();
+                $fakeActionRequest->setControllerPackageKey('Neos.Neos');
+                $fakeActionRequest->setControllerName('Frontend\\Node');
+                $fakeActionRequest->setControllerActionName('edit');
+
+                $fakeControllerContext = new ControllerContext(
+                    $fakeActionRequest,
+                    $controllerContext->getResponse(),
+                    $controllerContext->getArguments(),
+                    $controllerContext->getUriBuilder(),
+                );
+
+                $fusionView->setControllerContext($fakeControllerContext);
 
                 $fusionView->assign('value', $this->node);
                 $fusionView->setFusionPath($this->nodeDomAddress->getFusionPathForContentRendering());

Classes/Domain/Model/Feedback/Operations/RemoveNode.php

@@ -14,6 +14,7 @@
 use Neos\ContentRepositoryRegistry\ContentRepositoryRegistry;
 use Neos\Flow\Annotations as Flow;
 use Neos\Neos\FrontendRouting\NodeAddressFactory;
+use Neos\Neos\FrontendRouting\NodeAddress;
 use Neos\ContentRepository\Core\Projection\ContentGraph\Node;
 use Neos\Flow\Mvc\Controller\ControllerContext;
 use Neos\Neos\Ui\Domain\Model\AbstractFeedback;
@@ -25,6 +26,10 @@ class RemoveNode extends AbstractFeedback
 
     protected Node $parentNode;
 
+    private NodeAddress $nodeAddress;
+
+    private NodeAddress $parentNodeAddress;
+
     /**
      * @Flow\Inject
      * @var ContentRepositoryRegistry
@@ -37,6 +42,15 @@ public function __construct(Node $node, Node $parentNode)
         $this->parentNode = $parentNode;
     }
 
+    protected function initializeObject(): void
+    {
+        $contentRepository = $this->contentRepositoryRegistry->get($this->node->subgraphIdentity->contentRepositoryId);
+        $nodeAddressFactory = NodeAddressFactory::create($contentRepository);
+
+        $this->nodeAddress = $nodeAddressFactory->createFromNode($this->node);
+        $this->parentNodeAddress = $nodeAddressFactory->createFromNode($this->parentNode);
+    }
+
     public function getNode(): Node
     {
         return $this->node;
@@ -87,11 +101,9 @@ public function isSimilarTo(FeedbackInterface $feedback)
      */
     public function serializePayload(ControllerContext $controllerContext)
     {
-        $contentRepository = $this->contentRepositoryRegistry->get($this->node->subgraphIdentity->contentRepositoryId);
-        $nodeAddressFactory = NodeAddressFactory::create($contentRepository);
         return [
-            'contextPath' => $nodeAddressFactory->createFromNode($this->node)->serializeForUri(),
-            'parentContextPath' => $nodeAddressFactory->createFromNode($this->parentNode)->serializeForUri()
+            'contextPath' => $this->nodeAddress->serializeForUri(),
+            'parentContextPath' => $this->parentNodeAddress->serializeForUri()
         ];
     }
 }

Classes/Domain/Model/Feedback/Operations/RenderContentOutOfBand.php

@@ -178,7 +178,20 @@ protected function renderContent(ControllerContext $controllerContext): string|R
             $parentDomAddress = $this->getParentDomAddress();
             if ($parentDomAddress) {
                 $fusionView = new FusionView();
-                $fusionView->setControllerContext($controllerContext);
+
+                $fakeActionRequest = clone $controllerContext->getRequest();
+                $fakeActionRequest->setControllerPackageKey('Neos.Neos');
+                $fakeActionRequest->setControllerName('Frontend\\Node');
+                $fakeActionRequest->setControllerActionName('edit');
+
+                $fakeControllerContext = new ControllerContext(
+                    $fakeActionRequest,
+                    $controllerContext->getResponse(),
+                    $controllerContext->getArguments(),
+                    $controllerContext->getUriBuilder(),
+                );
+
+                $fusionView->setControllerContext($fakeControllerContext);
 
                 $fusionView->assign('value', $parentNode);
                 $fusionView->setFusionPath($parentDomAddress->getFusionPath());

Classes/Domain/Model/ReferencingChangeInterface.php

@@ -13,7 +13,6 @@
 
 use Neos\ContentRepository\Core\Projection\ContentGraph\Node;
 
-
 /**
  * A change that needs to reference another node
  */

Classes/Domain/Service/NodePropertyConverterService.php

@@ -144,7 +144,7 @@ public function getProperty(Node $node, $propertyName)
             );
 
             return $this->toNodeIdentifierStrings($references);
-        // Here, the normal property access logic starts.
+            // Here, the normal property access logic starts.
         } elseif ($propertyName[0] === '_' && $propertyName !== '_hiddenInIndex') {
             $propertyValue = ObjectAccess::getProperty($node, ltrim($propertyName, '_'));
         } else {

Classes/Fusion/Helper/NodeInfoHelper.php

@@ -211,7 +211,7 @@ protected function getUriInformation(Node $node, ControllerContext $controllerCo
         if (!$node->nodeType->isOfType($this->documentNodeTypeRole)) {
             return $nodeInfo;
         }
-        $nodeInfo['uri'] = $this->previewUri($node, $controllerContext);
+        $nodeInfo['uri'] = $this->editUri($node, $controllerContext);
         return $nodeInfo;
     }
 
@@ -281,8 +281,8 @@ protected function renderChildrenInformation(Node $node, string $nodeTypeFilterS
         // child nodes for content tree, must not include those nodes filtered out by `baseNodeType`
         $contentChildNodes = $subgraph->findChildNodes(
             $node->nodeAggregateId,
-            FindChildNodesFilter::create(nodeTypeConstraints:
-                $this->buildContentChildNodeFilterString()
+            FindChildNodesFilter::create(
+                nodeTypeConstraints: $this->buildContentChildNodeFilterString()
             )
         );
         $childNodes = $documentChildNodes->merge($contentChildNodes);
@@ -442,6 +442,14 @@ public function uri(Node|NodeAddress $nodeAddress, ControllerContext $controller
         return (string)NodeUriBuilder::fromRequest($controllerContext->getRequest())->uriFor($nodeAddress);
     }
 
+    public function editUri(Node $node, ControllerContext $controllerContext): string
+    {
+        $contentRepository = $this->contentRepositoryRegistry->get($node->subgraphIdentity->contentRepositoryId);
+        $nodeAddressFactory = NodeAddressFactory::create($contentRepository);
+        $nodeAddress = $nodeAddressFactory->createFromNode($node);
+        return (string)NodeUriBuilder::fromRequest($controllerContext->getRequest())->editUriFor($nodeAddress);
+    }
+
     public function previewUri(Node $node, ControllerContext $controllerContext): string
     {
         $contentRepository = $this->contentRepositoryRegistry->get($node->subgraphIdentity->contentRepositoryId);

Configuration/Settings.yaml

@@ -137,7 +137,7 @@ Neos:
             personalWorkspace: '${Neos.Ui.Workspace.getPersonalWorkspace(contentRepositoryId)}'
         ui:
           contentCanvas:
-            src: '${Neos.Ui.NodeInfo.uri(documentNode, controllerContext)}'
+            src: '${Neos.Ui.NodeInfo.editUri(documentNode, controllerContext)}'
             backgroundColor: '${Configuration.setting(''Neos.Neos.Ui.contentCanvas.backgroundColor'')}'
           debugMode: false
           editPreviewMode: '${q(user).property("preferences.preferences")["contentEditing.editPreviewMode"] || Configuration.setting(''Neos.Neos.userInterface.defaultEditPreviewMode'')}'

Resources/Private/Fusion/Prototypes/Page.fusion

@@ -19,7 +19,7 @@ prototype(Neos.Neos:Page) {
 
             @process.json = ${Json.stringify(value)}
             @process.wrapInJsObject = ${'<script>window[\'@Neos.Neos.Ui:DocumentInformation\']=' + value + '</script>'}
-            @if.inBackend = ${Neos.Node.inBackend(documentNode)}
+            @if.inBackend = ${Neos.Backend.isEditMode(request) || Neos.Backend.isPreviewMode(request)}
 
             // We need to ensure the JS backend information is always up to date, especially
             // when child nodes change. Otherwise errors like the following might happen:
@@ -35,11 +35,11 @@ prototype(Neos.Neos:Page) {
                 entryIdentifier {
                     jsBackendInfo = 'javascriptBackendInformation'
                     documentNode = ${Neos.Caching.entryIdentifierForNode(documentNode)}
-                    inBackend = ${Neos.Node.inBackend(documentNode)}
+                    inBackend = ${Neos.Backend.isEditMode(request) || Neos.Backend.isPreviewMode(request)}
                 }
                 entryTags {
                     1 = ${Neos.Caching.nodeTag(documentNode)}
-                    2 = ${Neos.Node.inBackend(documentNode) ? Neos.Caching.descendantOfTag(documentNode) : null}
+                    2 = ${(Neos.Backend.isEditMode(request) || Neos.Backend.isPreviewMode(request)) ? Neos.Caching.descendantOfTag(documentNode) : null}
                 }
             }
         }
@@ -51,18 +51,18 @@ prototype(Neos.Neos:Page) {
             compiledResourcePackage = ${Neos.Ui.StaticResources.compiledResourcePackage()}
 
             sectionName = 'guestFrameApplication'
-            @if.inBackend = ${Neos.Node.inBackend(documentNode)}
+            @if.inBackend = ${Neos.Backend.isEditMode(request) || Neos.Backend.isPreviewMode(request)}
         }
     }
 
     neosBackendContainer = '<div id="neos-backend-container"></div>'
     neosBackendContainer.@position = 'before closingBodyTag'
-    [email protected] = ${Neos.Node.inBackend(documentNode)}
+    [email protected] = ${Neos.Backend.isEditMode(request) || Neos.Backend.isPreviewMode(request)}
 
     neosBackendNotification = Neos.Fusion:Template {
         @position = 'before closingBodyTag'
         templatePath = 'resource://Neos.Neos.Ui/Private/Templates/Backend/GuestNotificationScript.html'
-        @if.inBackend = ${Neos.Node.inBackend(documentNode)}
+        @if.inBackend = ${Neos.Backend.isEditMode(request) || Neos.Backend.isPreviewMode(request)}
     }
 
     @exceptionHandler = 'Neos\\Neos\\Ui\\Fusion\\ExceptionHandler\\PageExceptionHandler'

Resources/Private/Fusion/Prototypes/Shortcut.fusion

@@ -9,7 +9,7 @@ prototype(Neos.Neos:Page) {
             }
 
             @if {
-                inBackend = ${Neos.Node.inBackend(documentNode)}
+                inBackend = ${Neos.Backend.isEditMode(request) || Neos.Backend.isPreviewMode(request)}
                 isShortcut = ${q(node).is('[instanceof Neos.Neos:Shortcut]')}
             }
         }