Unify SearchKeywords implementation
Closed, ResolvedPublic
Actions

Description

We should try to make all SearchKeywords a subclass of SimpleKeywordFeature.
The reason is to remove and centralize all the parsing logic out of these classes.

Related Objects
Search...

Status	Assigned	Task
Open	None	T94814 Fix: "Warning: Search backend error during .. took .." (tracking)
Open	dcausse	T185108 EPIC: Refactor query parsing logic
Resolved	dcausse	T186879 Unify SearchKeywords implementation

Event Timeline

dcausse triaged this task as Normal priority.Feb 9 2018, 1:51 PM

dcausse created this task.

Restricted Application edited projects, added Discovery-Search; removed Discovery-Search (Current work). · View Herald TranscriptFeb 9 2018, 1:51 PM

Change 408803 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/CirrusSearch@master] Make prefer-recent a SimpleKeyword

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

gerritbot added a project: Patch-For-Review.Feb 9 2018, 1:54 PM

Change 409353 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/CirrusSearch@master] Make local: a SimpleKeyword

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

dcausse moved this task from needs triage to Current work on the Discovery-Search board.Feb 9 2018, 4:28 PM

dcausse edited projects, added Discovery-Search (Current work); removed Discovery-Search.

dcausse moved this task from in progress to not in use - please delete on the Discovery-Search (Current work) board.

Change 409389 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/CirrusSearch@master] Hide SimpleKeywordFeature::getValueRegex()

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

Change 410175 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/CirrusSearch@master] Add SimpleKeywordFeature::getFeatureName()

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

Change 410176 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/CirrusSearch@master] Make morelike: a simple keyword

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

Change 410202 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/CirrusSearch@master] Add tests for prefix

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

Change 408803 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Make prefer-recent a SimpleKeyword

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

ReleaseTaggerBot added a project: MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)).Feb 13 2018, 6:00 PM

Change 409353 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Make local: a SimpleKeyword

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

ReleaseTaggerBot edited projects, added MW-1.31-release-notes (WMF-deploy-2018-02-20 (1.31.0-wmf.22)); removed MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)).Feb 13 2018, 9:00 PM

Change 409389 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Hide SimpleKeywordFeature::getValueRegex()

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

Change 410175 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Add SimpleKeywordFeature::getFeatureName()

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

Change 410176 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Make morelike: a simple keyword

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

Change 410202 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Add tests for prefix

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

Change 410944 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/CirrusSearch@master] Make prefix: a SimpleKeyword

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

Change 415321 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/CirrusSearch@master] Make regex keywords SimpleKeywordFeature

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

dcausse moved this task from not in use - please delete to Needs review on the Discovery-Search (Current work) board.Feb 28 2018, 5:54 PM

Change 410944 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Make prefix: a SimpleKeyword

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

ReleaseTaggerBot edited projects, added MW-1.31-release-notes (WMF-deploy-2018-03-06 (1.31.0-wmf.24)); removed MW-1.31-release-notes (WMF-deploy-2018-02-20 (1.31.0-wmf.22)).Mar 6 2018, 11:01 AM

Change 415321 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Make regex keywords SimpleKeywordFeature

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

ReleaseTaggerBot edited projects, added MW-1.31-release-notes (WMF-deploy-2018-03-20 (1.31.0-wmf.26)); removed MW-1.31-release-notes (WMF-deploy-2018-03-06 (1.31.0-wmf.24)).Mar 14 2018, 11:00 PM

dcausse moved this task from Needs review to Done on the Discovery-Search (Current work) board.Mar 27 2018, 5:34 PM

debt closed this task as Resolved.Apr 3 2018, 3:51 PM