mirror/php-e107
1
0
Fork 0
mirror of https://github.com/e107inc/e107.git synced 2025-07-31 20:00:37 +02:00
Code Issues Releases Wiki Activity

Removed pointless second sort in e_tree_model

Browse Source 
e_tree_model::prepareSimulatedCustomOrdering() performed an ORDER BY
second sort that doesn't add any benefit at all and causes #3086.

This second sort has been removed.

Fixes: #3086
This commit is contained in:
Deltik
2018-04-03 17:21:45 -05:00
parent 9726a70d2d
commit b3aa33afb5
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
e107_handlers/model_class.php
View File

@@ -3617,7 +3617,7 @@ class e_tree_model extends e_front_model
return "";
}, $db_query)
// Optimization goes with e_tree_model::moveRowsToTreeNodes()
. " ORDER BY " . $this->getParam('sort_parent') . ", " . $this->getParam('sort_field');
. " ORDER BY " . $this->getParam('sort_parent');
$this->setParam('db_query', $db_query);
}