Page MenuHomePhabricator

Update FlaggedRevs not to use deprecated EditPageBeforeEditChecks hook
Closed, ResolvedPublic1 Story Points

Description

The hook was deprecated in https://gerrit.wikimedia.org/r/#/c/344805/ (MW 1.29).

Use 'EditPageGetCheckboxesDefinition' instead, or 'EditPage::showStandardInputs:options' if you don't actually care about checkboxes and just want to add some HTML to the page.

Event Timeline

matmarex created this task.Mar 30 2017, 4:40 PM
Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptMar 30 2017, 4:40 PM
matmarex claimed this task.Mar 30 2017, 4:41 PM

Change 344808 had a related patch set uploaded (by Bartosz Dziewoński):
[mediawiki/extensions/FlaggedRevs@master] Update to use EditPageGetCheckboxesDefinition hook

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

Jdforrester-WMF triaged this task as High priority.
Jdforrester-WMF set the point value for this task to 1.
Jdforrester-WMF moved this task from To Triage to TR0: Interrupt on the VisualEditor board.

Change 344808 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@master] Update to use EditPageGetCheckboxesDefinition hook

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

Jdforrester-WMF closed this task as Resolved.Mar 30 2017, 5:49 PM
Jdforrester-WMF removed a project: Patch-For-Review.
Restricted Application added a project: User-Ryasmeen. · View Herald TranscriptMar 30 2017, 5:49 PM