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

[!!!][FEATURE] Rework Search Component system #3676

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

bmack
Copy link
Collaborator

@bmack bmack commented Jun 5, 2023

All Search Components have been reworked and the Component system have been moved to a PSR-14 Event
AfterSearchQueryHasBeenPreparedEvent.

This way, the registration works just as other PSR-14 events, and Dependency Injection is now used consistently across all Componenents.

The Component System is now merged with the previous hook $GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['solr']['modifySearchQuery'] and the ApacheSolrForTypo3\Solr\Search\Modifier interface.

Two other hooks have been migrated as well.

Tests who actually tested the registration have been removed, further existing tests have been adapted.

Fixes: #3675

All Search Components have been reworked and the Component
system have been moved to a PSR-14 Event
`AfterSearchQueryHasBeenPreparedEvent`.

This way, the registration works just as other PSR-14 events,
and Dependency Injection is now used consistently across all
Componenents.

The Component System is now merged with the previous hook
`$GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['solr']['modifySearchQuery']`
and the `ApacheSolrForTypo3\Solr\Search\Modifier` interface.

Two other hooks have been migrated as well.

Tests who actually tested the registration have been removed,
further existing tests have been adapted.

Fixes: TYPO3-Solr#3675
Copy link
Collaborator

@dkd-kaehm dkd-kaehm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@dkd-kaehm dkd-kaehm merged commit e8cd090 into TYPO3-Solr:main Jun 5, 2023
5 checks passed
@dkd-kaehm dkd-kaehm mentioned this pull request Oct 13, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TASK] Move SearchComponents to PSR-14 events
2 participants