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)
Fri, Sep 6, 10:03 AM
Unknown Object (File)
Wed, Aug 28, 8:38 AM
Unknown Object (File)
Sun, Aug 25, 7:04 AM
Unknown Object (File)
Aug 15 2024, 10:47 AM
Unknown Object (File)
Aug 11 2024, 8:23 AM
Unknown Object (File)
Aug 7 2024, 9:23 AM
Unknown Object (File)
Aug 3 2024, 9:09 AM
Unknown Object (File)
Jul 20 2024, 3:55 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).