Revision c92d3311
Added by Adam Price almost 10 years ago
| engines/bastion/app/assets/javascripts/bastion/widgets/nutupane.factory.js | ||
|---|---|---|
|
return;
|
||
|
}
|
||
|
|
||
|
var sortOrder, sortBy = column.id;
|
||
|
params["sort_by"] = column.id;
|
||
|
if (column.id === sort.by) {
|
||
|
sortOrder = (sort.order === 'ASC') ? 'DESC' : 'ASC';
|
||
|
params["sort_order"] = (sort.order === 'ASC') ? 'DESC' : 'ASC';
|
||
|
} else {
|
||
|
sortOrder = 'ASC';
|
||
|
params["sort_order"] = 'ASC';
|
||
|
}
|
||
|
|
||
|
column.sortOrder = sortOrder;
|
||
|
column.sortOrder = params["sort_order"];
|
||
|
column.active = true;
|
||
|
|
||
|
params['order'] = sortBy + ' ' + sortOrder;
|
||
|
|
||
|
self.table.rows = [];
|
||
|
self.query();
|
||
|
};
|
||
Also available in: Unified diff
Revert "fixes #6573 - foretello should error when sorting by non-sortable field, BZ 1110431"
This reverts commit 3ec51c074a3decb6656f85195d98b39f57ccf430.
will re-introduce this fix at a later date when we have some testing
around sorting