Page MenuHomePhabricator

Include maniphestTaskPHIDs into differential.query Conduit API method results
Closed, DuplicatePublic

Description

We should include maniphestTaskPHIDs into differential.query results.

Can we add something like this to DifferentialQueryConduitAPIMethod class?

$maniphest = 'PhabricatorManiphestApplication';
$maniphest_installed = PhabricatorApplication::isClassInstalled($maniphest);

// <...>

if ($maniphest_installed) {
  $taskPHIDs = PhabricatorEdgeQuery::loadDestinationPHIDs(
    $revision->getPHID(),
    DifferentialRevisionHasTaskEdgeType::EDGECONST);

  $tasks_key = id(new DifferentialManiphestTasksField)->getFieldKeyForConduit();
  $result[$tasks_key] = $taskPHIDs;
}

Or we need something "smarter"?

Event Timeline

Pawka raised the priority of this task from to Normal.
Pawka updated the task description. (Show Details)
Pawka added projects: Differential, Maniphest, Conduit.
Pawka added a subscriber: Pawka.
Pawka updated the task description. (Show Details)