diff options
Diffstat (limited to 'modules')
| -rw-r--r-- | modules/tag/controllers/tags.php | 15 | ||||
| -rw-r--r-- | modules/tag/models/tag.php | 18 |
2 files changed, 26 insertions, 7 deletions
diff --git a/modules/tag/controllers/tags.php b/modules/tag/controllers/tags.php index ee39bfec..94890639 100644 --- a/modules/tag/controllers/tags.php +++ b/modules/tag/controllers/tags.php @@ -23,15 +23,20 @@ class Tags_Controller extends REST_Controller { public function _show($tag) { $theme_name = module::get_var("core", "active_theme", "default"); $page_size = module::get_var("core", "page_size", 9); - - $template = new Theme_View("page.html", "tag", $theme_name); - $page = $this->input->get("page", "1"); + $children_count = $tag->items_count(); + $offset = ($page-1) * $page_size; + // Make sure that the page references a valid offset + if ($page < 1 || $page > ceil($children_count / $page_size)) { + Kohana::show_404(); + } + + $template = new Theme_View("page.html", "tag", $theme_name); $template->set_global('page_size', $page_size); $template->set_global('tag', $tag); - $template->set_global('children', $tag->items($page_size, ($page-1) * $page_size)); - $template->set_global('children_count', $tag->count); + $template->set_global('children', $tag->items($page_size, $offset)); + $template->set_global('children_count', $children_count); $template->content = new View("tag.html"); print $template; diff --git a/modules/tag/models/tag.php b/modules/tag/models/tag.php index de892db2..37fd92fb 100644 --- a/modules/tag/models/tag.php +++ b/modules/tag/models/tag.php @@ -24,14 +24,15 @@ class Tag_Model extends ORM { "name" => "required|length[4,32]"); /** - * Return all items associated with this tag. - * @param string $type the type of item (album, photo) + * Return all viewable items associated with this tag. * @param integer $limit number of rows to limit result to * @param integer $offset offset in result to start returning rows from + * @param string $type the type of item (album, photo) * @return ORM_Iterator */ public function items($limit=null, $offset=0, $type=null) { $model = ORM::factory("item") + ->viewable() ->join("items_tags", "items.id", "items_tags.item_id") ->where("items_tags.tag_id", $this->id); if ($type) { @@ -39,4 +40,17 @@ class Tag_Model extends ORM { } return $model->find_all($limit, $offset); } + + /** + * Return the count of all viewable items associated with this tag. + * @param string $type the type of item (album, photo) + * @return integer + */ + public function items_count($type=null) { + return ORM::factory("item") + ->viewable() + ->join("items_tags", "items.id", "items_tags.item_id") + ->where("items_tags.tag_id", $this->id) + ->count_all(); + } }
\ No newline at end of file |