teaminmedias-pluswerk / ke_search

Search Extension for TYPO3 Content Management System, including faceting search functions.
https://extensions.typo3.org/extension/ke_search/
GNU General Public License v3.0
35 stars 62 forks source link

tt_address records are only indexed from first storage pid #374

Closed akiessling closed 3 years ago

akiessling commented 3 years ago

It seems like this statement fetches only records from the first pid

https://github.com/teaminmedias-pluswerk/ke_search/blob/b0ce7c65350fd0e5c4627fc1358985ab996c3db8/Classes/Indexer/Types/TtAddress.php#L77

$queryBuilder->expr()->in(
                    'pid',
                    $queryBuilder->createNamedParameter(
                        implode(',', $indexPids), \PDO::PARAM_STR
                    )
                )

Compared to the statement for the news indexer, the creation of the named parameter should be dropped https://github.com/teaminmedias-pluswerk/ke_search/blob/b0ce7c65350fd0e5c4627fc1358985ab996c3db8/Classes/Indexer/Types/News.php#L88 Imploding the values also does not seem necessary, since the in expression already takes care of that '(' . implode(', ', (array)$value) . ')'