-
-
Notifications
You must be signed in to change notification settings - Fork 218
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
FEATURE: Improve flowQuery
find and children operation
The flowQuery operations find and children are fixed optimized for the new cr to use a combined query for nodetype and property criteria. The number of performed db-queries is the number of contextNodes times the number of filterGroups (comma separated parts) In addition the `find` operation now can also handle single property criteria and does not rely on having an instanceof filter first.
- Loading branch information
Showing
7 changed files
with
349 additions
and
118 deletions.
There are no files selected for viewing
16 changes: 16 additions & 0 deletions
16
Neos.ContentRepository.NodeAccess/Classes/Filter/NodeFilterCriteria.php
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| <?php | ||
|
|
||
| declare(strict_types=1); | ||
|
|
||
| namespace Neos\ContentRepository\NodeAccess\Filter; | ||
|
|
||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\NodeType\NodeTypeCriteria; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueCriteriaInterface; | ||
|
|
||
| readonly class NodeFilterCriteria | ||
| { | ||
| public function __construct( | ||
| public ?NodeTypeCriteria $nodeTypeCriteria = null, | ||
| public ?PropertyValueCriteriaInterface $propertyValueCriteria = null) { | ||
| } | ||
| } |
32 changes: 32 additions & 0 deletions
32
Neos.ContentRepository.NodeAccess/Classes/Filter/NodeFilterCriteriaGroup.php
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| <?php | ||
|
|
||
| declare(strict_types=1); | ||
|
|
||
| namespace Neos\ContentRepository\NodeAccess\Filter; | ||
|
|
||
| use Traversable; | ||
|
|
||
| /** | ||
| * @implements \IteratorAggregate<int, NodeFilterCriteria> | ||
| */ | ||
| readonly class NodeFilterCriteriaGroup implements \IteratorAggregate | ||
| { | ||
| /** | ||
| * @var array<int, NodeFilterCriteria> | ||
| */ | ||
| protected array $criteria; | ||
|
|
||
| public function __construct(NodeFilterCriteria ...$criteria) | ||
| { | ||
| $this->criteria = array_values($criteria); | ||
| } | ||
|
|
||
| /** | ||
| * @return Traversable<int, NodeFilterCriteria> | ||
| */ | ||
| public function getIterator(): Traversable | ||
| { | ||
| return new \ArrayIterator($this->criteria); | ||
| } | ||
|
|
||
| } |
125 changes: 125 additions & 0 deletions
125
Neos.ContentRepository.NodeAccess/Classes/Filter/NodeFilterCriteriaGroupFactory.php
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,125 @@ | ||
| <?php | ||
|
|
||
| declare(strict_types=1); | ||
|
|
||
| namespace Neos\ContentRepository\NodeAccess\Filter; | ||
|
|
||
| use Neos\ContentRepository\Core\NodeType\NodeTypeName; | ||
| use Neos\ContentRepository\Core\NodeType\NodeTypeNames; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\NodeType\NodeTypeCriteria; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\AndCriteria; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\NegateCriteria; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueContains; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueCriteriaInterface; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueEndsWith; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueEquals; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueGreaterThan; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueGreaterThanOrEqual; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueLessThan; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueLessThanOrEqual; | ||
| use Neos\ContentRepository\Core\Projection\ContentGraph\Filter\PropertyValue\Criteria\PropertyValueStartsWith; | ||
| use Neos\ContentRepository\Core\SharedModel\Node\PropertyName; | ||
| use Neos\Eel\FlowQuery\FizzleParser; | ||
|
|
||
| readonly class NodeFilterCriteriaGroupFactory | ||
| { | ||
| public static function createFromFizzleExpressionString (string $fizzleExpression): ?NodeFilterCriteriaGroup | ||
| { | ||
| $parsedFilter = FizzleParser::parseFilterGroup($fizzleExpression); | ||
| return self::createFromParsedFizzleExpression($parsedFilter); | ||
| } | ||
|
|
||
| /** | ||
| * @param mixed[] $parsedFizzleExpression | ||
| */ | ||
| public static function createFromParsedFizzleExpression (array $parsedFizzleExpression): ?NodeFilterCriteriaGroup | ||
| { | ||
| $filterCriteria = []; | ||
| if (is_array($parsedFizzleExpression) | ||
| && array_key_exists('name', $parsedFizzleExpression) && $parsedFizzleExpression['name'] === 'FilterGroup' | ||
| && array_key_exists('Filters', $parsedFizzleExpression) && is_array($parsedFizzleExpression['Filters']) | ||
| ) { | ||
| foreach ($parsedFizzleExpression['Filters'] as $filter) { | ||
| // anything but AttributeFilters yield a null result | ||
| if (array_key_exists('PathFilter', $filter) | ||
| || array_key_exists('PropertyNameFilter', $filter) | ||
| || array_key_exists('IdentifierFilter', $filter) | ||
| ) { | ||
| return null; | ||
| } | ||
| if (array_key_exists('AttributeFilters', $filter) && is_array ($filter['AttributeFilters'])) { | ||
|
|
||
| $allowedNodeTypeNames = NodeTypeNames::createEmpty(); | ||
| $disallowedNodeTypeNames = NodeTypeNames::createEmpty(); | ||
|
|
||
| /** | ||
| * @var PropertyValueCriteriaInterface[] | ||
| */ | ||
| $propertyCriteria = []; | ||
| foreach ($filter['AttributeFilters'] as $attributeFilter) { | ||
| $propertyPath = $attributeFilter['PropertyPath'] ?? null; | ||
| $operator = $attributeFilter['Operator'] ?? null; | ||
| $operand = $attributeFilter['Operand'] ?? null; | ||
| switch ($operator) { | ||
| case 'instanceof': | ||
| $allowedNodeTypeNames = $allowedNodeTypeNames->withAdditionalNodeTypeName(NodeTypeName::fromString($operand)); | ||
| break; | ||
| case '!instanceof': | ||
| $disallowedNodeTypeNames = $disallowedNodeTypeNames->withAdditionalNodeTypeName(NodeTypeName::fromString($operand)); | ||
| break; | ||
| case '=': | ||
| $propertyCriteria[] = PropertyValueEquals::create(PropertyName::fromString($propertyPath), $operand); | ||
| break; | ||
| case '!=': | ||
| $propertyCriteria[] = NegateCriteria::create(PropertyValueEquals::create(PropertyName::fromString($propertyPath), $operand)); | ||
| break; | ||
| case '^=': | ||
| $propertyCriteria[] = PropertyValueStartsWith::create(PropertyName::fromString($propertyPath), $operand); | ||
| break; | ||
| case '$=': | ||
| $propertyCriteria[] = PropertyValueEndsWith::create(PropertyName::fromString($propertyPath), $operand); | ||
| break; | ||
| case '*=': | ||
| $propertyCriteria[] = PropertyValueContains::create(PropertyName::fromString($propertyPath), $operand); | ||
| break; | ||
| case '>': | ||
| $propertyCriteria[] = PropertyValueGreaterThan::create(PropertyName::fromString($propertyPath), $operand); | ||
| break; | ||
| case '>=': | ||
| $propertyCriteria[] = PropertyValueGreaterThanOrEqual::create(PropertyName::fromString($propertyPath), $operand); | ||
| break; | ||
| case '<': | ||
| $propertyCriteria[] = PropertyValueLessThan::create(PropertyName::fromString($propertyPath), $operand); | ||
| break; | ||
| case '<=': | ||
| $propertyCriteria[] = PropertyValueLessThanOrEqual::create(PropertyName::fromString($propertyPath), $operand); | ||
| break; | ||
| default: | ||
| return null; | ||
| } | ||
| } | ||
|
|
||
| if (count($propertyCriteria) > 1) { | ||
| $propertyCriteriaCombined = array_shift($propertyCriteria); | ||
| while ($other = array_shift($propertyCriteria)) { | ||
| $propertyCriteriaCombined = AndCriteria::create($propertyCriteriaCombined, $other); | ||
| } | ||
| } elseif (count($propertyCriteria) == 1) { | ||
| $propertyCriteriaCombined = $propertyCriteria[0]; | ||
| } else { | ||
| $propertyCriteriaCombined = null; | ||
| } | ||
|
|
||
| $filterCriteria[] = new NodeFilterCriteria( | ||
| ($allowedNodeTypeNames->isEmpty() && $disallowedNodeTypeNames->isEmpty()) ? null : NodeTypeCriteria::create($allowedNodeTypeNames, $disallowedNodeTypeNames), | ||
| $propertyCriteriaCombined | ||
| ); | ||
| } else { | ||
| return null; | ||
| } | ||
| } | ||
| return new NodeFilterCriteriaGroup(...$filterCriteria); | ||
| } | ||
| return null; | ||
| } | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.