From b3aa33afb5a3420bd1c6413c796107846b1d1e33 Mon Sep 17 00:00:00 2001 From: Deltik Date: Tue, 3 Apr 2018 17:21:45 -0500 Subject: [PATCH] Removed pointless second sort in e_tree_model 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 --- e107_handlers/model_class.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/e107_handlers/model_class.php b/e107_handlers/model_class.php index 8d716382f..c905c8a65 100755 --- a/e107_handlers/model_class.php +++ b/e107_handlers/model_class.php @@ -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); }