Paths

Table of Contentst

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

Branch

tstatus

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 60
Build 60: [Placeholder Plan] Wait for 30 Seconds

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 21008

View Options

src/applications/maniphest/query/ManiphestTaskSearchEngine.php