From b9309e86c7e71ea1ee522e13438af83ef7017508 Mon Sep 17 00:00:00 2001 From: hukoeth Date: Fri, 3 Sep 2010 22:57:57 +0800 Subject: [PATCH 1/3] - Version 3 beta 1 - Added possibility to attach users to annotations - Moved the legend to resize_bottom - Hovering over the legend will highlight the corresponding annotation - Some UI improvements - User profile page shows the images associated with the user (not fully functional yet) --- .../controllers/admin_photoannotation.php | 8 + .../controllers/photoannotation.php | 259 ++++++++++-------- .../controllers/user_profile.php | 93 +++++++ .../photoannotation/css/photoannotation.css | 2 +- .../helpers/photoannotation_event.php | 10 + .../helpers/photoannotation_installer.php | 30 +- .../helpers/photoannotation_theme.php | 9 +- modules/photoannotation/js/jquery.annotate.js | 105 ++++--- modules/photoannotation/models/items_user.php | 21 ++ modules/photoannotation/module.info | 4 +- .../views/admin_photoannotation.html.php | 6 +- .../photoannotation_highlight_block.html.php | 69 ++++- .../photoannotation_user_profile.html.php | 47 ++++ .../views/user_profile_info.html.php | 9 + 14 files changed, 506 insertions(+), 166 deletions(-) create mode 100644 modules/photoannotation/controllers/user_profile.php create mode 100644 modules/photoannotation/models/items_user.php create mode 100644 modules/photoannotation/views/photoannotation_user_profile.html.php create mode 100644 modules/photoannotation/views/user_profile_info.html.php diff --git a/modules/photoannotation/controllers/admin_photoannotation.php b/modules/photoannotation/controllers/admin_photoannotation.php index 58d53444..e239e6b7 100644 --- a/modules/photoannotation/controllers/admin_photoannotation.php +++ b/modules/photoannotation/controllers/admin_photoannotation.php @@ -27,10 +27,14 @@ class Admin_Photoannotation_Controller extends Admin_Controller { $form = $this->_get_form(); if ($form->validate()) { + module::set_var( + "photoannotation", "showusers", $form->photoannotation->showusers->value, true); module::set_var( "photoannotation", "showfaces", $form->photoannotation->showfaces->value, true); module::set_var( "photoannotation", "shownotes", $form->photoannotation->shownotes->value, true); + module::set_var( + "photoannotation", "fullname", $form->photoannotation->fullname->value, true); message::success(t("Your settings have been saved.")); url::redirect("admin/photoannotation"); } @@ -47,10 +51,14 @@ class Admin_Photoannotation_Controller extends Admin_Controller { private function _get_form() { $form = new Forge("admin/photoannotation/handler", "", "post", array("id" => "g-admin-form")); $group = $form->group("photoannotation")->label(t("Photo annotation settings")); + $group->checkbox("showusers")->label(t("Show face annotation below photo.")) + ->checked(module::get_var("photoannotation", "showusers", false)); $group->checkbox("showfaces")->label(t("Show face annotation below photo.")) ->checked(module::get_var("photoannotation", "showfaces", false)); $group->checkbox("shownotes")->label(t("Show note annotations below photo.")) ->checked(module::get_var("photoannotation", "shownotes", false)); + $group->checkbox("fullname")->label(t("Show full name of a user instead of the username on annotations (username will be dispayed for users without a full name).")) + ->checked(module::get_var("photoannotation", "fullname", false)); $form->submit("submit")->value(t("Save")); return $form; } diff --git a/modules/photoannotation/controllers/photoannotation.php b/modules/photoannotation/controllers/photoannotation.php index be4b97ea..30f6ef65 100644 --- a/modules/photoannotation/controllers/photoannotation.php +++ b/modules/photoannotation/controllers/photoannotation.php @@ -18,23 +18,22 @@ * Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA. */ class photoannotation_Controller extends Controller { - public function save($item_data) { + public function save($item_id) { // Prevent Cross Site Request Forgery access::verify_csrf(); - //Get form data - $item = ORM::factory("item", $item_data); - $noteid = $_POST["noteid"]; + $item = ORM::factory("item", $item_id); + $annotate_id = $_POST["noteid"]; $notetype = $_POST["notetype"]; $str_y1 = $_POST["top"]; $str_x1 = $_POST["left"]; $str_y2 = $_POST["height"] + $str_y1; //Annotation uses area size, tagfaces uses positions $str_x2 = $_POST["width"] + $str_x1; //Annotation uses area size, tagfaces uses positions - $str_face_title = $_POST["text"]; + $item_title = $_POST["text"]; $tag_data = $_POST["tagsList"]; - $str_face_description = $_POST["desc"]; + $user_id = $_POST["userlist"]; + $description = $_POST["desc"]; $redir_uri = url::abs_site("{$item->type}s/{$item->id}"); - //Add tag to item, create tag if not exists if ($tag_data != "") { $tag = ORM::factory("tag")->where("name", "=", $tag_data)->find(); @@ -42,7 +41,6 @@ class photoannotation_Controller extends Controller { $tag->name = $tag_data; $tag->count = 0; } - $tag->add($item); $tag->count++; $tag->save(); @@ -50,102 +48,80 @@ class photoannotation_Controller extends Controller { } else { $tag_data = -1; } - - - // Decide if we are saving a face or a note. - - if ($noteid == "new") { - if ($tag_data == -1) { - if ($str_face_title == "") { - message::error(t("Please select a Tag or specify a Title.")); + //Save annotation + if ($annotate_id == "new") { //This is a new annotation + if ($user_id > -1) { //Save user + $this->_saveuser($user_id, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description); + } elseif ($tag_data > -1) { //Conversion user -> face + $this->_saveface($tag_data, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description); + } elseif ($item_title != "") { //Conversion user -> note + $this->_savenote($item_title, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description); + } else { //Somethings wrong + message::error(t("Please select a User or Tag or specify a Title.")); + url::redirect($redir_uri); + return; + } + } else { //This is an update to an existing annotation + switch ($notetype) { + case "user": //the original annotation is a user + $updateduser = ORM::factory("items_user") //load the existing user + ->where("id", "=", $annotate_id) + ->find(); + if ($user_id > -1) { //Conversion user -> user + $this->_saveuser($user_id, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description, $annotate_id); + } elseif ($tag_data > -1) { //Conversion user -> face + $this->_saveface($tag_data, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description); + $updateduser->delete(); //delete old user + } elseif ($item_title != "") { //Conversion user -> note + $this->_savenote($item_title, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description); + $updateduser->delete(); //delete old user + } else { //Somethings wrong + message::error(t("Please select a User or Tag or specify a Title.")); + url::redirect($redir_uri); + return; + } + break; + case "face": //the original annotation is a face + $updatedface = ORM::factory("items_face") //load the existing user + ->where("id", "=", $annotate_id) + ->find(); + if ($user_id > -1) { //Conversion face -> user + $this->_saveuser($user_id, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description); + $updatedface->delete(); //delete old face + } elseif ($tag_data > -1) { //Conversion face -> face + $this->_saveface($tag_data, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description, $annotate_id); + } elseif ($item_title != "") { //Conversion face -> note + $this->_savenote($item_title, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description); + $updatedface->delete(); //delete old face + } else { //Somethings wrong + message::error(t("Please select a User or Tag or specify a Title.")); + url::redirect($redir_uri); + return; + } + break; + case "note": //the original annotation is a note + $updatednote = ORM::factory("items_note") //load the existing user + ->where("id", "=", $annotate_id) + ->find(); + if ($user_id > -1) { //Conversion note -> user + $this->_saveuser($user_id, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description); + $updatednote->delete(); //delete old note + } elseif ($tag_data > -1) { //Conversion note -> face + $this->_saveface($tag_data, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description); + $updatednote->delete(); //delete old note + } elseif ($item_title != "") { //Conversion note -> note + $this->_savenote($item_title, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description, $annotate_id); + } else { //Somethings wrong + message::error(t("Please select a User or Tag or specify a Title.")); + url::redirect($redir_uri); + return; + } + break; + default: + message::error(t("Please select a User or Tag or specify a Title.")); url::redirect($redir_uri); return; - } - //Save note - $newnote = ORM::factory("items_note"); - $newnote->item_id = $item_data; - $newnote->x1 = $str_x1; - $newnote->y1 = $str_y1; - $newnote->x2 = $str_x2; - $newnote->y2 = $str_y2; - $newnote->title = $str_face_title; - $newnote->description = $str_face_description; - $newnote->save(); - } else { - // Save the new face to the database. - $newface = ORM::factory("items_face"); - $newface->tag_id = $tag_data; - $newface->item_id = $item_data; - $newface->x1 = $str_x1; - $newface->y1 = $str_y1; - $newface->x2 = $str_x2; - $newface->y2 = $str_y2; - $newface->description = $str_face_description; - $newface->save(); } - } else { //update existing annotation - if ($notetype == "face") { //this is a face - $updatedAnnotation = ORM::factory("items_face") - ->where("id", "=", $noteid) - ->find(); - if ($tag_data == -1) { //needs conversion to note - if ($str_face_title == "") { - message::error(t("Please select a Tag or specify a Title.")); - url::redirect($redir_uri); - return; - } - //Save note - $newnote = ORM::factory("items_note"); - $newnote->item_id = $item_data; - $newnote->x1 = $str_x1; - $newnote->y1 = $str_y1; - $newnote->x2 = $str_x2; - $newnote->y2 = $str_y2; - $newnote->title = $str_face_title; - $newnote->description = $str_face_description; - $newnote->save(); - $updatedAnnotation->delete(); - } else { //stays a face - $updatedAnnotation->tag_id = $tag_data; - $updatedAnnotation->item_id = $item_data; - $updatedAnnotation->x1 = $str_x1; - $updatedAnnotation->y1 = $str_y1; - $updatedAnnotation->x2 = $str_x2; - $updatedAnnotation->y2 = $str_y2; - $updatedAnnotation->description = $str_face_description; - $updatedAnnotation->save(); - } - } else { //this is a note - $updatedAnnotation = ORM::factory("items_note") - ->where("id", "=", $noteid) - ->find(); - if ($tag_data == -1) { //stays a note - if ($str_face_title == "") { - message::error(t("Please select a Tag or specify a Title.")); - url::redirect($redir_uri); - return; - } - $updatedAnnotation->item_id = $item_data; - $updatedAnnotation->x1 = $str_x1; - $updatedAnnotation->y1 = $str_y1; - $updatedAnnotation->x2 = $str_x2; - $updatedAnnotation->y2 = $str_y2; - $updatedAnnotation->title = $str_face_title; - $updatedAnnotation->description = $str_face_description; - $updatedAnnotation->save(); - } else { //needs conversion to a face - $newface = ORM::factory("items_face"); - $newface->tag_id = $tag_data; - $newface->item_id = $item_data; - $newface->x1 = $str_x1; - $newface->y1 = $str_y1; - $newface->x2 = $str_x2; - $newface->y2 = $str_y2; - $newface->description = $str_face_description; - $newface->save(); - $updatedAnnotation->delete(); - } - } } message::success(t("Annotation saved.")); url::redirect($redir_uri); @@ -162,20 +138,81 @@ class photoannotation_Controller extends Controller { $notetype = $_POST["notetype"]; $redir_uri = url::abs_site("{$item->type}s/{$item->id}"); - if ($noteid == "" || $notetype == "") { + if ($noteid == "") { message::error(t("Please select a tag or note to delete.")); url::redirect($redir_uri); return; } - if ($notetype == "face") { - db::build()->delete("items_faces")->where("id", "=", $noteid)->execute(); - message::success(t("Annotation deleted.")); - } elseif ($notetype == "note") { - db::build()->delete("items_notes")->where("id", "=", $noteid)->execute(); - message::success(t("Annotation deleted.")); - } else { - message::error(t("Please select a tag or note to delete.")); + switch ($notetype) { + case "user": + db::build()->delete("items_users")->where("id", "=", $noteid)->execute(); + break; + case "face": + db::build()->delete("items_faces")->where("id", "=", $noteid)->execute(); + break; + case "note": + db::build()->delete("items_notes")->where("id", "=", $noteid)->execute(); + break; + default: + message::error(t("Please select a tag or note to delete.")); + url::redirect($redir_uri); + return; } + message::success(t("Annotation deleted.")); url::redirect($redir_uri); } -} + + private function _saveuser($user_id, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description, $annotate_id = "") { + if ($annotate_id == "") { + $item_user = ORM::factory("items_user"); + } else { + $item_user = ORM::factory("items_user") + ->where("id", "=", $annotate_id) + ->find(); + } + $item_user->user_id = $user_id; + $item_user->item_id = $item_id; + $item_user->x1 = $str_x1; + $item_user->y1 = $str_y1; + $item_user->x2 = $str_x2; + $item_user->y2 = $str_y2; + $item_user->description = $description; + $item_user->save(); + } + + private function _saveface($tag_id, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description, $annotate_id = "") { + if ($annotate_id == "") { + $item_face = ORM::factory("items_face"); + } else { + $item_face = ORM::factory("items_face") + ->where("id", "=", $annotate_id) + ->find(); + } + $item_face->tag_id = $tag_id; + $item_face->item_id = $item_id; + $item_face->x1 = $str_x1; + $item_face->y1 = $str_y1; + $item_face->x2 = $str_x2; + $item_face->y2 = $str_y2; + $item_face->description = $description; + $item_face->save(); + } + + private function _savenote($item_title, $item_id, $str_x1, $str_y1, $str_x2, $str_y2, $description, $annotate_id = "") { + if ($annotate_id == "") { + $item_note = ORM::factory("items_note"); + } else { + $item_note = ORM::factory("items_note") + ->where("id", "=", $annotate_id) + ->find(); + } + $item_note->item_id = $item_id; + $item_note->x1 = $str_x1; + $item_note->y1 = $str_y1; + $item_note->x2 = $str_x2; + $item_note->y2 = $str_y2; + $item_note->title = $item_title; + $item_note->description = $description; + $item_note->save(); + } +} \ No newline at end of file diff --git a/modules/photoannotation/controllers/user_profile.php b/modules/photoannotation/controllers/user_profile.php new file mode 100644 index 00000000..20561c8e --- /dev/null +++ b/modules/photoannotation/controllers/user_profile.php @@ -0,0 +1,93 @@ +where("user_id", "=", $id)->find_all(); + $children_count = count($item_users); + foreach ($item_users as $item_user) { + $item_thumb = ORM::factory("item") + ->viewable() + ->where("type", "!=", "album") + ->where("id", ">=", $item_user->item_id) + ->find(); + $item_thumbs[] = $item_thumb; + } + $page_size = module::get_var("gallery", "page_size", 9); + $page = (int) Input::instance()->get("page", "1"); + $offset = ($page-1) * $page_size; + $max_pages = max(ceil($children_count / $page_size), 1); + + // Make sure that the page references a valid offset + if ($page < 1) { + url::redirect($album->abs_url()); + } else if ($page > $max_pages) { + url::redirect($album->abs_url("page=$max_pages")); + } + + $v->set_global("page", $page); + $v->set_global("max_pages", $max_pages); + $v->set_global("page_size", $page_size); + $v->set_global("userid", $id); + $v->set_global("children", array_slice($item_thumbs, $offset, $page_size)); + $v->set_global("children_count", $children_count); + $v->content = new View("photoannotation_user_profile.html"); + $v->content->user = $user; + $v->content->contactable = + !$user->guest && $user->id != identity::active_user()->id && $user->email; + $v->content->editable = + identity::is_writable() && !$user->guest && $user->id == identity::active_user()->id; + $event_data = (object)array("user" => $user, "content" => array()); + module::event("show_user_profile", $event_data); + $v->content->info_parts = $event_data->content; + $v->content = new View("dynamic.html"); + print $v; + } + + public function contact($id) { + $user = identity::lookup_user($id); + print user_profile::get_contact_form($user); + } + + public function send($id) { + access::verify_csrf(); + $user = identity::lookup_user($id); + $form = user_profile::get_contact_form($user); + if ($form->validate()) { + Sendmail::factory() + ->to($user->email) + ->subject(html::clean($form->message->subject->value)) + ->header("Mime-Version", "1.0") + ->header("Content-type", "text/html; charset=UTF-8") + ->reply_to($form->message->reply_to->value) + ->message(html::purify($form->message->message->value)) + ->send(); + message::success(t("Sent message to %user_name", array("user_name" => $user->display_name()))); + json::reply(array("result" => "success")); + } else { + json::reply(array("result" => "error", "html" => (string)$form)); + } + } +} diff --git a/modules/photoannotation/css/photoannotation.css b/modules/photoannotation/css/photoannotation.css index 49821e68..9ec14368 100644 --- a/modules/photoannotation/css/photoannotation.css +++ b/modules/photoannotation/css/photoannotation.css @@ -45,7 +45,7 @@ #image-annotate-edit-form { background: #FFFFFF none repeat scroll 0 0; border: 1px solid #000000; - height: 230px; + height: 300px; padding: 7px; position: absolute; width: 250px; diff --git a/modules/photoannotation/helpers/photoannotation_event.php b/modules/photoannotation/helpers/photoannotation_event.php index 4a973eb1..b7e1e15c 100644 --- a/modules/photoannotation/helpers/photoannotation_event.php +++ b/modules/photoannotation/helpers/photoannotation_event.php @@ -75,6 +75,16 @@ class photoannotation_event_Core { db::build()->delete("items_notes")->where("item_id", "=", $item->id)->execute(); } } + + static function user_deleted($old) { + // Check for and delete existing Annotations linked to that user. + $existingFaces = ORM::factory("items_user") + ->where("user_id", "=", $old->id) + ->find_all(); + if (count($existingFaces) > 0) { + db::build()->delete("items_users")->where("user_id", "=", $old->id)->execute(); + } + } static function admin_menu($menu, $theme) { $menu->get("settings_menu") diff --git a/modules/photoannotation/helpers/photoannotation_installer.php b/modules/photoannotation/helpers/photoannotation_installer.php index a00a74db..dd7a73c9 100644 --- a/modules/photoannotation/helpers/photoannotation_installer.php +++ b/modules/photoannotation/helpers/photoannotation_installer.php @@ -44,15 +44,42 @@ class photoannotation_installer { `description` varchar(2048) default NULL, PRIMARY KEY (`id`)) DEFAULT CHARSET=utf8;"); + + $db->query("CREATE TABLE IF NOT EXISTS {items_users} ( + `id` int(9) NOT NULL auto_increment, + `user_id` int(9) NOT NULL, + `item_id` int(9) NOT NULL, + `x1` int(9) NOT NULL, + `y1` int(9) NOT NULL, + `x2` int(9) NOT NULL, + `y2` int(9) NOT NULL, + `description` varchar(2048) default NULL, + PRIMARY KEY (`id`)) + DEFAULT CHARSET=utf8;"); // Set the module's version number. - module::set_version("photoannotation", 2); + module::set_version("photoannotation", 3); } static function upgrade($version) { if ($version == 1) { module::set_version("photoannotation", $version = 2); } + if ($version == 2) { + $db = Database::instance(); + $db->query("CREATE TABLE IF NOT EXISTS {items_users} ( + `id` int(9) NOT NULL auto_increment, + `user_id` int(9) NOT NULL, + `item_id` int(9) NOT NULL, + `x1` int(9) NOT NULL, + `y1` int(9) NOT NULL, + `x2` int(9) NOT NULL, + `y2` int(9) NOT NULL, + `description` varchar(2048) default NULL, + PRIMARY KEY (`id`)) + DEFAULT CHARSET=utf8;"); + module::set_version("photoannotation", $version = 3); + } } static function deactivate() { @@ -66,6 +93,7 @@ class photoannotation_installer { $db = Database::instance(); $db->query("DROP TABLE IF EXISTS {items_faces};"); $db->query("DROP TABLE IF EXISTS {items_notes};"); + $db->query("DROP TABLE IF EXISTS {items_users};"); module::delete("photoannotation"); } } diff --git a/modules/photoannotation/helpers/photoannotation_theme.php b/modules/photoannotation/helpers/photoannotation_theme.php index a2fbacd5..cbea80d1 100644 --- a/modules/photoannotation/helpers/photoannotation_theme.php +++ b/modules/photoannotation/helpers/photoannotation_theme.php @@ -25,9 +25,16 @@ class photoannotation_theme_Core { } } - static function photo_bottom($theme) { + static function resize_bottom($theme) { if ($theme->page_subtype == "photo") { return new View("photoannotation_highlight_block.html"); } } + + static function dynamic_top($theme) { + return new View("photoannotation_user_profile.html"); + if ($theme->page_type == "photoannotationuserprofile") { + return new View("photoannotation_user_profile.html"); + } + } } diff --git a/modules/photoannotation/js/jquery.annotate.js b/modules/photoannotation/js/jquery.annotate.js index 03d99ce9..49f3e580 100644 --- a/modules/photoannotation/js/jquery.annotate.js +++ b/modules/photoannotation/js/jquery.annotate.js @@ -25,6 +25,7 @@ this.csrf = opts.csrf; this.cssaclass = opts.cssaclass; this.rtlsupport = opts.rtlsupport; + this.users = opts.users // Add the canvas this.canvas = $('
'); @@ -60,14 +61,14 @@ if (this.useAjax) { $.fn.annotateImage.ajaxLoad(this); } else { - $.fn.annotateImage.load(this, this.labels, this.editable, this.csrf, this.deleteUrl, this.tags, this.saveUrl, this.cssaclass, this.rtlsupport); + $.fn.annotateImage.load(this, this.labels, this.editable, this.csrf, this.deleteUrl, this.tags, this.saveUrl, this.cssaclass, this.rtlsupport, this.users); } // Add the "Add a note" button if ($('#g-photoannotation-link').length != 0) { this.button = $('#g-photoannotation-link'); this.button.click(function() { - $.fn.annotateImage.add(image, opts.tags, opts.labels, opts.saveUrl, opts.csrf, opts.rtlsupport); + $.fn.annotateImage.add(image, opts.tags, opts.labels, opts.saveUrl, opts.csrf, opts.rtlsupport, opts.users); }); //this.canvas.after(this.button); } @@ -112,13 +113,13 @@ }); }; - $.fn.annotateImage.load = function(image, labels, editable, csrf, deleteUrl, tags, saveUrl, cssaclass, rtlsupport) { + $.fn.annotateImage.load = function(image, labels, editable, csrf, deleteUrl, tags, saveUrl, cssaclass, rtlsupport, users) { /// /// Loads the annotations from the notes property passed in on the /// options object. /// for (var i = 0; i < image.notes.length; i++) { - image.notes[image.notes[i]] = new $.fn.annotateView(image, image.notes[i], tags, labels, editable, csrf, deleteUrl, saveUrl, cssaclass, rtlsupport); + image.notes[image.notes[i]] = new $.fn.annotateView(image, image.notes[i], tags, labels, editable, csrf, deleteUrl, saveUrl, cssaclass, rtlsupport, users); } }; @@ -131,7 +132,7 @@ return now.getTime(); }; - $.fn.annotateImage.add = function(image, tags, labels, saveUrl, csrf, rtlsupport) { + $.fn.annotateImage.add = function(image, tags, labels, saveUrl, csrf, rtlsupport, users) { /// /// Adds a note to the image. /// @@ -139,18 +140,18 @@ image.mode = 'edit'; // Create/prepare the editable note elements - var editable = new $.fn.annotateEdit(image, null, tags, labels, saveUrl, csrf, rtlsupport); + var editable = new $.fn.annotateEdit(image, null, tags, labels, saveUrl, csrf, rtlsupport, users); - $.fn.annotateImage.createSaveButton(editable, image, null, rtlsupport); - $.fn.annotateImage.createCancelButton(editable, image, rtlsupport); + $.fn.annotateImage.createSaveButton(editable, image, null, rtlsupport, labels); + $.fn.annotateImage.createCancelButton(editable, image, rtlsupport, labels); } }; - $.fn.annotateImage.createSaveButton = function(editable, image, note, rtlsupport) { + $.fn.annotateImage.createSaveButton = function(editable, image, note, rtlsupport, labels) { /// /// Creates a Save button on the editable note. /// - var ok = $('OK'); + var ok = $('' + labels[8] + ''); ok.click(function() { var form = $('#image-annotate-edit-form form'); @@ -165,11 +166,11 @@ editable.form.append(ok); }; - $.fn.annotateImage.createCancelButton = function(editable, image, rtlsupport) { + $.fn.annotateImage.createCancelButton = function(editable, image, rtlsupport, labels) { /// /// Creates a Cancel button on the editable note. /// - var cancel = $('Cancel'); + var cancel = $('' + labels[9] + ''); cancel.click(function() { editable.destroy(); image.mode = 'view'; @@ -195,7 +196,7 @@ return '<input type="hidden" name="' + name + '" value="' + value + '" />
'; }; - $.fn.annotateEdit = function(image, note, tags, labels, saveUrl, csrf, rtlsupport) { + $.fn.annotateEdit = function(image, note, tags, labels, saveUrl, csrf, rtlsupport, users) { /// /// Defines an editable annotation area. /// @@ -231,16 +232,31 @@ // Add the note (which we'll load with the form afterwards) var selectedtag = ""; var notetitle = ""; + var username = ""; + var selecteduser = ""; if (this.note.notetype == "face") { selectedtag = this.note.text; + selecteduser = " selected=\"selected\""; + } else if (this.note.notetype == "user") { + username = this.note.text; } else { notetitle = this.note.text; + selecteduser = " selected=\"selected\""; } - var form = $('
' + labels[0] + '' + '' + labels[4] + '
' + labels[1] + '' + labels[2] + '
'); + var userdropdown = ''; + var form = $('
' + labels[10] + userdropdown + '
' + labels[4] + '
' + labels[0] + '' + '
' + labels[4] + '
' + labels[1] + '
' + labels[2] + '
'); this.form = form; - $('body').append(this.form); - $("#photoannotation-form").ready(function() { var url = tags; $("input#image-annotate-tag-text").autocomplete( @@ -251,23 +267,32 @@ } ); }); - - $("input#image-annotate-tag-text").keyup(function() { + $("input#image-annotate-tag-text").change(function() { if ($("input#image-annotate-tag-text").val() != "") { - $("textarea#image-annotate-text").html(""); - $("textarea#image-annotate-text").val(""); - $("textarea#image-annotate-text").text(""); + $("input#image-annotate-text").html(""); + $("input#image-annotate-text").val(""); + $("input#image-annotate-text").text(""); + $("select#photoannotation-user-list").val('-1'); } }); - - $("textarea#image-annotate-text").keyup(function() { - if ($("textarea#image-annotate-text").val() != "") { + $("input#image-annotate-text").keyup(function() { + if ($("input#image-annotate-text").val() != "") { $("input#image-annotate-tag-text").html(""); $("input#image-annotate-tag-text").val(""); $("input#image-annotate-tag-text").text(""); + $("select#photoannotation-user-list").val('-1'); + } + }); + $("select#photoannotation-user-list").keyup(function() { + if ($("select#photoannotation-user-list").val() != "-1") { + $("input#image-annotate-tag-text").html(""); + $("input#image-annotate-tag-text").val(""); + $("input#image-annotate-tag-text").text(""); + $("input#image-annotate-text").html(""); + $("input#image-annotate-text").val(""); + $("input#image-annotate-text").text(""); } }); - this.form.css('left', this.area.offset().left + 'px'); this.form.css('top', (parseInt(this.area.offset().top) + parseInt(this.area.height()) + 7) + 'px'); @@ -313,7 +338,7 @@ this.form.remove(); } - $.fn.annotateView = function(image, note, tags, labels, editable, csrf, deleteUrl, saveUrl, cssaclass, rtlsupport) { + $.fn.annotateView = function(image, note, tags, labels, editable, csrf, deleteUrl, saveUrl, cssaclass, rtlsupport, users) { /// /// Defines a annotation area. /// @@ -322,7 +347,7 @@ this.note = note; // Add the area - this.area = $('
'); + this.area = $('
'); image.canvas.children('.image-annotate-view').prepend(this.area); if (editable) { @@ -360,7 +385,7 @@ alink.unbind(); alink.attr ('href', '#'); alink.removeAttr ('rel'); - form.edit(tags, labels, saveUrl, csrf, rtlsupport); + form.edit(tags, labels, saveUrl, csrf, rtlsupport, users); }) this.delarea.hide(); this.editarea.hide(); @@ -394,7 +419,21 @@ annotation.editarea.hide(); } }); - + var legendspan = "#photoannotation-legend-" + this.note.notetype + "-" + this.note.noteid; + if ($(legendspan).length > 0) { + $(legendspan).hover(function() { + var legendsarea = "#photoannotation-area-" + note.notetype + "-" + note.noteid; + $(legendsarea).children('.image-annotate-view').show(); + $(".image-annotate-view").show(); + $(legendsarea).show(); + annotation.show(); + }, function() { + var legendsarea = "#photoannotation-area-" + note.notetype + "-" + note.noteid; + annotation.hide(); + $(legendsarea).children('.image-annotate-view').hide(); + $(".image-annotate-view").hide(); + }); + } if (editable) { this.delarea.hover(function() { annotation.delarea.show(); @@ -480,7 +519,7 @@ this.form.remove(); } - $.fn.annotateView.prototype.edit = function(tags, labels, saveUrl, csrf, rtlsupport) { + $.fn.annotateView.prototype.edit = function(tags, labels, saveUrl, csrf, rtlsupport, users) { /// /// Edits the annotation. /// @@ -489,9 +528,9 @@ var annotation = this; // Create/prepare the editable note elements - var editable = new $.fn.annotateEdit(this.image, this.note, tags, labels, saveUrl, csrf, rtlsupport); - $.fn.annotateImage.createSaveButton(editable, this.image, annotation, rtlsupport); - $.fn.annotateImage.createCancelButton(editable, this.image, rtlsupport); + var editable = new $.fn.annotateEdit(this.image, this.note, tags, labels, saveUrl, csrf, rtlsupport, users); + $.fn.annotateImage.createSaveButton(editable, this.image, annotation, rtlsupport, labels); + $.fn.annotateImage.createCancelButton(editable, this.image, rtlsupport, labels); } }; diff --git a/modules/photoannotation/models/items_user.php b/modules/photoannotation/models/items_user.php new file mode 100644 index 00000000..22d39a58 --- /dev/null +++ b/modules/photoannotation/models/items_user.php @@ -0,0 +1,21 @@ +

-

TagFaces module by rWatcher.
- This means that notes and faces that you create in either one will be shown and are editable by the other module as well.
- However since both modules do the same you cannot have both active at the same time.

+

TagFaces module by rWatcher.
+ This means that notes and faces that you create in either one will be shown and are editable by the other module as well. If you added users to an annotation area though they will only be displayed with the Photo Annotation module.
+ You cannot have both active at the same time.

If you decide to show annotations below the photo but they are displayed below the comments section (or any other data), please download and install the Module order module.") ?>

diff --git a/modules/photoannotation/views/photoannotation_highlight_block.html.php b/modules/photoannotation/views/photoannotation_highlight_block.html.php index 618153d0..e688d5d9 100644 --- a/modules/photoannotation/views/photoannotation_highlight_block.html.php +++ b/modules/photoannotation/views/photoannotation_highlight_block.html.php @@ -1,12 +1,19 @@ where("item_id", "=", $item->id) + ->find_all(); $existingFaces = ORM::factory("items_face") ->where("item_id", "=", $item->id) ->find_all(); $existingNotes = ORM::factory("items_note") ->where("item_id", "=", $item->id) ->find_all(); + $users = ORM::factory("user")->order_by("name", "ASC")->find_all(); + $fullname = module::get_var("photoannotation", "fullname", false); + $showusers = module::get_var("photoannotation", "showusers", false); + $showfaces = module::get_var("photoannotation", "showfaces", false); + $shownotes = module::get_var("photoannotation", "shownotes", false); if (locales::is_rtl()) { $rtl_support = "image-annotate-rtl"; } else { @@ -25,12 +32,37 @@ } // If it does, then insert some javascript and display an image map // to show where the faces are at. - if ((count($existingFaces) > 0) || (count($existingNotes) > 0)) { + if ((count($existingFaces) > 0) || (count($existingNotes) > 0) || (count($existingUsers) > 0)) { $jscode = "notes: [ "; + foreach ($existingUsers as $oneUser) { + $oneTag = ORM::factory("user", $oneUser->user_id); + if ($fullname && ($oneTag->full_name != "")) { + $user_text = $oneTag->full_name; + } else { + $user_text = $oneTag->name; + } + if ($showusers) { + $legend_users .= "id . "\">user_id) ."\">". html::clean($user_text) .", "; + } + $jscode .= "{ \"top\": ". $oneUser->y1 .",\n"; + $jscode .= "\"left\": ". $oneUser->x1 .",\n"; + $jscode .= "\"width\": ". ($oneUser->x2 - $oneUser->x1) .",\n"; + $jscode .= "\"height\": ". ($oneUser->y2 - $oneUser->y1) .",\n"; + $jscode .= "\"text\": \"". html::clean($user_text) ."\",\n"; + $jscode .= "\"description\": \"". html::clean($oneUser->description) ."\",\n"; + $jscode .= "\"noteid\": ". $oneUser->id .",\n"; + $jscode .= "\"notetype\": \"user\",\n"; + $jscode .= "\"editable\": true,\n"; + $jscode .= "\"url\": \"". user_profile::url($oneUser->user_id) ."\" },\n"; + } + if ($legend_users != "") { + $legend_users = trim($legend_users, ", "); + $legend_users = t("Users on this photo: ") . $legend_users; + } foreach ($existingFaces as $oneFace) { $oneTag = ORM::factory("tag", $oneFace->tag_id); - if (module::get_var("photoannotation", "showfaces", false)) { - $legend_faces .= "url() ."\">". html::clean($oneTag->name) .", "; + if ($showfaces) { + $legend_faces .= "id . "\">url() ."\">". html::clean($oneTag->name) .", "; } $jscode .= "{ \"top\": ". $oneFace->y1 .",\n"; $jscode .= "\"left\": ". $oneFace->x1 .",\n"; @@ -48,8 +80,8 @@ $legend_faces = t("Faces on this photo: ") . $legend_faces; } foreach ($existingNotes as $oneNote) { - if (module::get_var("photoannotation", "shownotes", false)) { - $legend_notes .= html::clean($oneNote->title) .", "; + if ($shownotes) { + $legend_notes .= "id . "\">". html::clean($oneNote->title) .", "; } $jscode .= "{ \"top\": ". $oneNote->y1 .",\n"; $jscode .= "\"left\": ". $oneNote->x1 .",\n"; @@ -69,15 +101,23 @@ $legend_notes = t("Notes on this photo: ") . $legend_notes; } } - $legend_display = $legend_faces; - if ($legend_display == "") { - $legend_display = $legend_notes; - } else { - if ($legend_notes != "") { - $legend_display = $legend_display ."
". $legend_notes; - } + $legend_display = ""; + if ($legend_users != "" || $legend_faces != "" || $legend_notes != "") { + $legend_display = $legend_users . "
" . $legend_faces . "
" . $legend_notes; + $legend_display = str_replace("

", "
", $legend_display); } - $labels_arraystring = "labels: [ '". t("Tag:") ."','". t("Note Title:") ."','". t("Description (optional):") ."','". t("Are you sure you want to delete this annotation?") ."','". t("or") ."','". t("Yes") ."','". t("No") ."','". t("Confirm deletion") ."' ],"; + $users_arraystring = "users: [ "; + foreach ($users as $user) { + if ($fullname && ($user->full_name != "")) { + $user_text = $user->full_name; + } else { + $user_text = $user->name; + } + $users_arraystring .= "{'name':'". html::clean($user_text) ."','id':'". $user->id ."'},"; + } + $users_arraystring = trim($users_arraystring, ","); + $users_arraystring .= " ],"; + $labels_arraystring = "labels: [ '". t("Tag:") ."','". t("Note Title:") ."','". t("Description (optional):") ."','". t("Are you sure you want to delete this annotation?") ."','". t("or") ."','". t("Yes") ."','". t("No") ."','". t("Confirm deletion") ."','". t("Save") ."','". t("Cancel") ."','". t("User") ."','". t("No user selected") ."' ],"; ?> +
+
+ + + + + id}") ?>"> + + + id}") ?>"> + + + id}") ?>"> + + + + + id}") ?>"> + + + +
+

+ " + alt="display_name()) ?>" + class="g-avatar g-left" width="40" height="40" /> + $user->display_name())) ?> +

