Page MenuHomePhabricator

Attach TaskPHIDs to commits in diffusion.getcommits
ClosedPublic

Authored by dctrwatson on Nov 27 2013, 10:21 PM.
Tags
None
Referenced Files
F14489871: D7668.diff
Thu, Jan 2, 1:13 AM
Unknown Object (File)
Wed, Dec 25, 1:58 PM
Unknown Object (File)
Sun, Dec 22, 4:35 PM
Unknown Object (File)
Thu, Dec 19, 6:01 PM
Unknown Object (File)
Tue, Dec 17, 7:40 AM
Unknown Object (File)
Tue, Dec 17, 2:29 AM
Unknown Object (File)
Sun, Dec 15, 10:02 AM
Unknown Object (File)
Sat, Dec 14, 11:16 AM

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).