From 819bd7f03b2c5cfd1cfc217e49abb1afd691aeb4 Mon Sep 17 00:00:00 2001 From: Bob Trahan Date: Mon, 11 Nov 2013 15:01:21 -0800 Subject: [PATCH] Add arcanist project phid to differential.query Summary: Fixes T3535. Also, flip flop on that spacing thing and make the spaces purdy Test Plan: got an arcanist projected phid in the json dict Reviewers: epriestley Reviewed By: epriestley CC: Korvin, epriestley, aran Maniphest Tasks: T3535 Differential Revision: https://secure.phabricator.com/D7565 --- .../ConduitAPI_differential_query_Method.php | 39 ++++++++++--------- 1 file changed, 20 insertions(+), 19 deletions(-) diff --git a/src/applications/differential/conduit/ConduitAPI_differential_query_Method.php b/src/applications/differential/conduit/ConduitAPI_differential_query_Method.php index 6096c8386d..8d9d62261b 100644 --- a/src/applications/differential/conduit/ConduitAPI_differential_query_Method.php +++ b/src/applications/differential/conduit/ConduitAPI_differential_query_Method.php @@ -202,27 +202,28 @@ final class ConduitAPI_differential_query_Method $auxiliary_fields = $this->loadAuxiliaryFields( $revision, $request->getUser()); $result = array( - 'id' => $id, - 'phid' => $revision->getPHID(), - 'title' => $revision->getTitle(), - 'uri' => PhabricatorEnv::getProductionURI('/D'.$id), - 'dateCreated' => $revision->getDateCreated(), - 'dateModified' => $revision->getDateModified(), - 'authorPHID' => $revision->getAuthorPHID(), - 'status' => $revision->getStatus(), - 'statusName' => + 'id' => $id, + 'phid' => $revision->getPHID(), + 'title' => $revision->getTitle(), + 'uri' => PhabricatorEnv::getProductionURI('/D'.$id), + 'dateCreated' => $revision->getDateCreated(), + 'dateModified' => $revision->getDateModified(), + 'authorPHID' => $revision->getAuthorPHID(), + 'status' => $revision->getStatus(), + 'statusName' => ArcanistDifferentialRevisionStatus::getNameForRevisionStatus( $revision->getStatus()), - 'branch' => $diff->getBranch(), - 'summary' => $revision->getSummary(), - 'testPlan' => $revision->getTestPlan(), - 'lineCount' => $revision->getLineCount(), - 'diffs' => $revision->getDiffIDs(), - 'commits' => $revision->getCommitPHIDs(), - 'reviewers' => array_values($revision->getReviewers()), - 'ccs' => array_values($revision->getCCPHIDs()), - 'hashes' => $revision->getHashes(), - 'auxiliary' => $auxiliary_fields, + 'branch' => $diff->getBranch(), + 'summary' => $revision->getSummary(), + 'testPlan' => $revision->getTestPlan(), + 'lineCount' => $revision->getLineCount(), + 'diffs' => $revision->getDiffIDs(), + 'commits' => $revision->getCommitPHIDs(), + 'reviewers' => array_values($revision->getReviewers()), + 'ccs' => array_values($revision->getCCPHIDs()), + 'hashes' => $revision->getHashes(), + 'auxiliary' => $auxiliary_fields, + 'arcanistProjectPHID' => $diff->getArcanistProjectPHID() ); // TODO: This is a hacky way to put permissions on this field until we