Page MenuHomePhabricator

Attach TaskPHIDs to commits in diffusion.getcommits
ClosedPublic

Authored by dctrwatson on Nov 27 2013, 10:21 PM.
Tags
None
Referenced Files
F18630113: D7668.id17318.diff
Sep 16 2025, 10:48 AM
F18627137: D7668.diff
Sep 16 2025, 3:17 AM
F18602747: D7668.diff
Sep 13 2025, 4:01 PM
F18516918: D7668.diff
Sep 5 2025, 1:02 PM
F18102728: D7668.id17318.diff
Aug 9 2025, 10:59 PM
F18099170: D7668.id17316.diff
Aug 9 2025, 2:20 AM
F18098986: D7668.id.diff
Aug 9 2025, 12:18 AM
F18098731: D7668.id17315.diff
Aug 8 2025, 10:54 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP49f3ff0e08f4: Attach TaskPHIDs to commits in diffusion.getcommits
Summary

Uses edge query to attach TaskPHIDs to commit objects

Test Plan

Use conduit to getcommits with attached tasks

Diff Detail

Branch
getcommits-taskphids
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

One simplification inline.

src/applications/diffusion/conduit/ConduitAPI_diffusion_getcommits_Method.php
278–283

I believe you can simplify this slightly as:

$task_type = PhabricatorEdgeConfig::TYPE_COMMIT_HAS_TASK;

// ...

$task_phids = $edge_query->getDestinationPHIDs(
  array($commit['commitPHID']),
  array($task_type));
dctrwatson updated this revision to Unknown Object (????).Nov 27 2013, 10:31 PM
dctrwatson updated this revision to Unknown Object (????).Nov 27 2013, 10:35 PM
  • Use in edge_query too
epriestley closed this revision.

Closed by commit rP49f3ff0e08f4 (authored by @dctrwatson, committed by @epriestley).