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

Replace node with endpoint #3665

Closed
wants to merge 8 commits into from

Conversation

froemken
Copy link
Contributor

@froemken froemken commented Jun 2, 2023

What this pr does

Replace Node with Endpoint

Fixes: #3043

sfroemkenjw and others added 6 commits June 2, 2023 12:14
* Removes last fragments of csh usage
* Removes CSH registration from ext_tables.php
* Removes unused imports from ext_tables.php

Fixes: TYPO3-Solr#3488
Relates: TYPO3-Solr#3376
$GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['solr']['Indexer']['indexPageSubstitutePageDocument']
is removed in favor of a new PSR-14 event:

"ApacheSolrForTypo3\Solr\Event\Indexing\AfterPageDocumentIsCreatedForIndexingEvent"

For this reason, the SubstitutePageIndexer Interface is removed.

At the same time, the "Typo3PageIndexer" class is now merged
into FrontendHelper/PageIndexer to avoid cross-cutting conerns
between classes and to show the dependencies to TSFE properly.
The UriStrategy concept was needed to generate
Frontend URLs in order to index a frontend page.

This concept was used in TYPO3 v9 to support site-based
routing and non-site-based projects.

This has now been removed as TYPO3 Core v11+ does only support
site-based frontend page resolving.

For this reason, the "UriStrategyFactory" and all the handling
has been removed, and the "TYPO3SiteStrategy" has been renamed
to "PageUriBuilder".

The hook logic to modify the indexing URL
`$GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['solr']['IndexQueuePageIndexer']['dataUrlModifier']`
and the according interface "PageIndexerDataUrlModifier" has been
removed and replaced by a new PSR-14 event:
`ApacheSolrForTypo3\Solr\Event\Indexing\AfterFrontendPageUriForIndexingHasBeenGeneratedEvent".

Usages of UriHelper have been reduced in favor of PSR-7 UriInterface.

IntegrationTests for the new Event have been added.
More hooks have been removed:

The hook :php:`$GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['solr']['postProcessIndexQueueInitialization']` and its
interface :php:`ApacheSolrForTypo3\Solr\IndexQueue\InitializationPostProcessor` are now superseded
by the PSR-14 event :php:`ApacheSolrForTypo3\Solr\Event\IndexQueue\AfterIndexQueueHasBeenInitializedEvent`

The hook :php:`$GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['solr']['postProcessFetchRecordsForIndexQueueItem']` is now superseded
by the PSR-14 event :php:`ApacheSolrForTypo3\Solr\Event\IndexQueue\AfterRecordsForIndexQueueItemsHaveBeenRetrievedEvent`

Tests have been added and documentation has been adapted.

Fixes: TYPO3-Solr#3666
…Documents"

* Uses ModuleProvider to check access to modules
* Removes unused imports
* Uses `f:be.link` for re-queue action in solr info module

Fixes: TYPO3-Solr#3580
Relates: TYPO3-Solr#3376
@dkd-friedrich
Copy link
Member

Branch has to be rebased, as maintainers currently cannot update the remote branch I've pushed the rebased changeset in another branch and opened a new PR: #3803

I'm closing this PR in favor of the new pull request.

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.

[BUG] solr_path_read does not work properly
4 participants