diff options
author | Andy Staudacher <andy.st@gmail.com> | 2009-07-27 11:25:23 -0700 |
---|---|---|
committer | Andy Staudacher <andy.st@gmail.com> | 2009-07-27 11:25:23 -0700 |
commit | d18f31ab218c8d79d97b19079ac0078e6155bc39 (patch) | |
tree | a4b37e831961540ee93e93f31f03d4b9bf984cc1 /modules | |
parent | 482e223b602d60eec424081d0328cde66d28a758 (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.php | 3 |
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 |