Bug #58567

Error in loadIndexerClass()

Added by Gerhard Rupp about 1 year ago. Updated about 1 year ago.

Status:Resolved Start date:2014-05-06
Priority:Must have Due date:
Assigned To:- % Done:

100%

Category:Indexed Search Spent time: -
Target version:-
TYPO3 Version:6.2 Is Regression:No
PHP Version:5.3 Sprint Focus:
Complexity:

Description

In the function loadIndexerClass() in CrawlerHook.php and CrawlerFilesHook.php there is a wrong reference to file class.indexer.php which was moved to file Indexer.php in subfolder Classes since TYPO3 6.2.


Related issues

related to Core - Bug #47852: Remove dummy files from ext:indexed_search Resolved 2013-05-02
related to Core - Bug #58877: class.indexer.php not available when start cli_dispatch.p... Resolved 2014-05-16

Associated revisions

Revision d2969cdb
Added by Markus Klein about 1 year ago

[BUGFIX] IndexSearch still uses require_once

Remove the old magic as classes are loaded properly already.

Resolves: #58567
Related: #47852
Releases: 6.2
Change-Id: Iec20bf0dbd6955950225936911eba604faf720d6
Reviewed-on: https://review.typo3.org/29898
Reviewed-by: Oliver Klee
Reviewed-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1 Updated by Markus Klein about 1 year ago

Does it work if you remove the call to this function completely?
A first look tells me that this is completely superfluous already.

#2 Updated by Gerrit Code Review about 1 year 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 https://review.typo3.org/29898

#3 Updated by Gerhard Rupp about 1 year ago

Patch seems to work. There are no errors in the php error log and indexing is OK.

#4 Updated by Gerrit Code Review about 1 year ago

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/29898

#5 Updated by Gerrit Code Review about 1 year 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/29898

#6 Updated by Gerrit Code Review about 1 year 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/29898

#7 Updated by Gerrit Code Review about 1 year ago

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

#8 Updated by Gerrit Code Review about 1 year ago

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

#9 Updated by Markus Klein about 1 year ago

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

Also available in: Atom PDF