From 31a654b61284e0f848645aeb436d0335bc765abf Mon Sep 17 00:00:00 2001
From: Cash Costello <cash.costello@gmail.com>
Date: Tue, 25 Aug 2009 12:03:29 +0000
Subject: cleaned up tagging views to better integrate with image class

---
 views/default/object/image.php        | 45 ++++++++++++-----------------------
 views/default/tidypics/image_menu.php |  2 +-
 views/default/tidypics/tagging.php    | 30 +++++++++++++----------
 3 files changed, 34 insertions(+), 43 deletions(-)

(limited to 'views/default')

diff --git a/views/default/object/image.php b/views/default/object/image.php
index dd225f02f..3b7917223 100644
--- a/views/default/object/image.php
+++ b/views/default/object/image.php
@@ -8,27 +8,15 @@
 	include_once dirname(dirname(dirname(dirname(__FILE__)))) . "/lib/exif.php";
 
 	$image = $vars['entity'];
-	$file_guid = $image->getGUID();
+	$image_guid = $image->getGUID();
 	$tags = $image->tags;
 	$title = $image->title;
 	$desc = $image->description;
 	$owner = $image->getOwnerEntity();
 	$friendlytime = friendly_time($image->time_created);
-
 	$mime = $image->mimetype;
 
 
