Details
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T413959 Refactor wbsearchentities to use hexagonal architecture | |||
| Open | None | T413963 Create controller implementations calling the corresponding entity type-specific use case | |||
| Open | None | T404043 [EPIC] Use Item/PropertyPrefixSearch use case implementations for wbsearchentities | |||
| Open | None | T421994 🧄 Use ItemPrefixSearch use case implementations for wbsearchentities | |||
| Open | Silvan_WMDE | T422137 🧄 Use ItemWbSearchEntitiesController in repo/WikibaseRepo.controllers.php |
Event Timeline
Change #1276405 had a related patch set uploaded (by Silvan Heintze; author: Silvan Heintze):
[mediawiki/extensions/Wikibase@master] Search: Enable ItemWbSearchEntitiesController
Change #1277094 had a related patch set uploaded (by Silvan Heintze; author: Silvan Heintze):
[mediawiki/extensions/WikibaseCirrusSearch@master] Adjust SearchEntitiesIntegrationTest to new WbSearchEntitiesController
Using ItemWbSearchEntitiesController unconditionally here, would break wbsearchentities for Wikibase instances without ElasticSearch, which is probably not intended. The proposed patch keeps the fallback controller if CirrusSearch is not enabled - let's discuss what is the best solution.
Change #1277094 merged by jenkins-bot:
[mediawiki/extensions/WikibaseCirrusSearch@master] Adjust SearchEntitiesIntegrationTest to new WbSearchEntitiesController
Change #1276405 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Search: Enable ItemWbSearchEntitiesController