Last modified: 2014-07-01 19:41:27 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T66505, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 64505 - list=allusers auactiveusers recenteditcount value is multiplied by the number of groups
list=allusers auactiveusers recenteditcount value is multiplied by the number...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
1.24rc
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-04-27 14:02 UTC by kynikos
Modified: 2014-07-01 19:41 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description kynikos 2014-04-27 14:02:57 UTC
Consider the following queries:

* http://test2.wikipedia.org/w/api.php?action=query&list=allusers&augroup=reviewer&aufrom=Selenium_user&aulimit=1&auactiveusers=1
* http://test2.wikipedia.org/w/api.php?action=query&list=allusers&augroup=reviewer|editor&aufrom=Selenium_user&aulimit=1&auactiveusers=1
* http://test2.wikipedia.org/w/api.php?action=query&list=allusers&augroup=reviewer|editor|qa_automation&aufrom=Selenium_user&aulimit=1&auactiveusers=1

As you can see the value of recenteditcount is x in the first query, 2x in the second query and 3x in the third query. My guess is that the first one returns the correct value, while the other two multiply it by the number of queried  (with augroup) groups that Selenium_user belongs to.

Now consider the following query:

* http://test2.wikipedia.org/w/api.php?action=query&list=allusers&auprop=groups&aufrom=Selenium_user&aulimit=1&auactiveusers=1

The value of recenteditcount is now 4x, which I guess is the number of the groups that augroup could accept and that Selenium_user belongs to. The fact that augroup is used or not doesn't influence the result in this case.

I think the value of recenteditcount should be x in all the above queries.

(Not relevant, but this bug was originally discovered in https://github.com/kynikos/wiki-monkey/issues/174#issuecomment-41472026 )
Comment 1 kynikos 2014-04-27 14:13:40 UTC
Sorry for the typo in the original subject (list=allursers should be list=allusers).

https://bugzilla.wikimedia.org/show_bug.cgi?id=64507 may be related.
Comment 2 Gerrit Notification Bot 2014-04-28 16:51:39 UTC
Change 130093 had a related patch set uploaded by Anomie:
API: Fix queries for list=allusers&auactiveusers

https://gerrit.wikimedia.org/r/130093
Comment 3 Gerrit Notification Bot 2014-07-01 19:14:57 UTC
Change 130093 merged by jenkins-bot:
API: Fix queries for list=allusers&auactiveusers

https://gerrit.wikimedia.org/r/130093
Comment 4 Brad Jorsch 2014-07-01 19:41:27 UTC
Should be deployed to WMF wikis with 1.24wmf12, see https://www.mediawiki.org/wiki/MediaWiki_1.24/Roadmap for the schedule.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links