Page MenuHomePhabricator

ordermethod ignored in form 2 [param1, param2]
Open, LowPublic

Description

Author: mediawiki_bug

Description:
Explodes the ordermethod argument prior to parsing

See http://semeb.com/dpldemo/index.php?title=DPL:Bug_Reports_Archive#ordermethod_ignored_form2_demo
e.g. ordermethod=titlewithoutnamespace,firstedit won't work.

The arg is tested as
if( in_array($sArg, $wgDPL2Options['ordermethod']) ) {
which will not work because $sArg hasn't been exploded yet.

Patch included.


Version: unspecified
Severity: normal

Attached:

Details

Reference
bz10858

Event Timeline

bzimport raised the priority of this task from to Low.Nov 21 2014, 9:48 PM
bzimport set Reference to bz10858.
bzimport added a subscriber: Unknown Object (MLST).

sumanah wrote:

Adding "need-review" keyword to indicate patch awaits review.