Page MenuHomePhabricator

Attach TaskPHIDs to commits in diffusion.getcommits
ClosedPublic

Authored by dctrwatson on Nov 27 2013, 10:21 PM.
Tags
None
Referenced Files
F14032929: D7668.id17315.diff
Sat, Nov 9, 4:12 PM
F14032782: D7668.id17316.diff
Sat, Nov 9, 3:39 PM
F14032749: D7668.id17317.diff
Sat, Nov 9, 3:31 PM
F14006763: D7668.id17317.diff
Mon, Oct 28, 4:35 PM
F14006082: D7668.id17315.diff
Mon, Oct 28, 12:46 AM
F13980122: D7668.id17316.diff
Sat, Oct 19, 8:35 AM
F13977833: D7668.diff
Fri, Oct 18, 8:17 PM
F13974802: D7668.diff
Fri, Oct 18, 7:26 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).