Page MenuHomePhabricator

Ids option in forceSearchIndex.php is broken
Closed, ResolvedPublic

Description

Any usage of this option resutls in:

--ids cannot be used with deletes/archive/from/to/fromId/toId/limit

This is because we check $this->indexUpdates before we set it and it defaults to null.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptDec 18 2017, 10:03 PM
Smalyshev triaged this task as Normal priority.Dec 18 2017, 10:12 PM

Change 399103 had a related patch set uploaded (by Smalyshev; owner: Smalyshev):
[mediawiki/extensions/CirrusSearch@master] Fix --ids option

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

Change 399103 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Fix --ids option

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

debt closed this task as Resolved.Jan 8 2018, 2:32 PM
debt claimed this task.