Skip to content

Commit

Permalink
BUGFIX: Add content element wrapping to restricted nodes (#3888)
Browse files Browse the repository at this point in the history
Previously opening a document where some nodes were not permitted to be edited was hard to navigate, as the content element wrapping was not rendered. Clicking into the elements showed no outline and navigation via the content tree was not possible.
But more importantly inserting nodes into the content collection were likely misplaced until reloading as the Neos Ui does not understand the structure of the site.

Instead - as reading the nodes and its information is permitted and needed for rendering - we will always wrap content elements if editable or not.

To avoid having to boot the ckeditor for inline elements we instead evaluate the edit permission there and decide to hide the inline element wrapping. (Previously they would be rendered but not booted because the neos ui didnt had the outer content element wrapping)
  • Loading branch information
mhsdesign authored Dec 6, 2024
1 parent 1b6f722 commit 0e6c71c
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion Classes/Aspects/AugmentationAspect.php
Original file line number Diff line number Diff line change
Expand Up @@ -145,6 +145,10 @@ public function editableElementAugmentation(JoinPointInterface $joinPoint)
return $content;
}

if ($this->nodeAuthorizationService->isGrantedToEditNode($node) === false) {
return $content;
}

$content = $joinPoint->getAdviceChain()->proceed($joinPoint);

$attributes = [
Expand All @@ -166,6 +170,6 @@ protected function needsMetadata(NodeInterface $node, $renderCurrentDocumentMeta
/** @var $contentContext ContentContext */
$contentContext = $node->getContext();

return ($contentContext->isInBackend() === true && ($renderCurrentDocumentMetadata === true || $this->nodeAuthorizationService->isGrantedToEditNode($node) === true));
return $contentContext->isInBackend() === true || $renderCurrentDocumentMetadata === true;
}
}

0 comments on commit 0e6c71c

Please sign in to comment.