revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-24 14:30:56 +01:00
Code Issues Releases Wiki Activity

Clarify that 'order' is an optional parameter in Conduit API methods

Browse source 
Summary: Fixes T8603. For automatic 'order' parameters provided by infrastructure en route to T7715, clarify that they are optional (we will use the default builtin order for the underlying Query if an order is not provided).

Test Plan: Used web UI to see "optional" hint.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T8603

Differential Revision: https://secure.phabricator.com/D13342
This commit is contained in:
epriestley 2015-06-18 13:05:44 -07:00
parent 2eb73619d1
commit 90078fe06e
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/conduit/method/ConduitAPIMethod.php
View file

@ -30,7 +30,7 @@ abstract class ConduitAPIMethod
$query = $this->newQueryObject(); $query = $this->newQueryObject();
if ($query) { if ($query) {
$types['order'] = 'order'; $types['order'] = 'optional order';
$types += $this->getPagerParamTypes(); $types += $this->getPagerParamTypes();
} }