1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-23 07:12:41 +01:00

make sort controls only show up if sorting will work

Summary: T2326 tells the tale. this is the fix.

Test Plan: verified that queries that shouldn't be sortable weren't. also had a phlog in there a bit to sanity check things faster

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Maniphest Tasks: T2326

Differential Revision: https://secure.phabricator.com/D4816
This commit is contained in:
Bob Trahan 2013-02-04 18:59:46 -08:00
parent c7635d8bc6
commit 112c2ebfbe

View file

@ -330,8 +330,8 @@ final class ManiphestTaskListController extends ManiphestController {
$group = $query->getParameter('group'); $group = $query->getParameter('group');
$order = $query->getParameter('order'); $order = $query->getParameter('order');
$is_draggable = $is_draggable =
($group == 'priority') || ($order == 'priority') &&
($group == 'none' && $order == 'priority'); ($group == 'none' || $group == 'priority');
$lists = new AphrontNullView(); $lists = new AphrontNullView();
$lists->appendChild('<div class="maniphest-group-container">'); $lists->appendChild('<div class="maniphest-group-container">');