@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Add `activeDiffPHID` to differential.query
Summary: Ref T4809. This saves us a few round trips to find a Buildable, and generally makes the notion of "active" more explicit (i.e., not just the diff with the largest ID). In the future, we may let you revert to previous diffs, which would make the "largest number" rule not always correct.
Test Plan: Ran `differential.query`, got sensible results.
Reviewers: btrahan
Reviewed By: btrahan
Subscribers: epriestley
Maniphest Tasks: T4809
Differential Revision: https://secure.phabricator.com/D8800
+1
+1
src/applications/differential/conduit/ConduitAPI_differential_query_Method.php
+1
src/applications/differential/conduit/ConduitAPI_differential_query_Method.php
···
218
218
'summary' => $revision->getSummary(),
219
219
'testPlan' => $revision->getTestPlan(),
220
220
'lineCount' => $revision->getLineCount(),
221
+
'activeDiffPHID' => $diff->getPHID(),
221
222
'diffs' => $revision->getDiffIDs(),
222
223
'commits' => $revision->getCommitPHIDs(),
223
224
'reviewers' => array_values($revision->getReviewers()),