Bug #64716

indexed search broken after moving "SearchResultContentObject" to compatibility6 extension

Added by Tymoteusz Motylewski over 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2015-02-01
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Indexed search still uses the old cobject in two places:

typo3/sysext/indexed_search/Classes/Controller/SearchController.php
and
typo3/sysext/indexed_search/Classes/Controller/SearchFormController.php

just to call one method from SearchResultContentObject->register_and_explode_search_string()

we should copy the function to indexed search, so we don't depend on deprecated cobject


Related issues

Related to TYPO3 Core - Task #64696: Move legacy "search" CType to compatibility6 Closed 2015-01-31

Associated revisions

Revision 61fd8e12 (diff)
Added by Benni Mack over 4 years ago

[BUGFIX] Indexed search broken after moving "SearchResultContentObject"

Indexed search still uses the old cobject in both
plugins.

The problem arieses in just to call one method from
SearchResultContentObject->register_and_explode_search_string()

The necessary functions are moved to the common
IndexedSearchUtility.

Resolves: #64716
Releases: master
Change-Id: I386330f1f66342ee838e3616cac7f0924bd31522
Reviewed-on: http://review.typo3.org/36566
Reviewed-by: Tymoteusz Motylewski <>
Tested-by: Tymoteusz Motylewski <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review over 4 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36566

#2 Updated by Gerrit Code Review over 4 years ago

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

#3 Updated by Benni Mack over 4 years ago

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

#4 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF