Page MenuHomePhabricator

Attach TaskPHIDs to commits in diffusion.getcommits
ClosedPublic

Authored by dctrwatson on Nov 27 2013, 10:21 PM.
Tags
None
Referenced Files
F15396369: D7668.id17318.diff
Sun, Mar 16, 1:49 PM
F15377138: D7668.id17317.diff
Thu, Mar 13, 8:13 AM
Unknown Object (File)
Mon, Feb 24, 3:06 AM
Unknown Object (File)
Wed, Feb 19, 5:44 PM
Unknown Object (File)
Feb 19 2025, 1:07 PM
Unknown Object (File)
Feb 13 2025, 10:40 PM
Unknown Object (File)
Feb 8 2025, 12:37 AM
Unknown Object (File)
Feb 6 2025, 6:20 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).