Page MenuHomePhabricator

Use mediawiki.Api
Closed, ResolvedPublic

Description

There are multiple instances of the following:
$.post( mw.util.wikiScript( 'api' ), params, successFunction );
Which can be replaced with the following for consistency:
new mw.Api().post( params ).done( successFunction );


Version: unspecified
Severity: trivial
Whiteboard: gci2014

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 22 2014, 2:53 AM
bzimport set Reference to bz62956.
bzimport added a subscriber: Unknown Object (MLST).

Seen in ext.translate.special.aggregategroups.js (4 times) and in ext.translate.messagetable.js.

Change 138037 had a related patch set uploaded by Gerrit Patch Uploader:
Use mw.Api

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

Change 139380 had a related patch set uploaded by Kunalgrover05:
Use mw.api in Special:AggregateGroups

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

The requested changes in resources/js/ext.translate.special.aggregategroups.js have been made by Kunal.

ext.translate.messagetable.js still needs to be updated.

Change 139380 merged by jenkins-bot:
Use mw.api in Special:AggregateGroups

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

Change 213250 had a related patch set uploaded (by Nemo bis):
Use mw.Api().post() in ext.translate.messagetable.js

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

Change 213250 merged by jenkins-bot:
Use mw.Api().post() in ext.translate.messagetable.js

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

Nikerabbit claimed this task.

There are two $.post and one $.get left, but neither calls the API. I consider this task fixed.