HomePhabricator

Replace "getQueryParams()" callsites in Phabricator

Description

Replace "getQueryParams()" callsites in Phabricator

Summary: See D20136. This method is sort of inherently bad because it is destructive for some inputs (x=1&x=2) and had "PHP-flavored" behavior for other inputs (x[]=1&x[]=2). Move to explicit ...AsMap and ...AsPairList methods.

Test Plan: Bit of an adventure, see inlines in a minute.

Reviewers: amckinley

Reviewed By: amckinley

Differential Revision: https://secure.phabricator.com/D20141

Details

Provenance
epriestleyAuthored on Feb 11 2019, 7:14 PM
epriestleyPushed on Feb 12 2019, 2:37 PM
Reviewer
amckinley
Differential Revision
D20141: Replace "getQueryParams()" callsites in Phabricator
Parents
rP187356fea515: Let the top-level exception handler dump a stack trace if we reach debug mode…
Branches
Unknown
Tags
Unknown
Build Status
Buildable 21952
Build 29975: Run Core Tests