Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NodeSearchService removed #43

Open
pKallert opened this issue Jan 31, 2024 · 2 comments · Fixed by #99 · May be fixed by #119
Open

NodeSearchService removed #43

pKallert opened this issue Jan 31, 2024 · 2 comments · Fixed by #99 · May be fixed by #119
Assignees

Comments

@pKallert
Copy link

In neos/neos-development-collection#4555 (comment) we removed the NodeSearchService. There was also a discussion about re-adding the Service, but was decided against: neos/neos-development-collection#3771

There are Upgrade instructions in the PR to migrate, but we should think about adding a comment or hint directly using rector.

@dlubitz dlubitz mentioned this issue Feb 2, 2024
9 tasks
@dlubitz
Copy link
Contributor

dlubitz commented Nov 21, 2024

- $nodes = $this->nodeSearchService->findByProperties(string $term, array $searchNodeTypes);
+ $nodes = $this->subgraph->findDescendantNodes(
+     $entryNode->nodeAggregateId,
+     FindDescendantNodesFilter::create(
+         nodeTypes: NodeTypeConstraints::create(
+             NodeTypeNames::fromStringArray($searchNodeTypes),
+             NodeTypeNames::createEmpty()
+         ),
+         searchTerm: $term,
+     )
+ );

@dlubitz
Copy link
Contributor

dlubitz commented Dec 13, 2024

We also need to check for the interface: Neos\Neos\Domain\Service\NodeSearchServiceInterface

@dlubitz dlubitz reopened this Dec 13, 2024
@dlubitz dlubitz self-assigned this Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants