Extend siblings callbacks to take a $limit and an $offset for navigating

large sibling sets.  Useful for the thumbnav module since we don't want to
iterate a thousand siblings to find the one we care about.  Fixes #1999.

2 files changed, 6 insertions, 4 deletions

diff --git a/modules/gallery/controllers/albums.php b/modules/gallery/controllers/albums.php
index d545b415..0fb033a8 100644
--- a/modules/gallery/controllers/albums.php
+++ b/modules/gallery/controllers/albums.php
@@ -98,9 +98,9 @@ class Albums_Controller extends Items_Controller {
                  "breadcrumbs" => Breadcrumb::array_from_item_parents($item));
   }
 
-  static function get_siblings($item) {
+  static function get_siblings($item, $limit=null, $offset=null) {
     // @todo consider creating Item_Model::siblings() if we use this more broadly.
-    return $item->parent()->viewable()->children();
+    return $item->parent()->viewable()->children($limit, $offset);
   }
 
   public function create($parent_id) {
diff --git a/modules/gallery/libraries/Theme_View.php b/modules/gallery/libraries/Theme_View.php
index cf384109..986fc8a2 100644
--- a/modules/gallery/libraries/Theme_View.php
+++ b/modules/gallery/libraries/Theme_View.php
@@ -86,8 +86,10 @@ class Theme_View_Core extends Gallery_View {
     return $this->item;
   }
 
-  public function siblings() {
-    return call_user_func_array($this->siblings_callback[0], $this->siblings_callback[1]);
+  public function siblings($limit=null, $offset=null) {
+    return call_user_func_array(
+      $this->siblings_callback[0],
+      array_merge($this->siblings_callback[1], array($offset, $limit)));
   }
 
   public function tag() {