revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-01-17 10:11:10 +01:00
Code Issues Releases Wiki Activity

Add arcanist project phid to differential.query

Browse source 
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
This commit is contained in:
Bob Trahan 2013-11-11 15:01:21 -08:00
parent f35ce505a1
commit 819bd7f03b
1 changed files with 20 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

39
src/applications/differential/conduit/ConduitAPI_differential_query_Method.php
View file

@ -202,27 +202,28 @@ final class ConduitAPI_differential_query_Method
$auxiliary_fields = $this->loadAuxiliaryFields( $auxiliary_fields = $this->loadAuxiliaryFields(
$revision, $request->getUser()); $revision, $request->getUser());
$result = array( $result = array(
'id' => $id, 'id' => $id,
'phid' => $revision->getPHID(), 'phid' => $revision->getPHID(),
'title' => $revision->getTitle(), 'title' => $revision->getTitle(),
'uri' => PhabricatorEnv::getProductionURI('/D'.$id), 'uri' => PhabricatorEnv::getProductionURI('/D'.$id),
'dateCreated' => $revision->getDateCreated(), 'dateCreated' => $revision->getDateCreated(),
'dateModified' => $revision->getDateModified(), 'dateModified' => $revision->getDateModified(),
'authorPHID' => $revision->getAuthorPHID(), 'authorPHID' => $revision->getAuthorPHID(),
'status' => $revision->getStatus(), 'status' => $revision->getStatus(),
'statusName' => 'statusName' =>
ArcanistDifferentialRevisionStatus::getNameForRevisionStatus( ArcanistDifferentialRevisionStatus::getNameForRevisionStatus(
$revision->getStatus()), $revision->getStatus()),
'branch' => $diff->getBranch(), 'branch' => $diff->getBranch(),
'summary' => $revision->getSummary(), 'summary' => $revision->getSummary(),
'testPlan' => $revision->getTestPlan(), 'testPlan' => $revision->getTestPlan(),
'lineCount' => $revision->getLineCount(), 'lineCount' => $revision->getLineCount(),
'diffs' => $revision->getDiffIDs(), 'diffs' => $revision->getDiffIDs(),
'commits' => $revision->getCommitPHIDs(), 'commits' => $revision->getCommitPHIDs(),
'reviewers' => array_values($revision->getReviewers()), 'reviewers' => array_values($revision->getReviewers()),
'ccs' => array_values($revision->getCCPHIDs()), 'ccs' => array_values($revision->getCCPHIDs()),
'hashes' => $revision->getHashes(), 'hashes' => $revision->getHashes(),
'auxiliary' => $auxiliary_fields, 'auxiliary' => $auxiliary_fields,
'arcanistProjectPHID' => $diff->getArcanistProjectPHID()
); );
// TODO: This is a hacky way to put permissions on this field until we // TODO: This is a hacky way to put permissions on this field until we