diff options
| -rw-r--r-- | core/config/routes.php | 6 | ||||
| -rw-r--r-- | core/controllers/album.php | 10 | ||||
| -rw-r--r-- | core/controllers/item.php | 7 | ||||
| -rw-r--r-- | core/controllers/photo.php | 10 | ||||
| -rw-r--r-- | core/controllers/rest.php | 24 | ||||
| -rw-r--r-- | modules/comment/controllers/comment.php | 88 | ||||
| -rw-r--r-- | modules/comment/controllers/comments.php | 76 | ||||
| -rw-r--r-- | modules/comment/helpers/comment.php | 74 | ||||
| -rw-r--r-- | modules/comment/views/comment.html.php (renamed from modules/comment/views/comment_list.html.php) | 6 | ||||
| -rw-r--r-- | modules/comment/views/comment_form.html.php | 35 | ||||
| -rw-r--r-- | modules/comment/views/show_comments.html.php | 11 | ||||
| -rw-r--r-- | themes/default/js/comment.js | 20 | ||||
| -rw-r--r-- | themes/default/views/page.html.php | 1 | ||||
| -rw-r--r-- | themes/default/views/photo.html.php | 2 |
14 files changed, 230 insertions, 140 deletions
diff --git a/core/config/routes.php b/core/config/routes.php index de66b8fb..465fb885 100644 --- a/core/config/routes.php +++ b/core/config/routes.php @@ -24,11 +24,7 @@ $config['^rest'] = null; $config['^rest/.*'] = null; $config['^(\w+)/(\d+)$'] = '$1/dispatch/$2'; +$config['^form/(\w+)/(.*)$'] = '$1/form/$2'; // For now our default page is the scaffolding. $config['_default'] = 'welcome'; - -// Special routes for the comment module. -// @todo Dynamically load this. -$config['photo/(\d+)/comments/add$'] = 'comment/add/$1'; -$config['photo/(\d+)/comments$'] = 'comment/get_item_comments/$1'; diff --git a/core/controllers/album.php b/core/controllers/album.php index 63290853..87e27ac5 100644 --- a/core/controllers/album.php +++ b/core/controllers/album.php @@ -19,6 +19,16 @@ */ class Album_Controller extends Item_Controller { + /** + * @see Rest_Controller::_form($resource) + */ + public function _form($comment) { + throw new Exception("@todo Comment_Controller::_get NOT IMPLEMENTED"); + } + + /** + * @see Rest_Controller::_get($resource) + */ public function _get($item) { // @todo: these need to be pulled from the database $theme_name = "default"; diff --git a/core/controllers/item.php b/core/controllers/item.php index e9ff03e6..027dcada 100644 --- a/core/controllers/item.php +++ b/core/controllers/item.php @@ -20,6 +20,13 @@ class Item_Controller extends REST_Controller { protected $resource_type = "item"; + /** + * @see Rest_Controller::_form($resource) + */ + public function _form($comment) { + throw new Exception("@todo Comment_Controller::_get NOT IMPLEMENTED"); + } + public function _get($item) { // Redirect to the more specific resource type, since it will render // differently. We could also just delegate here, but it feels more appropriate diff --git a/core/controllers/photo.php b/core/controllers/photo.php index 78287afa..4c3154bd 100644 --- a/core/controllers/photo.php +++ b/core/controllers/photo.php @@ -19,6 +19,16 @@ */ class Photo_Controller extends Item_Controller { + /** + * @see Rest_Controller::_form($resource) + */ + public function _form($comment) { + throw new Exception("@todo Comment_Controller::_get NOT IMPLEMENTED"); + } + + /** + * @see Rest_Controller::_get($resource) + */ public function _get($item) { $template = new View("page.html"); diff --git a/core/controllers/rest.php b/core/controllers/rest.php index 09c85653..4a6c4eb4 100644 --- a/core/controllers/rest.php +++ b/core/controllers/rest.php @@ -39,6 +39,10 @@ * public function _delete(ORM $comment) { * // Handle DELETE request * } + * + * public function form(ORM $comment) { + * // Show a form for creating a new comment + * } * } * * A request to http://example.com/gallery3/comment/3 will result in a call to @@ -85,6 +89,20 @@ abstract class REST_Controller extends Controller { } } + public function form($id) { + if ($this->resource_type == null) { + throw new Exception("@todo ERROR_MISSING_RESOURCE_TYPE"); + } + + // @todo this needs security checks + $resource = ORM::factory($this->resource_type, $id); + if (!$resource->loaded) { + return Kohana::show_404(); + } + + return $this->_form($resource); + } + /** * Perform a GET request on this resource * @param ORM $resource the instance of this resource type @@ -108,4 +126,10 @@ abstract class REST_Controller extends Controller { * @param ORM $resource the instance of this resource type */ abstract public function _delete($resource); + + /** + * Present a form for adding a new resource + * @param ORM $resource the resource container for instances of this resource type + */ + abstract public function _form($resource); } diff --git a/modules/comment/controllers/comment.php b/modules/comment/controllers/comment.php index d2135cf9..a8f52443 100644 --- a/modules/comment/controllers/comment.php +++ b/modules/comment/controllers/comment.php @@ -21,50 +21,29 @@ class Comment_Controller extends REST_Controller { protected $resource_type = "comment"; /** - * Return the form for adding comments. + * Present a form for editing a comment + * @see Rest_Controller::form($resource) */ - public function _get_form($comment) { - $form = new Forge(url::current(true), "", "post", array("id" => "gComment")); - $group = $form->group(_("Add Comment")); - $group->input("author") - ->label(_("Author")) - ->id("gAuthor") - ->class(null) - ->value($comment->author); - $group->input("email") - ->label(_("Email")) - ->id("gEmail") - ->class(null) - ->value($comment->email); - $group->textarea("text") - ->label(_("Text")) - ->id("gText") - ->class(null) - ->value($comment->text); - $group->hidden("item_id") - ->value($comment->item_id); - $group->submit(_("Add")); - - $this->_add_validation_rules(ORM::factory("comment")->validation_rules, $form); - - return $form; + public function _form($comment) { + $form = comment::get_edit_form($comment); + print $form->render("form.html"); } /** - * @todo Refactor this into a more generic location + * Get an existing comment. + * @see Rest_Controller::_get($resource) */ - private function _add_validation_rules($rules, $form) { - foreach ($form->inputs as $name => $input) { - if (isset($input->inputs)) { - $this->_add_validation_rules($rules, $input); - } - if (isset($rules[$name])) { - $input->rules($rules[$name]); - } - } + public function _get($comment) { + $v = new View("comment.html"); + $v->comment = $comment; + print $v; } - public function add($item_id) { + /** + * Update existing comment. + * @see Rest_Controller::_put($resource) + */ + public function _put($comment) { $comment = ORM::factory('comment'); $comment->item_id = $item_id; @@ -77,43 +56,16 @@ class Comment_Controller extends REST_Controller { $comment->datetime = time(); $comment->item_id = $this->input->post('item_id'); $comment->save(); - - $data = array('valid' => true, 'html' => sprintf(comment::show_comment_list($item_id))); - } else { - $data = array('valid' => false, 'html' => sprintf($form->render())); + return; } - - if (request::method() == "get") { - print $data['html']; - } else if (request::method() == "post") { - print json_encode($data); - } - } - - public function get_item_comments($item_id) { - print comment::show_comment_list($item_id); - } - - /** - * Get an existing comment. - * @see Rest_Controller::_get($resource) - */ - public function _get($user) { - throw new Exception("@todo Comment_Controller::_get NOT IMPLEMENTED"); - } - - /** - * Update existing comment. - * @see Rest_Controller::_put($resource) - */ - public function _put($resource) { - throw new Exception("@todo Comment_Controller::_put NOT IMPLEMENTED"); + print $form->render(); } /** + * Add a new comment * @see Rest_Controller::_post($resource) */ - public function _post($user) { + public function _post($comment) { throw new Exception("@todo Comment_Controller::_post NOT IMPLEMENTED"); } diff --git a/modules/comment/controllers/comments.php b/modules/comment/controllers/comments.php new file mode 100644 index 00000000..a0233a0b --- /dev/null +++ b/modules/comment/controllers/comments.php @@ -0,0 +1,76 @@ +<?php defined("SYSPATH") or die("No direct script access."); +/** + * Gallery - a web based photo album viewer and editor + * Copyright (C) 2000-2008 Bharat Mediratta + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or (at + * your option) any later version. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA. + */ +class Comments_Controller extends REST_Controller { + protected $resource_type = "item"; + + /** + * Present a form for adding a new comment to this item + * @see Rest_Controller::form($resource) + */ + public function _form($item) { + $form = comment::get_add_form($item); + print $form->render("form.html"); + } + + /** + * Show the comment collection + * @see Rest_Controller::_get($resource) + */ + public function _get($item) { + print comment::get_comments($item); + } + + /** + * Update existing comment collection. + * @see Rest_Controller::_put($resource) + */ + public function _put($item) { + throw new Exception("@todo Comment_Controller::_put NOT IMPLEMENTED"); + } + + /** + * Add a new comment to the collection + * @see Rest_Controller::_post($resource) + */ + public function _post($item) { + $form = comment::get_add_form($item); + if ($form->validate()) { + $comment = ORM::factory('comment'); + $comment->author = $this->input->post('author'); + $comment->email = $this->input->post('email'); + $comment->text = $this->input->post('text'); + $comment->datetime = time(); + $comment->item_id = $item->id; + $comment->save(); + + header("HTTP/1.1 201 Created"); + header("Location: " . url::site("comment/{$comment->id}")); + } + print $form->render("form.html"); + } + + /** + * Delete existing comment collection. + * @see Rest_Controller::_delete($resource) + */ + public function _delete($item) { + throw new Exception("@todo Comment_Controller::_delete NOT IMPLEMENTED"); + } +} diff --git a/modules/comment/helpers/comment.php b/modules/comment/helpers/comment.php index 9eed8451..1c876f10 100644 --- a/modules/comment/helpers/comment.php +++ b/modules/comment/helpers/comment.php @@ -54,26 +54,68 @@ class Comment_Core { return $comment->save(); } - static function show_comments($item_id) { - $v = new View('show_comments.html'); - $v->comment_list = Comment::show_comment_list($item_id); - $v->comment_form = Comment::show_comment_form($item_id); - $v->render(true); + static function get_add_form($item) { + $form = new Forge(url::site("comments/{$item->id}"), "", "post", array("id" => "gCommentForm")); + $group = $form->group(_("Add Comment")); + $group->input("author") ->label(_("Author")) ->id("gAuthor"); + $group->input("email") ->label(_("Email")) ->id("gEmail"); + $group->textarea("text")->label(_("Text")) ->id("gText"); + $group->submit(_("Add")); + + comment::_add_validation_rules("comment", $form); + return $form; + } + + static function get_edit_form($comment) { + $form = new Forge(url::site("comment/{$comment->id}"), "", "post", array("id" => "gCommentForm")); + $group = $form->group(_("Edit Comment")); + $group->input("author") ->label(_("Author")) ->id("gAuthor") ->value($comment->author); + $group->input("email") ->label(_("Email")) ->id("gEmail") ->value($comment->email); + $group->textarea("text")->label(_("Text")) ->id("gText") ->value($comment->text); + $group->submit(_("Edit")); + + comment::_add_validation_rules("comment", $form); + return $form; + } + + /** + * @todo Refactor this into a more generic location + */ + private static function _add_validation_rules($model_name, $form) { + $rules = ORM::factory($model_name)->validation_rules; + foreach ($form->inputs as $name => $input) { + if (isset($input->inputs)) { + comment::_add_validation_rules($model_name, $input); + } + if (isset($rules[$name])) { + $input->rules($rules[$name]); + } + } } - static function show_comment_list($item_id) { - $v = new View('comment_list.html'); - $v->item_id = $item_id; - $v->comments = ORM::factory('comment')->where('item_id', $item_id) - ->orderby('datetime', 'desc') - ->find_all()->as_array(); - return $v; + static function block($theme, $show_add_form=true) { + $block = new Block; + $block->id = "gComment"; + $block->title = _("Comments"); + $block->content = comment::get_comments($theme->item()); + + if ($show_add_form) { + $block->content .= comment::get_add_form($theme->item())->render("form.html"); + } + return $block; } - static function show_comment_form($item_id) { - $v = new View('comment_form.html'); - $v->item_id = $item_id; - return $v; + static function get_comments($item) { + $comments = array("<ul>"); + foreach (ORM::factory('comment')->where('item_id', $item->id) + ->orderby('datetime', 'asc') + ->find_all() as $comment) { + $v = new View("comment.html"); + $v->comment = $comment; + $comments[] = $v; + } + $comments[] = "</ul>"; + return implode("\n", $comments); } /** diff --git a/modules/comment/views/comment_list.html.php b/modules/comment/views/comment.html.php index 16371541..2b306404 100644 --- a/modules/comment/views/comment_list.html.php +++ b/modules/comment/views/comment.html.php @@ -1,13 +1,11 @@ <? defined("SYSPATH") or die("No direct script access."); ?> -<? foreach (array_reverse($comments) as $index => $comment): ?> -<li id="gComment-<?= $index; ?>" class="gComment <?= text::alternate("gEven", "gOdd") ?>"> +<li id="gComment-<?= $comment->id; ?>"> <p> <a href="#" class="gAuthor"><?= $comment->author ?></a> <?= comment::format_elapsed_time($comment->datetime) ?>, <span class="gUnderstate"><?= strftime('%c', $comment->datetime) ?></span> </p> <div> - <?= $comment->text ?> + <?= $comment->text ?> </div> </li> -<? endforeach; ?> diff --git a/modules/comment/views/comment_form.html.php b/modules/comment/views/comment_form.html.php deleted file mode 100644 index 86c76dba..00000000 --- a/modules/comment/views/comment_form.html.php +++ /dev/null @@ -1,35 +0,0 @@ -<? defined("SYSPATH") or die("No direct script access."); ?> -<script type="text/javascript"> - // <![CDATA[ -function show_comment_add_form(url) { - $("#gCommentAddLink").hide(); - $.get(url, function(data) { - $("#gCommentAddFormContainer").html(data); - ajaxify_comment_add_form(); - }); -} - -function ajaxify_comment_add_form() { - $("form#gComment").ajaxForm({ - dataType: 'json', - success: function(response_data, status_text) { - if (response_data['valid']) { - $("#gCommentThread").html(response_data["html"]); - $("#gCommentAddFormContainer").html(""); - $("#gCommentAddLink").show(); - } else { - $("#gCommentAddFormContainer").html(response_data["html"]); - ajaxify_comment_add_form(); - } - }, - }); -} - // ]]> -</script> -<span id="gCommentAddLink"> - <a href="javascript:show_comment_add_form('<?= url::site("photo/{$item_id}/comments/add") ?>')"> - <?= _("Add Comment") ?> - </a> -</span> -<div id="gCommentAddFormContainer"></div> - diff --git a/modules/comment/views/show_comments.html.php b/modules/comment/views/show_comments.html.php deleted file mode 100644 index 35b2c67a..00000000 --- a/modules/comment/views/show_comments.html.php +++ /dev/null @@ -1,11 +0,0 @@ -<? defined("SYSPATH") or die("No direct script access."); ?> -<div id="gComments"> - <? if ($comment_list): ?> - <h2><?= _("Comments") ?></h2> - <ul id="gCommentThread"> - <?= $comment_list ?> - </ul> - <? endif ?> - - <?= $comment_form ?> -</div> diff --git a/themes/default/js/comment.js b/themes/default/js/comment.js new file mode 100644 index 00000000..5d2ed599 --- /dev/null +++ b/themes/default/js/comment.js @@ -0,0 +1,20 @@ +$("document").ready(function() { + ajaxify_comment_form(); +}); + +function ajaxify_comment_form() { + $("#gCommentForm").ajaxForm({ + target: "#gCommentForm", + complete: function(xhr, statusText) { + ajaxify_comment_form(); + if (xhr.status == 201) { + $.get(xhr.getResponseHeader("Location"), function(data, textStatus) { + $("#gComment div.gBlockContent ul:first").append(data); + $("#gComment div.gBlockContent ul:first li:last").hide().slideDown(); + } + ); + $("#gCommentForm").clearForm(); + } + } + }); +} diff --git a/themes/default/views/page.html.php b/themes/default/views/page.html.php index ac2a326b..75c9a1a9 100644 --- a/themes/default/views/page.html.php +++ b/themes/default/views/page.html.php @@ -14,6 +14,7 @@ <script src="<?= url::file("lib/jquery.js") ?>" type="text/javascript"></script> <script src="<?= url::file("lib/jquery.form.js") ?>" type="text/javascript"></script> <script src="<?= $theme->url("js/user.js") ?>" type="text/javascript"></script> + <script src="<?= $theme->url("js/comment.js") ?>" type="text/javascript"></script> <? if ($user): ?> <script src="<?= url::file("lib/jquery.jeditable.js") ?>" type="text/javascript"></script> <? endif; ?> diff --git a/themes/default/views/photo.html.php b/themes/default/views/photo.html.php index 1d54fe7d..8b8f44d4 100644 --- a/themes/default/views/photo.html.php +++ b/themes/default/views/photo.html.php @@ -9,5 +9,5 @@ <h1><?= $item->title_edit ?></h1> <div><?= $item->description_edit ?></div> - <? comment::show_comments($item->id); ?> + <?= comment::block($theme, true); ?> </div> |