Page MenuHomePhabricator

Attach TaskPHIDs to commits in diffusion.getcommits
ClosedPublic

Authored by dctrwatson on Nov 27 2013, 10:21 PM.
Tags
None
Referenced Files
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
Unknown Object (File)
Fri, Dec 13, 8:25 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

Lint
Lint Skipped
Unit
Tests Skipped

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