Page MenuHomePhabricator

More-robust search for task assignees
ClosedPublic

Authored by amckinley on Apr 30 2018, 7:11 PM.
Tags
None
Referenced Files
F18756411: D19417.id.diff
Sun, Oct 5, 11:21 AM
F18708796: D19417.diff
Sun, Sep 28, 10:53 PM
F18656268: D19417.id46446.diff
Mon, Sep 22, 9:59 PM
F18509606: D19417.id.diff
Sep 5 2025, 3:30 AM
F18502821: D19417.diff
Sep 4 2025, 10:40 PM
F18167805: D19417.id46448.diff
Aug 15 2025, 1:19 PM
F18110588: D19417.id.diff
Aug 11 2025, 9:24 PM
F18106184: D19417.diff
Aug 10 2025, 9:51 PM
Subscribers

Details

Summary

See discussion in D19415.

Test Plan

Searched for some owners, found tasks as expected.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20257
Build 27495: Run Core Tests
Build 27494: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Apr 30 2018, 7:16 PM
This revision was automatically updated to reflect the committed changes.

This actually caused an AphrontParameterQueryException while loading the Maniphest home page due to $this->ownerPHIDs being an empty array. I attempted to fix it by changing line 588 to:

if (($this->ownerPHIDs !== null) and !empty($this->ownerPHIDs)) {

which seems to work. I have no idea what side-effects this "fix" might cause though, I'm still waiting for my team to complain.

If you'd like that fixed, please file a bug report on the Discourse forum, including reproduction instructions that others can follow to reproduce the issue. A link to a page on this install which fatals would be sufficient.