Last modified: 2014-04-17 18:59:00 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 T65545, the corresponding Phabricator task for complete and up-to-date bug report information.

Bug 63545 - When too many action=query list/prop/meta items are selected, it's loading forever


Summary:	When too many action=query list/prop/meta items are selected, it's loading fo...

Status:	RESOLVED FIXED

Product:	MediaWiki extensions
Classification:	Unclassified
Component:	ApiSandbox (Other open bugs)
Version:	unspecified
Hardware:	All All

Importance:	Normal normal (vote)
Target Milestone:	---
Assigned To:	Bartosz Dziewoński

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2014-04-04 19:13 UTC by Liangent
Modified:	2014-04-17 18:59 UTC (History)
CC List:	4 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Liangent 2014-04-04 19:13:49 UTC

API action=paraminfo says:

    "warnings": {
        "paraminfo": {
            "*": "Too many values supplied for parameter 'querymodules': the limit is 50"
        }
    },

and browser console says:

TypeError: actionInfo is undefined load.php:2

in this case paraminfo should be queries in multiple batches.

Note that the limit for users with apihighlimits (eg. sysops, bots) should be 500.

Comment 1 Alex Monk 2014-04-04 19:24:22 UTC

Ugh. When I made Gerrit change #112307 I didn't expect you to be trying anything close to 50 at once

Comment 2 Gerrit Notification Bot 2014-04-07 15:30:25 UTC

Change 124334 had a related patch set uploaded by Bartosz Dziewoński:
Rewrite getParamInfo() to actually handle caching and continuation

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

Comment 3 Bartosz Dziewoński 2014-04-07 15:31:53 UTC

Not really a fault of that change, the code for this looked like it supported this kind of calls but actually didn't. Thorough testing would be appreciated :)

Comment 4 Gerrit Notification Bot 2014-04-17 17:15:28 UTC

Change 124334 merged by jenkins-bot:
Rewrite getParamInfo() to actually handle caching and continuation

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

Wikimedia Bugzilla is closed!

Search

Personal tools

Navigation

Links