Page MenuHomePhabricator

ApiQueryLogEvents: let letype accept multiple values
Closed, DeclinedPublic

Description

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.


Version: unspecified
Severity: enhancement

Details

Reference
bz29120

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 21 2014, 11:28 PM
bzimport set Reference to bz29120.

(In reply to comment #1)

r88722

Reverted. The way this stuff is indexed, querying multiple types isn't something we can do very nicely.