Task #88741

Epic #87193: Remove deprecated code from the core

Remove cHash calculation in indexed search

Added by Benni Mack 11 months ago. Updated 10 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.


Related issues

Related to TYPO3 Core - Bug #79108: Indexed Search cHash calculation error when indexing a single record Closed 2016-12-29

Associated revisions

Revision 0c531014 (diff)
Added by Benni Mack 11 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 11 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 11 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 11 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 11 months ago

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

#5 Updated by Benni Mack 10 months ago

  • Status changed from Resolved to Closed

#6 Updated by Susanne Moog 3 months ago

  • Related to Bug #79108: Indexed Search cHash calculation error when indexing a single record added

Also available in: Atom PDF