Page MenuHomePhabricator

action=feedrecentchanges is missing parameters from generated documentation
Open, Needs TriagePublic

Description

Hi, i have a personal wiki on Miraheze and by default some of the newly created namespace start on 3000 and increasing (3002, 3004, and so on).
In the Github for API: https://github.com/wikimedia/mediawiki/blob/REL1_34/includes/api/ApiBase.php#L75
in line 75 says:

* - namespace: An integer representing a MediaWiki namespace. Forces PARAM_ALL = true to

But in this documentation page: https://www.mediawiki.org/w/api.php?action=help&modules=feedrecentchanges
it says:

namespace
Namespace to limit the results to.

One of the following values: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 90, 91, 92, 93, 100, 101, 102, 103, 104, 105, 106, 107, 486, 487, 828, 829, 1198, 1199, 2300, 2301, 2302, 2303, 2600, 5500, 5501

This section of the documentation page leads to confusion. If the Github is saying right, then "An integer" like 3000 works.

Other things missing from the documentation page MediaWiki API help are:
&hidepageedits=1
&hidelog=1
&hideWikibase=1

thanks

Event Timeline

Restricted Application added subscribers: RhinosF1, Reception123, Aklapper. · View Herald Transcript

This section of the documentation page leads to confusion. If the Github is saying right, then "An integer" like 3000 works.

Yes, it would work if there's a NS 3000 on the wiki. On mediawiki.org there isn't, so it doesn't appear.

The mediawiki.org api generated help page is not going to be representative for your own wiki.

I don't think there is a bug there.

Other things missing from the documentation page MediaWiki API help are:
&hidepageedits=1
&hidelog=1

ChangesListSpecialPage adds those two parameters. They need adding/documenting in ApiFeedRecentChanges I guess

&hideWikibase=1

Obviously an issue with MediaWiki-extensions-WikibaseClient, not MW core. Probably needs to do something with the APIGetAllowedParams hook to add them

Reedy renamed this task from Request to update MediaWiki API help. action=feedrecentchanges to include more namespaces and missing settings to action=feedrecentchanges is missing parameters from generated documentation.Oct 26 2020, 8:22 PM

@Reedy: We also had the confusion over &namespace= accepts multiple values seperated by ; on Special:RC but the api doesn't. Could the api accept multiple namespace values?

@Reedy: We also had the confusion over &namespace= accepts multiple values seperated by ; on Special:RC but the api doesn't. Could the api accept multiple namespace values?

Probably

Just needs a ApiBase::PARAM_ISMULTI => true, adding to ApiFeedRecentChanges::getAllowedParams() in the namespace array (like other API modules do) if the underlying class can do multiple NS fine. Should be enough easy to test.

Separate bugs should be filed for separate issues. There's at least 3 different issues (depending on how you count) here, and 1 is seemingly invalid.

Hi, What is the next step on solving / improving / updating this. Is someone working on it or is in hiatus?
If I can help with something, tell me or send me a message.