summaryrefslogtreecommitdiff
path: root/modules/gallery/models
diff options
context:
space:
mode:
authorBharat Mediratta <bharat@menalto.com>2009-06-04 22:11:08 -0700
committerBharat Mediratta <bharat@menalto.com>2009-06-04 22:11:08 -0700
commit4f0a3fefa035ec351bc881093f49f9bc81941f3d (patch)
treea0a9631d33550b35db767809008ca84e238004f0 /modules/gallery/models
parent54927248b0b7b7e71b905c3be129305ba98e0456 (diff)
Fix a bug in Item_Model::get_position() where we incorrectly using the
grandparent id. Oops. This caused navigation from photo back up to album to be broken. Also update Photos_Controller to use the active sort order.. it was still hardcoded to use the id. It's more efficient now, yay. Fixes ticket #340.
Diffstat (limited to 'modules/gallery/models')
-rw-r--r--modules/gallery/models/item.php2
1 files changed, 1 insertions, 1 deletions
diff --git a/modules/gallery/models/item.php b/modules/gallery/models/item.php
index 10bad0b2..7dce9e51 100644
--- a/modules/gallery/models/item.php
+++ b/modules/gallery/models/item.php
@@ -380,7 +380,7 @@ class Item_Model extends ORM_MPTT {
public function get_position($child_id) {
$result = Database::instance()->query("
SELECT COUNT(*) AS position FROM {items}
- WHERE parent_id = {$this->parent_id}
+ WHERE parent_id = {$this->id}
AND {$this->sort_column} <= (SELECT {$this->sort_column}
FROM {items} WHERE id = $child_id)
ORDER BY {$this->sort_column} {$this->sort_order}");