Page MenuHomePhabricator

Use authoritative PHIDs for Herald "Task Projects" condition
ClosedPublic

Authored by epriestley on Jul 26 2014, 12:07 PM.
Tags
None
Referenced Files
F18765708: D10060.id24188.diff
Tue, Oct 7, 12:37 PM
F18712025: D10060.id.diff
Mon, Sep 29, 5:14 AM
F18703008: D10060.id.diff
Sun, Sep 28, 2:34 AM
F18695546: D10060.diff
Sat, Sep 27, 6:01 AM
F18692531: D10060.id.diff
Fri, Sep 26, 11:06 PM
F18635806: D10060.id.diff
Sep 17 2025, 2:48 AM
F18626057: D10060.id.diff
Sep 16 2025, 12:15 AM
F18616610: D10060.id24186.diff
Sep 14 2025, 6:19 PM
Subscribers

Details

Summary

Fixes T5707. After work on T5245, the $task object may not have the correct set of task PHIDs attached to it when Herald is invoked.

Instead, just fetch the authoritative set. This is simple, and Herald is not super performance sensitive (happens on a write) so hitting the DB once is fine.

Test Plan

Created a task with projects, verified the projects field contained the correct values when processed by Herald.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Use authoritative PHIDs for Herald "Task Projects" condition.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: joshuaspence, chad, btrahan.
This revision is now accepted and ready to land.Jul 26 2014, 1:47 PM
epriestley updated this revision to Diff 24188.

Closed by commit rP0b5fcb36ae8c (authored by @epriestley).