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

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