Page MenuHomePhabricator

Ensure that we always set the action parameter as a query parameter when POSTing in WikiLambda code
Closed, ResolvedPublic

Description

Per parent.

  • Use of mw.api in resources/ext.wikilambda.app/utils/apiUtils.js (or make this automatic upstream?)
  • Use in AbstractWikiRequest

Event Timeline

Use of mw.api in resources/ext.wikilambda.app/utils/apiUtils.js (or make this automatic upstream?)

apiUtils.js uses mw.Api, and mw.Api adds the action parameter to query parameters since https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1267086, so I guess this part is done?

Use of mw.api in resources/ext.wikilambda.app/utils/apiUtils.js (or make this automatic upstream?)

apiUtils.js uses mw.Api, and mw.Api adds the action parameter to query parameters since https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1267086, so I guess this part is done?

Yup! Thanks. The other bit is still to do.

Change #1277652 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):

[mediawiki/extensions/WikiLambda@master] AbstractWikiRequest: Add the 'action' parameter as a query parameter

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

It should be that simple, but I don't have the means to test it locally.

It should be that simple, but I don't have the means to test it locally.

Amazing, thanks; I will test it now.

Change #1277652 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] AbstractWikiRequest: Add the 'action' parameter as a query parameter

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