Last modified: 2011-06-02 16:15:16 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 T31120, the corresponding Phabricator task for complete and up-to-date bug report information.

Bug 29120 - ApiQueryLogEvents: let letype accept multiple values


Summary:	ApiQueryLogEvents: let letype accept multiple values

Status:	RESOLVED WONTFIX

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

Importance:	Unprioritized enhancement (vote)
Target Milestone:	---
Assigned To:	Roan Kattouw

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-05-24 05:19 UTC by merl
Modified:	2011-06-02 16:15 UTC (History)
CC List:	5 users (show)

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

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

Description merl 2011-05-24 05:19:05 UTC

I cannot find any reason why letype must not be "PARAM_ISMULTI".

leaction must be single value because the where condition contains two columns (otherwise this would get much more complicated).
But letype is only used if leaction is null and the addWhereFld() function can accept multiply values without any logical change because the condition uses only a single column.
So i think only getAllowedParams must be changed to support this enhancement.

Comment 1 Sam Reed (reedy) 2011-05-24 16:04:35 UTC

r88722

Comment 2 Roan Kattouw 2011-06-02 16:15:16 UTC

(In reply to comment #1)
> r88722
Reverted. The way this stuff is indexed, querying multiple types isn't something we can do very nicely.

Wikimedia Bugzilla is closed!

Search

Personal tools

Navigation

Links