If deepcat: feature gets more than CirrusSearchCategoryMax categories, it should not use them but instead ignore the keyword and produce search warning.
The warning would be:
"cirrussearch-feature-deepcat-toomany": "Deep category query returned too many categories",
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Smalyshev | T181549 [epic] Subcategory searching | |||
| Resolved | Smalyshev | T188350 Produce search error if deepcat search returns too many categories |
Change 414866 had a related patch set uploaded (by Smalyshev; owner: Smalyshev):
[mediawiki/extensions/CirrusSearch@master] Produce warning and fail deepcat filter if too many categories
Change 414866 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Produce warning and fail deepcat filter if too many categories
@Smalyshev yay it is live :) I'm just surprised that there are still search results when I query for deepcategory:philosophy. Where do these articles come from?
@Lea_WMDE looks like it just runs empty query. Which is probably a bug. Filed T189331.
Hi @Smalyshev I just saw that the query for deepcategory:philosophy still returns "There were no results matching the query." and not the error message :/
Hmm weird looks like the message is somehow lost from the context. Not sure what is happening, I'll debug it.
Change 427287 had a related patch set uploaded (by Smalyshev; owner: Smalyshev):
[mediawiki/extensions/CirrusSearch@master] Don't suppress warnings when keywords failed
Change 427287 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Don't suppress warnings when keywords failed