Page MenuHomePhabricator

🧹️ Stop using TypeDispatchingEntitySearchHelper in the searchEntities maintenance script
Closed, ResolvedPublic

Description

Ask around if anyone uses this. Remove if not, use a controller otherwise.

Event Timeline

I suspect that the searchEntities maintenance script is mainly used for items and properties, if at all. If we don't want to remove it completely, we could just stop depending on the generic WikibaseRepo::getEntitySearchHelper that works for all entity types and instead use WbSearch::getItemSearchHelper and WbSearch::getPropertySearchHelper for items and properties respectively. That would of course break support for other entity types, but that's probably no big deal.

Change #1294311 had a related patch set uploaded (by Mona_WMDE; author: Mona_WMDE):

[mediawiki/extensions/Wikibase@master] Search: Use ItemSearchHelper and PropertySearchHelper in searchEntities maintenance script

https://gerrit.wikimedia.org/r/1294311

Change #1293733 had a related patch set uploaded (by Jakob; author: Jakob):

[mediawiki/extensions/Wikibase@master] Avoid generic search helper in searchEntities

https://gerrit.wikimedia.org/r/1293733

Change #1294311 abandoned by Mona_WMDE:

[mediawiki/extensions/Wikibase@master] Search: Use ItemSearchHelper and PropertySearchHelper in searchEntities maintenance script

Reason:

duplicate patch

https://gerrit.wikimedia.org/r/1294311

Change #1293733 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Avoid generic search helper in searchEntities

https://gerrit.wikimedia.org/r/1293733

Change #1295867 had a related patch set uploaded (by Itamar Givon; author: Itamar Givon):

[mediawiki/extensions/Wikibase@master] Search: Remove searchEntities maintenance script

https://gerrit.wikimedia.org/r/1295867

Change #1295867 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Search: Remove searchEntities maintenance script

https://gerrit.wikimedia.org/r/1295867