Fix an issue with multiple "open" statuses and the "View All" button on Projects
ClosedPublic
Actions

Authored by epriestley on Apr 24 2014, 3:35 PM.

Details

Reviewers

btrahan
chad

Commits

Restricted Diffusion Commit
rPea66aead59a5: Fix an issue with multiple "open" statuses and the "View All" button on Projects

Summary

See IRC. We construct this a little bit wrong if there are multiple "open" statuses. Use a more modern construction.

Test Plan

Hit ?statuses=wontfix,invalid, etc. Clicked "view all" from projects.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 21008.Apr 24 2014, 3:35 PM

epriestley retitled this revision from to Fix an issue with multiple "open" statuses and the "View All" button on Projects.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: btrahan, chad.

Herald added a subscriber: epriestley. · View Herald TranscriptApr 24 2014, 3:35 PM

Harbormaster completed remote builds in B60: Diff 21008.Apr 24 2014, 3:36 PM

chad accepted this revision.Apr 24 2014, 4:07 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Apr 24 2014, 4:07 PM

Closed by commit rPea66aead59a5 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

query/

ManiphestTaskSearchEngine.php

10 lines

project/

controller/

PhabricatorProjectProfileController.php

4 lines

Diff 21010

View Options

src/applications/maniphest/query/ManiphestTaskSearchEngine.php