+ +
+

title) ?>

+
+ view ?> +
+
+ +
diff --git a/modules/photoannotation/views/user_profile_info.html.php b/modules/photoannotation/views/user_profile_info.html.php new file mode 100644 index 00000000..e559abda --- /dev/null +++ b/modules/photoannotation/views/user_profile_info.html.php @@ -0,0 +1,9 @@ + + + $value): ?> + + + + + +
From bfba13f19d0055491c2898ce6df53c1c86d6401a Mon Sep 17 00:00:00 2001 From: mamouneyya Date: Fri, 3 Sep 2010 14:37:31 +0800 Subject: [PATCH 2/3] merge with latest Wind theme updates --- themes/browny_admin_wind/views/admin.html.php | 8 ++++--- themes/browny_wind/views/album.html.php | 2 ++ themes/browny_wind/views/page.html.php | 21 ++++++++++++------- 3 files changed, 20 insertions(+), 11 deletions(-) diff --git a/themes/browny_admin_wind/views/admin.html.php b/themes/browny_admin_wind/views/admin.html.php index f9ef18c7..c4b51f41 100644 --- a/themes/browny_admin_wind/views/admin.html.php +++ b/themes/browny_admin_wind/views/admin.html.php @@ -11,7 +11,7 @@ - " type="image/x-icon" /> + " type="image/x-icon" /> css("yui/reset-fonts-grids.css") ?> css("themeroller/ui.base.css") ?> @@ -77,9 +77,11 @@ admin_page_bottom() ?> diff --git a/themes/browny_wind/views/album.html.php b/themes/browny_wind/views/album.html.php index de196be0..1a56af67 100644 --- a/themes/browny_wind/views/album.html.php +++ b/themes/browny_wind/views/album.html.php @@ -16,7 +16,9 @@
  • thumb_top($child) ?> + has_thumb()): ?> thumb_img(array("class" => "g-thumbnail")) ?> + thumb_bottom($child) ?> context_menu($child, "#g-item-id-{$child->id} .g-thumbnail") ?> diff --git a/themes/browny_wind/views/page.html.php b/themes/browny_wind/views/page.html.php index 72df044a..45ad51a7 100644 --- a/themes/browny_wind/views/page.html.php +++ b/themes/browny_wind/views/page.html.php @@ -23,7 +23,7 @@ - " type="image/x-icon" /> + " type="image/x-icon" /> css("yui/reset-fonts-grids.css") ?> css("superfish/css/superfish.css") ?> css("themeroller/ui.base.css") ?> @@ -101,18 +101,23 @@ > - - url($parent->id == $theme->item()->parent_id ? "show={$theme->item()->id}" : null) ?>"> - title) ?> + + title, + module::get_var("gallery", "visible_title_length"))) ?>
    • -
  • ">item()->title) ?>
    • +
  • "> + item()->title, + module::get_var("gallery", "visible_title_length"))) ?> +
    • From d216ae83b7f97f9b47f1f816f125d10051ef3800 Mon Sep 17 00:00:00 2001 From: dmolavi Date: Sat, 4 Sep 2010 08:11:52 +0800 Subject: [PATCH 3/3] Updated module version to reflect added Exif update capability. Also update copyright year on one of my ecard files. --- modules/autorotate/module.info | 2 +- modules/ecard/helpers/ecard_block.php | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/autorotate/module.info b/modules/autorotate/module.info index 17c62229..42a108fb 100644 --- a/modules/autorotate/module.info +++ b/modules/autorotate/module.info @@ -1,3 +1,3 @@ name = "Autorotate" description = "Rotate an image automatically on upload based on EXIF data" -version = 1 \ No newline at end of file +version = 2 \ No newline at end of file diff --git a/modules/ecard/helpers/ecard_block.php b/modules/ecard/helpers/ecard_block.php index 23136d55..051c55c6 100644 --- a/modules/ecard/helpers/ecard_block.php +++ b/modules/ecard/helpers/ecard_block.php @@ -1,7 +1,7 @@