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

[!!!][TASK] Remove deprecated functionality #3585

Merged
merged 1 commit into from
May 30, 2023

Conversation

bmack
Copy link
Collaborator

@bmack bmack commented Mar 31, 2023

What this pr does

This change removes deprecated features of EXT:solr

a) PageDocumentPostProcessor

The interface PageDocumentPostProcessor
is removed, along with the registration
in ext_localconf.php.

The PostProcessor is now removed in favor
of the PageIndexerDocumentsModifier interface,
and can be used already in EXT:solr v11.

Check in your extensions' ext_localconf.php
and look for
$GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['solr']['Indexer']['indexPagePostProcessPageDocument'] to see if you are affected.

b) UrlHelper methods

In addition, some rather unintrusive, deprecated
methods in UrlHelper have been removed
as UrlHelper is just a small
extension for UriInterface, and might
vanish altogether in the future.

Relates: #3376

Todo:

  • Rebase and change to main as target branch

@dkd-friedrich
Copy link
Member

dkd-friedrich commented Apr 3, 2023

Good idea to remove the deprecated functions, but I see further steps that are necessary in this process:

  • Section "indexPagePostProcessPageDocument" should be removed from documentation (Documentation/Development/Indexing.rst)
  • InvalidArgumentException include can be removed as no longer used in UrlHelper
  • AbstractUriStrategy is still using deprecated methods (setPath, setScheme, setPort, setHost) of the UrlHelper and must be adapted
  • UrlHelperTest is still using deprecated methods (getUrl, removeQueryParameter, setPath, setScheme, setPort, setHost) of the UrlHelper and must be adapted

@bmack Do you have time for the adaptions or should I take care of this?

@dkd-kaehm dkd-kaehm force-pushed the task/3376-TYPO3_12_compatibility branch 2 times, most recently from b846714 to 6cffa97 Compare April 8, 2023 15:44
@dkd-kaehm dkd-kaehm force-pushed the task/3376-TYPO3_12_compatibility branch from e7e2e9d to e9e7248 Compare May 15, 2023 13:07
@dkd-kaehm dkd-kaehm changed the base branch from task/3376-TYPO3_12_compatibility to main May 16, 2023 06:27
@dkd-kaehm dkd-kaehm force-pushed the deprecated-code-v12 branch 2 times, most recently from e2f77ba to 2fb69cc Compare May 16, 2023 07:21
This change removes deprecated features of EXT:solr

a) PageDocumentPostProcessor

The interface PageDocumentPostProcessor
is removed, along with the registration
in ext_localconf.php.

The PostProcessor is now removed in favor
of the PageIndexerDocumentsModifier interface,
and can be used already in EXT:solr v11.

Check in your extensions' ext_localconf.php
and look for
`$GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['solr']['Indexer']['indexPagePostProcessPageDocument']`
to see if you are affected.

In addition, some rather unintrusive, deprecated
methods in UrlHelper have been removed
as UrlHelper is just a small
extension for UriInterface, and might
vanish altogether in the future.

Relates: TYPO3-Solr#3376
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 192664d into TYPO3-Solr:main May 30, 2023
6 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.

None yet

3 participants