summaryrefslogtreecommitdiff
path: root/modules
diff options
context:
space:
mode:
authorAndy Staudacher <andy.st@gmail.com>2009-07-27 11:25:23 -0700
committerAndy Staudacher <andy.st@gmail.com>2009-07-27 11:25:23 -0700
commitd18f31ab218c8d79d97b19079ac0078e6155bc39 (patch)
treea4b37e831961540ee93e93f31f03d4b9bf984cc1 /modules
parent482e223b602d60eec424081d0328cde66d28a758 (diff)
Revert "Remove an useless ORDER BY."... the order by wasn't all that useless.
This reverts commit c80d2da0a95a63b76f5a4c835f1a0e1022ec2f53. Conflicts: modules/gallery/models/item.php
Diffstat (limited to 'modules')
-rw-r--r--modules/gallery/models/item.php3
1 files changed, 2 insertions, 1 deletions
diff --git a/modules/gallery/models/item.php b/modules/gallery/models/item.php
index 45561380..c1d11a96 100644
--- a/modules/gallery/models/item.php
+++ b/modules/gallery/models/item.php
@@ -387,7 +387,8 @@ class Item_Model extends ORM_MPTT {
SELECT COUNT(*) AS position FROM {items}
WHERE parent_id = {$this->id}
AND `{$this->sort_column}` $comp (SELECT `{$this->sort_column}`
- FROM {items} WHERE id = $child_id)")->current()->position;
+ FROM {items} WHERE id = $child_id)
+ ORDER BY `{$this->sort_column}` {$this->sort_order}")->current()->position;
// We stopped short of our target value in the sort (notice that we're using a < comparator
// above) because it's possible that we have duplicate values in the sort column. An