Bug #64886

indexed search extbase plugin doesn't work after moving class alias to compat extension

Added by Tymoteusz Motylewski almost 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Category:
Indexed Search
Target version:
-
Start date:
2015-02-05
Due date:
% Done:

100%

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

Description

Indexed search extbase experimental plugin (pi2) doesn't work after moving class alias to compatibility6 extension.

The indexed search plugin is registered like that:

\TYPO3\CMS\Extbase\Utility\ExtensionUtility::configurePlugin($_EXTKEY, 'Pi2', array('Search' => 'form,search'), array('Search' => 'form,search'));

Which makes Extbase to generate following TS:

tt_content.list.20.indexedsearch_pi2 = USER
tt_content.list.20.indexedsearch_pi2 {
userFunc = TYPO3\CMS\Extbase\Core\Bootstrap->run
extensionName = IndexedSearch
pluginName = Pi2
}

Which then makes extbase to look for non namespaced class:
Tx_IndexedSearch_Controller_SearchController

instead of:

\TYPO3\CMS\IndexedSearch\Controller\SearchController

we can fix it in 2 ways:
1. Not breaking - move class alias for this clas from compatibility6 extension back to indexed search
2. Breaking change:
Change plugin registration code to:

\TYPO3\CMS\Extbase\Utility\ExtensionUtility::configurePlugin('TYPO3\CMS.'.$_EXTKEY, 'Pi2', array('Search' => 'form,search'), array('Search' => 'form,search'));


Related issues

Related to TYPO3 Core - Bug #64280: Supply ClassAliasMap for EXT: indexed_search Closed 2015-01-14
Related to TYPO3 Core - Task #62990: Move classAliasMaps into ext:compatibility6 Closed 2014-11-15

Associated revisions

Revision db6973ba (diff)
Added by Tymoteusz Motylewski almost 5 years ago

[BUGFIX] Make indexed search extbase plugin work after class alias change

Indexed search extbase experimental plugin (pi2) requires class alias
to work. This patch moves it back from compatibility6 extension
to indexed_search.

Resolves: #64886
Releases: master
Change-Id: I36d288d80453a0772181358222608e1df83f9cf6
Reviewed-on: http://review.typo3.org/36681
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>

History

#1 Updated by Gerrit Code Review almost 5 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/36681

#2 Updated by Tymoteusz Motylewski almost 5 years ago

  • Category set to Indexed Search
  • Assignee set to Tymoteusz Motylewski
  • Sprint Focus set to Stabilization Sprint

#3 Updated by Tymoteusz Motylewski almost 5 years ago

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

#4 Updated by Daniel Ostmann over 4 years ago

For me it did not work in 7.3.1. I had to use this TypoScript setup:

tt_content.list.20.indexedsearch_pi2.vendorName = TYPO3\CMS

#5 Updated by Tymoteusz Motylewski over 4 years ago

Hi Daniel,
I can not reproduce the issue any longer.
Can you please outline some steps which are needed to reproduce the issue?

#6 Updated by Daniel Ostmann about 4 years ago

Yes, in version 6.2.15 it still have to add the vendor in indexedsearch_pi2.

Well, the steps... Just create the experimental-plugin and fire up a search in frontend. I didn't get any results until I set the vendor.

In 7.5.0 I did not test it.

#7 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF