Task #88741

Epic #87193: Remove deprecated code from the core

Remove cHash calculation in indexed search

Added by Benni Mack 4 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Indexed Search
Start date:
2019-07-13
Due date:
% Done:

100%

TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

cHash calculation is not needed as it's all done by link generation now.

Associated revisions

Revision 0c531014 (diff)
Added by Benni Mack 4 months ago

[!!!][TASK] Remove cHash calculation in indexed search

cHash is now added automatically when a URL is generated.
The relevant query parameters when indexing
are stored in "static_page_arguments", which allows
to remove the database field "cHashParams".

Therefor it is not necessary anymore to configure if
cHash should be taken into account when creating
a configuration for indexed search.

Instead, when linking to a page on a search result,
the page arguments are added. In addition, when
pages are indexed, only the static page arguments
are evaluated.

Debug Information when indexing is also adding
data more sensibly via json_encode/decode.

Effectively, this means that specific handling
for cHash resolving is fully removed from
EXT:indexed_search.

Related: #87193
Resolves: #88741
Releases: master
Change-Id: I84738612d42615a3ac24d271c5509b52467d81af
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/61284
Tested-by: Anja Leichsenring <>
Tested-by: TYPO3com <>
Tested-by: Andreas Fernandez <>
Reviewed-by: Anja Leichsenring <>
Reviewed-by: Andreas Fernandez <>

History

#1 Updated by Gerrit Code Review 4 months ago

  • Status changed from New to Under Review

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61284

#2 Updated by Gerrit Code Review 4 months ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61284

#3 Updated by Gerrit Code Review 4 months ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61284

#4 Updated by Benni Mack 4 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#5 Updated by Benni Mack 4 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF