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)
Mar 11 2026, 2:32 PM
Unknown Object (File)
Mar 10 2026, 11:58 PM
Unknown Object (File)
Feb 22 2026, 7:17 AM
Unknown Object (File)
Feb 10 2026, 2:35 PM
Unknown Object (File)
Feb 10 2026, 2:35 PM
Unknown Object (File)
Feb 1 2026, 7:32 AM
Unknown Object (File)
Jan 31 2026, 9:57 PM
Unknown Object (File)
Jan 8 2026, 10:27 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).