matthew.britton wrote:

Actually, turns out it's a bit more broken.

In trunk, the equivalent for rollback also gives the error "The title parameter must be set". On WMF sites it works:

http://en.wikipedia.org/w/api.php?action=paraminfo&modules=rollback

This may be related to the recent changes to "required" parameter validation in the API.

Also in trunk, *all* querymodules (e.g. action=paraminfo&querymodules=allpages) cause paraminfo to fail, with an error like this one:

Catchable fatal error: Argument 1 passed to ApiQueryBase::__construct() must be an instance of ApiQuery, instance of ApiParamInfo given, called in ...\includes\api\ApiQueryBase.php on line 467 and defined in ...\includes\api\ApiQueryBase.php on line 43

matthew.britton wrote:

Querymodule breakage was caused by r72387.

I was thinking for the former it must be more widespread (and presumably it is/should be for all modules that have minimum definitely required parameters)

soxred93 wrote:

It looks like the paraminfo errors is caused because before PARAM_REQUIRED, required parameter validations were done in the execute() method. Now they're done in the validate parameter methods. When using ApiParaminfo, the execute() method is not called, but the validate parameter methods are. I'm pretty sure this is the problem.

matthew.britton wrote:

addressed querymodule breakage in r73596

(In reply to comment #3)

I was thinking for the former it must be more widespread (and presumably it
is/should be for all modules that have minimum definitely required parameters)

Only rollback and userrights are affected (haven't tested all extension modules).

That's intriguing... In theory, the purge module (among others?), should also be.

Need to get my dev machine back installed (moved it from my parents last night), and have a poke

soxred93 wrote:

Was going to have a poke at this as well, but realized that when I upgraded to my new OS, I didn't copy the mysql databases. I'll copy them from the backup tonight and have a look.

Interesting, on ApiUserrights, disable PARAM_REQUIRED on user, and get

<error code="unknownerror" info="Unknown error: ``&#039;&#039;" xml:space="preserve">

And this goes back further, it occurs in 1.16

Userrights didn't exist in 1.15

r74098