Page MenuHomePhabricator

Attach TaskPHIDs to commits in diffusion.getcommits
ClosedPublic

Authored by dctrwatson on Nov 27 2013, 10:21 PM.
Tags
None
Referenced Files
F20912904: D7668.diff
Sat, Jun 27, 12:52 AM
Unknown Object (File)
Mar 11 2026, 2:32 PM
Unknown Object (File)
Mar 10 2026, 11:58 PM
Unknown Object (File)
Feb 22 2026, 7:17 AM
Unknown Object (File)
Feb 10 2026, 2:35 PM
Unknown Object (File)
Feb 10 2026, 2:35 PM
Unknown Object (File)
Feb 1 2026, 7:32 AM
Unknown Object (File)
Jan 31 2026, 9:57 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).