Page MenuHomePhabricator

deepcat should not be case sensitive on first letter of category name
Closed, ResolvedPublic

Description

I quote:

The deepcat example (deepcat:"musicals") seems to be off. For some reason, if the category's first letter is not capitalized, the search doesn't seem to search in the subcategories. First see Category:Musicals and then compare these searches: deepcat:"musicals" vs.deepcat:"Musicals". The latter search also gives the following warning: "''A warning has occurred while searching: Deep category query returned too many categories''". Maybe we should change the example search and category to a smaller category with correct capitalization, for example to deepcat:"Musicals by topic"?

I think this indeed a bit weird and possibly should automatically take into account Manual:$wgCapitalLinks and Manual:$wgCapitalLinkOverrides

Event Timeline

TheDJ created this task.Jul 22 2019, 10:07 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 22 2019, 10:07 AM
debt triaged this task as Normal priority.Jul 25 2019, 5:05 PM
debt edited projects, added Discovery-Search (Current work); removed Discovery-Search.

Change 532742 had a related patch set uploaded (by EBernhardson; owner: EBernhardson):
[mediawiki/extensions/CirrusSearch@master] Use makeTitleSafe to normalize deepcat inputs

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

Change 532742 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Use makeTitleSafe to normalize deepcat inputs

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

debt closed this task as Resolved.Thu, Sep 5, 6:31 PM