Merge pull request #68 from sitegeist/tetheredNodes

Properly handle tethered nodes
sitegeist · Nov 17, 2023 · 3e42990 · 3e42990
2 parents da9ba82 + 48fa93a
commit 3e42990
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 11 deletions.
diff --git a/Classes/Command/TaxonomyCommandController.php b/Classes/Command/TaxonomyCommandController.php
@@ -74,7 +74,6 @@ public function taxonomiesCommand(string $vocabulary, string $path = ''): void
         if (!$startPoint) {
             $this->outputLine('nothing found');
             $this->quit(1);
-            throw new StopCommandException();
         }
 
         $subtree = $this->taxonomyService->findSubtree($startPoint);

diff --git a/Classes/Service/TaxonomyService.php b/Classes/Service/TaxonomyService.php
@@ -21,8 +21,8 @@
 use Neos\ContentRepository\Core\NodeType\NodeTypeName;
 use Neos\ContentRepository\Core\NodeType\NodeTypeNames;
 use Neos\ContentRepository\Core\Projection\ContentGraph\ContentSubgraphInterface;
-use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\FindAncestorNodesFilter;
 use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\FindChildNodesFilter;
+use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\FindClosestNodeFilter;
 use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\FindSubtreeFilter;
 use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\NodeType\NodeTypeCriteria;
 use Neos\ContentRepository\Core\Projection\ContentGraph\Node;
@@ -81,18 +81,18 @@ public function findVocabularyForNode(Node $node): Node
     {
         $subgraph = $this->crRegistry->subgraphForNode($node);
 
-        $ancestors = $subgraph->findAncestorNodes(
+        $vocabularyNode = $subgraph->findClosestNode(
             $node->nodeAggregateId,
-            FindAncestorNodesFilter::create(
+            FindClosestNodeFilter::create(
                 nodeTypes: NodeTypeCriteria::create(
                     NodeTypeNames::fromArray([ $this->getVocabularyNodeTypeName()]),
                     NodeTypeNames::createEmpty()
                 )
             )
         );
 
-        if ($result = $ancestors->first()) {
-            return $result;
+        if ($vocabularyNode) {
+            return $vocabularyNode;
         }
 
         throw new \InvalidArgumentException('node seems to be outside of vocabulary');

diff --git a/Resources/Private/Fusion/Backend/Form/Taxonomy.fusion b/Resources/Private/Fusion/Backend/Form/Taxonomy.fusion
@@ -26,7 +26,12 @@ prototype(Sitegeist.Taxonomy:Form.Taxonomy) < prototype(Neos.Fusion:Component) {
                         {props.i18nTaxonomy.id('properties.name')}
                         <span @if={props.defaultTaxonomy}>: {props.defaultTaxonomy.properties.name}</span>
                     </label>
-                    <Neos.Fusion.Form:Input attributes.class="neos-span6" field.name="name" attributes.required attributes.pattern="[a-z0-9]+" />
+                    <Neos.Fusion.Form:Input
+                        attributes.class="neos-span6"
+                        field.name="name"
+                        attributes.required attributes.pattern="[a-z0-9]+"
+                        attributes.disabled="1" attributes.disabled.@if={props.taxonomyNode.classification.value == 'tethered'}
+                    />
                 </div>
 
                 <div class="neos-control-group">

diff --git a/Resources/Private/Fusion/Backend/Form/Vocabulary.fusion b/Resources/Private/Fusion/Backend/Form/Vocabulary.fusion
@@ -24,7 +24,13 @@ prototype(Sitegeist.Taxonomy:Form.Vocabulary) < prototype(Neos.Fusion:Component)
                     <label class="neos-control-label" for="name">
                         {props.i18nVocabulary.id('properties.name')}
                     </label>
-                    <Neos.Fusion.Form:Input attributes.class="neos-span6" field.name="name" attributes.required attributes.pattern="[a-z0-9]+" />
+                    <Neos.Fusion.Form:Input
+                        attributes.class="neos-span6"
+                        field.name="name"
+                        attributes.required attributes.pattern="[a-z0-9]+"
+                        attributes.readonly="1" attributes.readonly.@if={props.vocabularyNode.classification.value == 'tethered'}
+
+                    />
                 </div>
 
                 <div class="neos-control-group">

diff --git a/Resources/Private/Fusion/Backend/Views/Taxonomy.List.fusion b/Resources/Private/Fusion/Backend/Views/Taxonomy.List.fusion
@@ -107,7 +107,7 @@ prototype(Sitegeist.Taxonomy:Views.Module.Taxonomy.List.Item) < prototype(Neos.F
                             <i class="fas fa-pencil-alt"></i>
                         </Neos.Fusion:Link.Action>
 
-                        <Neos.Fusion:Fragment @if={!props.taxonomySubtree.autoCreated}>
+                        <Neos.Fusion:Fragment @if={props.taxonomySubtree.node.classification.value != 'tethered'}>
                             &nbsp;
                             <button class="neos-button neos-button-danger"
                                 title={props.i8n.id('generic.delete')}

diff --git a/Resources/Private/Fusion/Backend/Views/Vocabulary.List.fusion b/Resources/Private/Fusion/Backend/Views/Vocabulary.List.fusion
@@ -57,7 +57,7 @@ prototype(Sitegeist.Taxonomy:Views.Module.Vocabulary.List) < prototype(Neos.Fusi
                                         <i class="fas fa-pencil-alt"></i>
                                     </Neos.Fusion:Link.Action>
 
-                                    <Neos.Fusion:Fragment @if={!vocabulary.node.autoCreated} >
+                                    <Neos.Fusion:Fragment @if={vocabulary.classification.value != 'tethered'} >
                                         &nbsp;
                                         <button class="neos-button neos-button-danger" title={props.i18n.id('generic.delete')} data-toggle="modal" href={'#deleteVocabulary_' + vocabulary.nodeAggregateId.value} data-neos-toggle="tooltip">
                                             <i class="fas fa-trash-alt icon-white"></i>
@@ -68,7 +68,7 @@ prototype(Sitegeist.Taxonomy:Views.Module.Vocabulary.List) < prototype(Neos.Fusi
 
                             </div>
 
-                            <Neos.Fusion:Fragment @if={!vocabulary.node.autoCreated} >
+                            <Neos.Fusion:Fragment @if={vocabulary.classification.value != 'tethered'} >
                                 <div class="neos-hide" id={'deleteVocabulary_' + vocabulary.nodeAggregateId.value}>
                                     <div class="neos-modal-centered">
                                         <div class="neos-modal-content">