From 29efb6ba9f3e2cf15f29b509f985890c33bc08fa Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Sat, 14 Nov 2009 16:18:49 -0800
Subject: Rename "pager" to "paginator" so that we differentiate page.html.php
 from paginator.html.php

---
 modules/gallery/libraries/Theme_View.php | 16 +++---
 modules/search/views/search.html.php     |  2 +-
 themes/wind/views/album.html.php         |  2 +-
 themes/wind/views/dynamic.html.php       |  2 +-
 themes/wind/views/movie.html.php         |  2 +-
 themes/wind/views/pager.html.php         | 82 ------------------------------
 themes/wind/views/paginator.html.php     | 86 ++++++++++++++++++++++++++++++++
 themes/wind/views/photo.html.php         |  2 +-
 8 files changed, 100 insertions(+), 94 deletions(-)
 delete mode 100644 themes/wind/views/pager.html.php
 create mode 100644 themes/wind/views/paginator.html.php

diff --git a/modules/gallery/libraries/Theme_View.php b/modules/gallery/libraries/Theme_View.php
index 5e8bc728..19dc0829 100644
--- a/modules/gallery/libraries/Theme_View.php
+++ b/modules/gallery/libraries/Theme_View.php
@@ -140,26 +140,28 @@ class Theme_View_Core extends Gallery_View {
    *
    * See themes/wind/views/pager.html for documentation on the variables generated here.
    */
-  public function pager() {
-    $v = new View("pager.html");
+  public function paginator() {
+    $v = new View("paginator.html");
     $v->page_type = $this->page_type;
     $v->first_page_url = null;
     $v->previous_page_url = null;
     $v->next_page_url = null;
     $v->last_page_url = null;
 
-    if ($this->page_type == "album") {
+    if ($this->page_type == "album" || $this->page_type = "tag") {
       $v->page = $this->page;
       $v->max_pages = $this->max_pages;
       $v->total = $this->children_count;
+
+      $model = $this->page_type == "album" ? $this->item : $this->tag;
       if ($this->page != 1) {
-        $v->first_page_url = $this->item->url();
-        $v->previous_page_url = $this->item->url("page=" . ($this->page - 1));
+        $v->first_page_url = $model->url();
+        $v->previous_page_url = $model->url("page=" . ($this->page - 1));
       }
 
       if ($this->page != $this->max_pages) {
-        $v->next_page_url = $this->item->url("page=" . ($this->page + 1));
-        $v->last_page_url = $this->item->url("page={$this->max_pages}");
+        $v->next_page_url = $model->url("page=" . ($this->page + 1));
+        $v->last_page_url = $model->url("page={$this->max_pages}");
       }
 
       $v->first_visible_position = ($this->page - 1) * $this->page_size + 1;
diff --git a/modules/search/views/search.html.php b/modules/search/views/search.html.php
index 4b67157e..fdf22a9e 100644
--- a/modules/search/views/search.html.php
+++ b/modules/search/views/search.html.php
@@ -40,7 +40,7 @@
     </li>
     <? endforeach ?>
   </ul>
-  <?= $theme->pager() ?>
+  <?= $theme->paginator() ?>
 
   <? else: ?>
   <p>
diff --git a/themes/wind/views/album.html.php b/themes/wind/views/album.html.php
index 1163630d..2c2b54eb 100644
--- a/themes/wind/views/album.html.php
+++ b/themes/wind/views/album.html.php
@@ -38,4 +38,4 @@
 </ul>
 <?= $theme->album_bottom() ?>
 
-<?= $theme->pager() ?>
+<?= $theme->paginator() ?>
diff --git a/themes/wind/views/dynamic.html.php b/themes/wind/views/dynamic.html.php
index 51e48dc0..a8a4d362 100644
--- a/themes/wind/views/dynamic.html.php
+++ b/themes/wind/views/dynamic.html.php
@@ -26,4 +26,4 @@
 </ul>
 <?= $theme->dynamic_bottom() ?>
 
-<?= $theme->pager() ?>
+<?= $theme->paginator() ?>
diff --git a/themes/wind/views/movie.html.php b/themes/wind/views/movie.html.php
index a44b891a..27c293ce 100644
--- a/themes/wind/views/movie.html.php
+++ b/themes/wind/views/movie.html.php
@@ -2,7 +2,7 @@
 <div id="g-item">
   <?= $theme->photo_top() ?>
 
-  <?= $theme->pager() ?>
+  <?= $theme->paginator() ?>
 
   <div id="g-movie" class="ui-helper-clearfix">
     <?= $item->movie_img(array("class" => "g-movie", "id" => "g-movie-id-{$item->id}")) ?>
diff --git a/themes/wind/views/pager.html.php b/themes/wind/views/pager.html.php
deleted file mode 100644
index 51d52021..00000000
--- a/themes/wind/views/pager.html.php
+++ /dev/null
@@ -1,82 +0,0 @@
-<?php defined("SYSPATH") or die("No direct script access.") ?>
-<?
-// This is a generic paginator for album, photo and movie pages.  Depending on the page type,
-// there are different sets of variables available.  With this data, you can make a paginator
-// that lets you say "You're viewing photo 5 of 35", or "You're viewing photos 10 - 18 of 37"
-// for album views.
-//
-// Available variables for all page types:
-//   $page_type               - "album", "movie" or "photo"
-//   $previous_page_url       - the url to the previous page, if there is one
-//   $next_page_url           - the url to the next page, if there is one
-//   $total                   - the total number of photos in this album
-//
-// Available for the "album" page type:
-//   $page                    - what page number we're on
-//   $max_pages               - the maximum page number
-//   $page_size               - the page size
-//   $first_page_url          - the url to the first page, or null if we're on the first page
-//   $last_page_url           - the url to the last page, or null if we're on the last page
-//   $first_visible_position  - the position number of the first visible photo on this page
-//   $last_visible_position   - the position number of the last visible photo on this page
-//
-// Available for "photo" and "movie" page types:
-//   $position                - the position number of this photo
-?>
-
-<ul class="g-pager ui-helper-clearfix">
-  <li class="g-first">
-  <? if ($page_type == "album"): ?>
-    <? if (isset($first_page_url)): ?>
-      <a href="<?= $first_page_url ?>" class="g-button ui-icon-left ui-state-default ui-corner-all">
-        <span class="ui-icon ui-icon-seek-first"></span><?= t("First") ?></a>
-    <? else: ?>
-      <a class="g-button ui-icon-left ui-state-disabled ui-corner-all">
-        <span class="ui-icon ui-icon-seek-first"></span><?= t("First") ?></a>
-    <? endif ?>
-  <? endif ?>
-
-  <? if (isset($previous_page_url)): ?>
-    <a href="<?= $previous_page_url ?>" class="g-button ui-icon-left ui-state-default ui-corner-all">
-      <span class="ui-icon ui-icon-seek-prev"></span><?= t("Previous") ?></a>
-  <? else: ?>
-    <a class="g-button ui-icon-left ui-state-disabled ui-corner-all">
-      <span class="ui-icon ui-icon-seek-prev"></span><?= t("Previous") ?></a>
-  <? endif ?>
-  </li>
-
-  <li class="g-info">
-    <? if ($page_type == "album"): ?>
-      <?= /* @todo This message isn't easily localizable */
-          /* @todo does this really need to be a t2?  why not just skip the msg when there's 1 photo? */
-          t2("Photo %from_number of %count",
-             "Photos %from_number - %to_number of %count",
-             $total,
-             array("from_number" => $first_visible_position,
-                   "to_number" => $last_visible_position,
-                   "count" => $total)) ?>
-    <? else: ?>
-      <?= t("%position of %total", array("position" => $position, "total" => $total)) ?>
-    <? endif ?>
-  </li>
-
-  <li class="g-text-right">
-  <? if (isset($next_page_url)): ?>
-    <a href="<?= $next_page_url ?>" class="g-button ui-icon-right ui-state-default ui-corner-all">
-      <span class="ui-icon ui-icon-seek-next"></span><?= t("Next") ?></a>
-  <? else: ?>
-    <a class="g-button ui-state-disabled ui-icon-right ui-corner-all">
-      <span class="ui-icon ui-icon-seek-next"></span><?= t("Next") ?></a>
-  <? endif ?>
-
-  <? if ($page_type == "album"): ?>
-    <? if (isset($last_page_url)): ?>
-      <a href="<?= $last_page_url ?>" class="g-button ui-icon-right ui-state-default ui-corner-all">
-        <span class="ui-icon ui-icon-seek-end"></span><?= t("Last") ?></a>
-    <? else: ?>
-      <a class="g-button ui-state-disabled ui-icon-right ui-corner-all">
-        <span class="ui-icon ui-icon-seek-end"></span><?= t("Last") ?></a>
-    <? endif ?>
-  <? endif ?>
-  </li>
-</ul>
diff --git a/themes/wind/views/paginator.html.php b/themes/wind/views/paginator.html.php
new file mode 100644
index 00000000..5d300cf4
--- /dev/null
+++ b/themes/wind/views/paginator.html.php
@@ -0,0 +1,86 @@
+<?php defined("SYSPATH") or die("No direct script access.") ?>
+<?
+// This is a generic paginator for album, photo and movie pages.  Depending on the page type,
+// there are different sets of variables available.  With this data, you can make a paginator
+// that lets you say "You're viewing photo 5 of 35", or "You're viewing photos 10 - 18 of 37"
+// for album views.
+//
+// Available variables for all page types:
+//   $page_type               - "album", "movie", "photo" or "tag"
+//   $previous_page_url       - the url to the previous page, if there is one
+//   $next_page_url           - the url to the next page, if there is one
+//   $total                   - the total number of photos in this album
+//
+// Available for the "album" and "tag" page types:
+//   $page                    - what page number we're on
+//   $max_pages               - the maximum page number
+//   $page_size               - the page size
+//   $first_page_url          - the url to the first page, or null if we're on the first page
+//   $last_page_url           - the url to the last page, or null if we're on the last page
+//   $first_visible_position  - the position number of the first visible photo on this page
+//   $last_visible_position   - the position number of the last visible photo on this page
+//
+// Available for "photo" and "movie" page types:
+//   $position                - the position number of this photo
+//
+?>
+
+<ul class="g-pager ui-helper-clearfix">
+  <li class="g-first">
+  <? if ($page_type == "album" || $page_type == "tag"): ?>
+    <? if (isset($first_page_url)): ?>
+      <a href="<?= $first_page_url ?>" class="g-button ui-icon-left ui-state-default ui-corner-all">
+        <span class="ui-icon ui-icon-seek-first"></span><?= t("First") ?></a>
+    <? else: ?>
+      <a class="g-button ui-icon-left ui-state-disabled ui-corner-all">
+        <span class="ui-icon ui-icon-seek-first"></span><?= t("First") ?></a>
+    <? endif ?>
+  <? endif ?>
+
+  <? if (isset($previous_page_url)): ?>
+    <a href="<?= $previous_page_url ?>" class="g-button ui-icon-left ui-state-default ui-corner-all">
+      <span class="ui-icon ui-icon-seek-prev"></span><?= t("Previous") ?></a>
+  <? else: ?>
+    <a class="g-button ui-icon-left ui-state-disabled ui-corner-all">
+      <span class="ui-icon ui-icon-seek-prev"></span><?= t("Previous") ?></a>
+  <? endif ?>
+  </li>
+
+  <li class="g-info">
+    <? if ($total): ?>
+      <? if ($page_type == "album" || $page_type == "tag"): ?>
+        <?= /* @todo This message isn't easily localizable */
+            t2("Photo %from_number of %count",
+               "Photos %from_number - %to_number of %count",
+               $total,
+               array("from_number" => $first_visible_position,
+                     "to_number" => $last_visible_position,
+                     "count" => $total)) ?>
+      <? else: ?>
+        <?= t("%position of %total", array("position" => $position, "total" => $total)) ?>
+      <? endif ?>
+    <? else: ?>
+      <?= t("No photos") ?>
+    <? endif ?>
+  </li>
+
+  <li class="g-text-right">
+  <? if (isset($next_page_url)): ?>
+    <a href="<?= $next_page_url ?>" class="g-button ui-icon-right ui-state-default ui-corner-all">
+      <span class="ui-icon ui-icon-seek-next"></span><?= t("Next") ?></a>
+  <? else: ?>
+    <a class="g-button ui-state-disabled ui-icon-right ui-corner-all">
+      <span class="ui-icon ui-icon-seek-next"></span><?= t("Next") ?></a>
+  <? endif ?>
+
+  <? if ($page_type == "album" || $page_type == "tag"): ?>
+    <? if (isset($last_page_url)): ?>
+      <a href="<?= $last_page_url ?>" class="g-button ui-icon-right ui-state-default ui-corner-all">
+        <span class="ui-icon ui-icon-seek-end"></span><?= t("Last") ?></a>
+    <? else: ?>
+      <a class="g-button ui-state-disabled ui-icon-right ui-corner-all">
+        <span class="ui-icon ui-icon-seek-end"></span><?= t("Last") ?></a>
+    <? endif ?>
+  <? endif ?>
+  </li>
+</ul>
diff --git a/themes/wind/views/photo.html.php b/themes/wind/views/photo.html.php
index 091fd7c5..e0fae3f1 100644
--- a/themes/wind/views/photo.html.php
+++ b/themes/wind/views/photo.html.php
@@ -15,7 +15,7 @@
 <div id="g-item">
   <?= $theme->photo_top() ?>
 
-  <?= $theme->pager() ?>
+  <?= $theme->paginator() ?>
 
   <div id="g-photo">
     <?= $theme->resize_top($item) ?>
-- 
cgit v1.2.3