From 1258ed348b4452da2e1ff807d4883ce8dc38a54e Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Fri, 23 Jan 2015 07:16:15 +1100 Subject: [PATCH] Add return type to `conduit.query` Summary: Fixes T6950. Adds the return type of Conduit API methods to the `conduit.query` call. Test Plan: Called `echo '{}' | arc call-conduit conduit.query` and verified that the return types were present in the response. Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley, #blessed_reviewers Subscribers: Korvin, epriestley Maniphest Tasks: T6950 Differential Revision: https://secure.phabricator.com/D11466 --- src/applications/conduit/method/ConduitQueryConduitAPIMethod.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/applications/conduit/method/ConduitQueryConduitAPIMethod.php b/src/applications/conduit/method/ConduitQueryConduitAPIMethod.php index b30112e7c9..0a3fcc2d1b 100644 --- a/src/applications/conduit/method/ConduitQueryConduitAPIMethod.php +++ b/src/applications/conduit/method/ConduitQueryConduitAPIMethod.php @@ -32,6 +32,7 @@ final class ConduitQueryConduitAPIMethod extends ConduitAPIMethod { foreach ($classes as $class) { $names_to_params[$class->getAPIMethodName()] = array( 'params' => $class->defineParamTypes(), + 'return' => $class->defineReturnType(), ); }