-/////////////////////////////////////////////////////
-// get photo tags from database
-	$photo_tags_json = "\"\"";
-	
-	$tag_info = $image->getPhotoTags();
-	if ($tag_info) {
-		$photo_tags = $tag_info['raw'];
-		$photo_tags_json = $tag_info['json'];
-		$photo_tag_links = $tag_info['links'];
-	}
-
 /********************************************************************
  *
  * search view of an image
@@ -39,7 +27,7 @@
 		if (get_input('search_viewtype') == "gallery") {
 			?>
 			<div class="tidypics_album_images">
-				<a href="<?php echo $image->getURL();?>"><img src="<?php echo $vars['url'];?>mod/tidypics/thumbnail.php?file_guid=<?php echo $file_guid;?>&size=small" alt="thumbnail"/></a>
+				<a href="<?php echo $image->getURL();?>"><img src="<?php echo $vars['url'];?>mod/tidypics/thumbnail.php?file_guid=<?php echo $image_guid;?>&size=small" alt="thumbnail"/></a>
 			</div>
 			<?php
 		}
@@ -51,7 +39,7 @@
 			if ($numcomments)
 				$info .= ", <a href=\"{$image->getURL()}\">" . sprintf(elgg_echo("comments")) . " (" . $numcomments . ")</a>";
 			$info .= "</p>";
-			$icon = "<a href=\"{$image->getURL()}\">" . '<img src="' . $vars['url'] . 'mod/tidypics/thumbnail.php?file_guid=' . $file_guid . '&size=thumb" alt="' . $title . '" /></a>';
+			$icon = "<a href=\"{$image->getURL()}\">" . '<img src="' . $vars['url'] . 'mod/tidypics/thumbnail.php?file_guid=' . $image_guid . '&size=thumb" alt="' . $title . '" /></a>';
 
 			echo elgg_view_listing($icon, $info);
 		}
@@ -62,7 +50,7 @@
  ****************************************************************/
 	} else if (get_context() == "front") {
 ?>
-		<a href="<?php echo $image->getURL();?>"><img src="<?php echo $vars['url'];?>mod/tidypics/thumbnail.php?file_guid=<?php echo $file_guid;?>&amp;size=thumb" class="tidypics_album_cover" alt="<?php echo $title; ?>" title="<?php echo $title; ?>" /></a>
+		<a href="<?php echo $image->getURL();?>"><img src="<?php echo $vars['url'];?>mod/tidypics/thumbnail.php?file_guid=<?php echo $image_guid;?>&amp;size=thumb" class="tidypics_album_cover" alt="<?php echo $title; ?>" title="<?php echo $title; ?>" /></a>
 <?php
 	} else {
 
@@ -74,7 +62,7 @@
 		if (!$vars['full']) {
 ?>
 	<div class="tidypics_album_images">
-		<a href="<?php echo $image->getURL();?>"><img src="<?php echo $vars['url'];?>mod/tidypics/thumbnail.php?file_guid=<?php echo $file_guid;?>&size=small" alt="thumbnail"/></a>
+		<a href="<?php echo $image->getURL();?>"><img src="<?php echo $vars['url'];?>mod/tidypics/thumbnail.php?file_guid=<?php echo $image_guid;?>&size=small" alt="thumbnail"/></a>
 	</div>
 <?php
 		} else {
@@ -101,8 +89,8 @@
 				
 				// only non-owner views count
 				if ($owner->guid != $view->owner_guid)
-					create_annotation($file_guid, "tp_view", "1", "integer", $the_viewer, ACCESS_PUBLIC);
-				$views_a = get_annotations($file_guid, "object", "image", "tp_view", "", 0, 9999);
+					create_annotation($image_guid, "tp_view", "1", "integer", $the_viewer, ACCESS_PUBLIC);
+				$views_a = get_annotations($image_guid, "object", "image", "tp_view", "", 0, 9999);
 				$views = count($views_a);
 			
 				$my_views = 0;
@@ -125,7 +113,7 @@
 
 			$album = get_entity($image->container_guid);
 
-			$current = array_search($file_guid, $_SESSION['image_sort']);
+			$current = array_search($image_guid, $_SESSION['image_sort']);
 
 			if (!$current) {  // means we are no longer using the correct album array
 
@@ -138,7 +126,7 @@
 				}
 
 				if ($_SESSION['image_sort'])
-					$current = array_search($file_guid, $_SESSION['image_sort']);
+					$current = array_search($image_guid, $_SESSION['image_sort']);
 			}
 
 			if ($current != 0)
@@ -177,9 +165,9 @@
 		<div id="tidypics_image_wrapper">
 			<?php
 				if (get_plugin_setting('download_link', 'tidypics') != "disabled") {  
-					echo "<a href=\"{$vars['url']}action/tidypics/download?file_guid={$file_guid}&amp;view=inline\" title=\"{$title}\"><img id=\"tidypics_image\"  src=\"{$vars['url']}mod/tidypics/thumbnail.php?file_guid={$file_guid}&amp;size=large\" alt=\"{$title}\" /></a>";
+					echo "<a href=\"{$vars['url']}action/tidypics/download?file_guid={$image_guid}&amp;view=inline\" title=\"{$title}\"><img id=\"tidypics_image\"  src=\"{$vars['url']}mod/tidypics/thumbnail.php?file_guid={$image_guid}&amp;size=large\" alt=\"{$title}\" /></a>";
 				} else {
-					echo "<img id=\"tidypics_image\"  src=\"{$vars['url']}mod/tidypics/thumbnail.php?file_guid={$file_guid}&amp;size=large\" alt=\"{$title}\" />";
+					echo "<img id=\"tidypics_image\"  src=\"{$vars['url']}mod/tidypics/thumbnail.php?file_guid={$image_guid}&amp;size=large\" alt=\"{$title}\" />";
 				}
 			?>
 			<div class="clearfloat"></div>
@@ -188,19 +176,16 @@
 			// image menu (start tagging, download, etc.)
 			
 			echo '<div id="tidypics_controls"><ul>';
-			echo elgg_view('tidypics/image_menu', array('file_guid' => $file_guid, 
+			echo elgg_view('tidypics/image_menu', array('image_guid' => $image_guid, 
 														'viewer' => $viewer,
 														'owner' => $owner,
-														'anytags' => $photo_tags != '',
+														'anytags' => $image->isPhotoTagged(),
 														'album' => $album, ) );
 			echo '</ul></div>'; 
 			
 			// tagging code
 			if (get_plugin_setting('tagging', 'tidypics') != "disabled") {
-				echo elgg_view('tidypics/tagging', array(	'photo_tags' => $photo_tags, 
-															'links' => $photo_tag_links,
-															'photo_tags_json' => $photo_tags_json,
-															'file_guid' => $file_guid,
+				echo elgg_view('tidypics/tagging', array(	'image' => $image, 
 															'viewer' => $viewer,
 															'owner' => $owner, ) );
 			}
@@ -208,7 +193,7 @@
 			
 			if (get_plugin_setting('exif', 'tidypics') == "enabled") {
 ?>
-				<?php echo elgg_view('tidypics/exif', array('guid'=> $file_guid)); ?>
+				<?php echo elgg_view('tidypics/exif', array('guid'=> $image_guid)); ?>
 <?php		} ?>
 		<div class="tidypics_info">
 <?php if (!is_null($tags)) { ?>
diff --git a/views/default/tidypics/image_menu.php b/views/default/tidypics/image_menu.php
index d6568360c..afbb34f21 100644
--- a/views/default/tidypics/image_menu.php
+++ b/views/default/tidypics/image_menu.php
@@ -10,7 +10,7 @@
 	 *
 	 **************************************************************************/
 
-	$image_guid = $vars['file_guid'];
+	$image_guid = $vars['image_guid'];
 	$viewer = $vars['viewer'];
 	$owner = $vars['owner'];
 	$anytags = $vars['anytags'];
diff --git a/views/default/tidypics/tagging.php b/views/default/tidypics/tagging.php
index f911fe217..7062d942a 100644
--- a/views/default/tidypics/tagging.php
+++ b/views/default/tidypics/tagging.php
@@ -1,20 +1,26 @@
 <?php
-	$photo_tags = $vars['photo_tags'];
-	$links = $vars['links'];
-	$photo_tags_json = $vars['photo_tags_json'];
-	$file_guid = $vars['file_guid'];
+
+	$image = $vars['image'];
 	$viewer = $vars['viewer'];
 	$owner = $vars['owner'];
 
+	// get photo tags
+	$tag_info = $image->getPhotoTags();
+
+	// defining json text as "" makes sure the tagging javascript code doesn't throw errors if no tags
+	$photo_tags_json = "\"\"";
+	if ($tag_info) {
+		$photo_tags_json = $tag_info['json'];
+	}
 	
-	if ($photo_tags) { 
+	if ($tag_info) { 
 ?>
 <div id="tidypics_phototags_list">
 	<h3><?php echo elgg_echo('tidypics:inthisphoto') ?></h3>
 	<ul>
 <?php
-		foreach ($links as $id=>$link) {
-			echo "<li><a class='tidypics_phototag_links' id='taglink{$id}' href='{$link[1]}'>{$link[0]}</a></li>";
+		foreach ($tag_info['links'] as $id=>$link) {
+			echo "<li><a class='tidypics_phototag_links' id='taglink{$id}' href='{$link['url']}'>{$link['text']}</a></li>";
 		}
 ?>
 	</ul>
@@ -41,7 +47,7 @@
 		ksort($friend_array);
 
 		$content = "<div id='tidypics_tagmenu_left'>";
-		$content .= "<input type='hidden' name='image_guid' value='{$file_guid}' />";
+		$content .= "<input type='hidden' name='image_guid' value='{$image->guid}' />";
 		$content .= "<input type='hidden' name='coordinates' id='coordinates' value='' />";
 		$content .= "<input type='hidden' name='user_id' id='user_id' value='' />";
 		$content .= "<input type='hidden' name='word' id='word' value='' />";
@@ -66,11 +72,11 @@
 <div id="tidypics_delete_tag_menu" class="tidypics_popup">
 <div id='tidypics_popup_header'><h3><?php echo elgg_echo('tidypics:deltag_title'); ?></h3></div>
 <?php
-	if ($photo_tags) {
-		$content = "<input type='hidden' name='image_guid' value='{$file_guid}' />";
-		foreach ($links as $id => $text) {
+	if ($tag_info) {
+		$content = "<input type='hidden' name='image_guid' value='{$image->guid}' />";
+		foreach ($tag_info['links'] as $id => $link) {
 			$name = "tags[{$id}]";
-			$content .= elgg_view("input/checkboxes", array('options' => array($text[0] => $text[0]), 'internalname' => $name, 'value' => '' ));
+			$content .= elgg_view("input/checkboxes", array('options' => array($link['text'] => $link['text']), 'internalname' => $name, 'value' => '' ));
 		}
 
 		$content .= "<input type='submit' value='" . elgg_echo('tidypics:actiondelete') . "' class='submit_button' />";
-- 
cgit v1.2.3