From 2bdee22bed6325ad2e9ed469f0dd90a0b104adcd Mon Sep 17 00:00:00 2001
From: "Michael J. Rubinsky" <mrubinsk@horde.org>
Date: Mon, 3 Jan 2011 13:24:00 -0500
Subject: [PATCH] Add back navigation for subgalleries.

Now we correctly go back to the gallery's parent, if applicable,
otherwise to the gallerylist
---
 ansel/js/mobile.js                      | 25 +++++++++++++++++++++++--
 ansel/lib/Ajax/Application.php          | 19 +------------------
 ansel/lib/Gallery.php                   | 22 ++++++++++++++++++++++
 ansel/templates/mobile/gallery.html.php |  1 +
 4 files changed, 47 insertions(+), 20 deletions(-)

diff --git a/ansel/js/mobile.js b/ansel/js/mobile.js
index 6e55f5aa0..c5803b6b0 100644
--- a/ansel/js/mobile.js
+++ b/ansel/js/mobile.js
@@ -13,6 +13,11 @@
  */
 var AnselMobile = {
 
+    /**
+     * The currently displayed gallery
+     *
+     * @var object
+     */
     currentGallery: null,
 
     /**
@@ -65,14 +70,14 @@ var AnselMobile = {
     {
         // TODO: error checks, build any subgallery lists etc...
         if ($.mobile.currentPage != 'galleryview' &&
-            AnselMobile.currentGallery && (r.id == AnselMobile.currentGallery)) {
+            AnselMobile.currentGallery && (r.id == AnselMobile.currentGallery.id)) {
 
             $.mobile.changePage('galleryview', 'slide', false, true);
             return;
         }
 
         //AnselMobile.imgs = r.imgs;
-        AnselMobile.currentGallery = r.id;
+        AnselMobile.currentGallery = r;
         if (r.sg.length) {
             var l = $('<ul>').addClass('anselgalleries').attr({ 'data-role': 'listview', 'data-inset': 'true' });
             $('#thumbs').before(AnselMobile.buildGalleryList(l, r.sg).listview());
@@ -88,6 +93,13 @@ var AnselMobile = {
             var img = $('<li>').addClass('anselthumb').append($('<a>').attr({ 'href': '#' }).append($('<img>').attr({ src: i.url })));
             $('#thumbs').append(img);
         });
+        if (r.p) {
+            $('#gallerybackbutton .ui-btn-text').text(r.pn);
+            $('#gallerybackbutton').attr({ 'action': 'gallery', 'gallery-id': r.p });
+        } else {
+            $('#gallerybackbutton .ui-btn-text').text($.mobile.page.prototype.options.backBtnText);
+            $('#gallerybackbutton').attr({ 'action': 'home' });
+        }
         AnselMobile.centerGrid();
     },
 
@@ -127,6 +139,15 @@ var AnselMobile = {
             if (elt.hasClass('ansel-gallery')) {
                 AnselMobile.toGallery(elt.attr('ansel-gallery-id'));
             }
+            if (elt.attr('id') == 'gallerybackbutton') {
+                switch (elt.attr('action')) {
+                case 'home':
+                    $.mobile.changePage('gallerylist', 'slide', true, true);
+                    break;
+                case 'gallery':
+                    AnselMobile.toGallery(elt.attr('gallery-id'));
+                }
+            }
             elt = elt.parent();
         }
     },
diff --git a/ansel/lib/Ajax/Application.php b/ansel/lib/Ajax/Application.php
index a47199dd3..5deb7ee76 100644
--- a/ansel/lib/Ajax/Application.php
+++ b/ansel/lib/Ajax/Application.php
@@ -24,24 +24,7 @@ class Ansel_Ajax_Application extends Horde_Core_Ajax_Application
      *
      * @return mixed  False on failure, object representing the gallery with
      *                the following structure:
-     * <pre>
-     * 'id' - gallery id
-     * 'n'  - gallery name
-     * 'dc' - date created
-     * 'dm' - date modified
-     * 'd'  - description
-     * 'ki' - key image
-     * 'sg' - an object with the following properties:
-     *      'n'  - gallery name
-     *      'dc' - date created
-     *      'dm' - date modified
-     *      'd'  - description
-     *      'ki' - key image
-     *
-     *  'imgs' - an array of image objects with the following properties:
-     *      'id'  - the image id
-     *      'url' - the image url
-     * </pre>
+     * @see Ansel_Gallery::toJson()
      */
     public function getGallery()
     {
diff --git a/ansel/lib/Gallery.php b/ansel/lib/Gallery.php
index 37ce737f1..a28ff897b 100644
--- a/ansel/lib/Gallery.php
+++ b/ansel/lib/Gallery.php
@@ -988,6 +988,26 @@ class Ansel_Gallery extends Horde_Share_Object_Sql_Hierarchical implements Seria
      * @param boolean $full  Return all information (subgalleries and images)?
      *
      * @return StdClass  An object describing the gallery
+     * <pre>
+     * 'id' - gallery id
+     * 'p'  - gallery's parent's id (null if top level)
+     * 'pn' - gallery's parent's name (null if top level)
+     * 'n'  - gallery name
+     * 'dc' - date created
+     * 'dm' - date modified
+     * 'd'  - description
+     * 'ki' - key image
+     * 'sg' - an object with the following properties:
+     *      'n'  - gallery name
+     *      'dc' - date created
+     *      'dm' - date modified
+     *      'd'  - description
+     *      'ki' - key image
+     *
+     *  'imgs' - an array of image objects with the following properties:
+     *      'id'  - the image id
+     *      'url' - the image url
+     * </pre>
      */
     public function toJson($full = false)
     {
@@ -1003,9 +1023,11 @@ class Ansel_Gallery extends Horde_Share_Object_Sql_Hierarchical implements Seria
         $parents = $this->get('parents');
         if (empty($parents)) {
             $json->p = null;
+            $json->pn - null;
         } else {
             $parents = explode(':', $parents);
             $json->p = array_pop($parents);
+            $json->pn = $GLOBALS['injector']->getInstance('Ansel_Storage')->getGallery($json->p)->get('name');
         }
 
         if ($full) {
diff --git a/ansel/templates/mobile/gallery.html.php b/ansel/templates/mobile/gallery.html.php
index 9097e9ad9..bfa434ef0 100644
--- a/ansel/templates/mobile/gallery.html.php
+++ b/ansel/templates/mobile/gallery.html.php
@@ -1,5 +1,6 @@
 <div data-role="page" data-theme="h" id="galleryview">
   <div data-role="header">
+    <a data-role="button" data-icon="arrow-l" href="#" id="gallerybackbutton"></a>
     <h1></h1>
   </div>
   <div id="anselgalleryview">
-- 
2.11.0