From 73403645808a81425381f43c25e0ea2c92705312 Mon Sep 17 00:00:00 2001
From: Cash Costello <cash.costello@gmail.com>
Date: Sun, 25 Jul 2010 02:01:36 +0000
Subject: first go at brining tidypics up to code standards

---
 pages/lists/recentlycommented.php | 111 +++++++++++++++++++-------------------
 1 file changed, 54 insertions(+), 57 deletions(-)

(limited to 'pages/lists/recentlycommented.php')

diff --git a/pages/lists/recentlycommented.php b/pages/lists/recentlycommented.php
index 1d5f7aea7..f070e8563 100644
--- a/pages/lists/recentlycommented.php
+++ b/pages/lists/recentlycommented.php
@@ -1,36 +1,36 @@
 <?php
 
-	/**
-	 * Images recently commented on - world view only
-	 * 
-	 */
+/**
+ * Images recently commented on - world view only
+ *
+ */
 
-	// Load Elgg engine
-	include_once dirname(dirname(dirname(dirname(dirname(__FILE__))))) . "/engine/start.php";
+// Load Elgg engine
+include_once dirname(dirname(dirname(dirname(dirname(__FILE__))))) . "/engine/start.php";
 
-	// world view - set page owner to logged in user
-	if (isloggedin()) {
-		set_page_owner(get_loggedin_userid());
-	}
-	
-	// allow other plugins to override the slideshow
-	$slideshow_link = trigger_plugin_hook('tp_slideshow', 'album', array(), null);
-	if ($slideshow_link) {
-		add_submenu_item(elgg_echo('album:slideshow'),
-						$slideshow_link,
-						'photos' );
-	}
-	
-	
-	global $CONFIG;
-	$prefix = $CONFIG->dbprefix;
-	$max_limit = 200; //get extra because you'll have multiple views per image in the result set
-	$max = 16; //controls how many actually show on screen
-	
-	//this works but is wildly inefficient
-	//$annotations = get_annotations(0, "object", "image", "tp_view", "", "", 5000);
-	
-	$sql = "SELECT distinct (ent.guid), ann1.time_created
+// world view - set page owner to logged in user
+if (isloggedin()) {
+	set_page_owner(get_loggedin_userid());
+}
+
+// allow other plugins to override the slideshow
+$slideshow_link = trigger_plugin_hook('tp_slideshow', 'album', array(), null);
+if ($slideshow_link) {
+	add_submenu_item(elgg_echo('album:slideshow'),
+			$slideshow_link,
+			'photos' );
+}
+
+
+global $CONFIG;
+$prefix = $CONFIG->dbprefix;
+$max_limit = 200; //get extra because you'll have multiple views per image in the result set
+$max = 16; //controls how many actually show on screen
+
+//this works but is wildly inefficient
+//$annotations = get_annotations(0, "object", "image", "tp_view", "", "", 5000);
+
+$sql = "SELECT distinct (ent.guid), ann1.time_created
 			FROM " . $prefix . "entities ent
 			INNER JOIN " . $prefix . "entity_subtypes sub ON ent.subtype = sub.id
 			AND sub.subtype = 'image'
@@ -39,34 +39,31 @@
 			AND ms.string = 'generic_comment'			
 			ORDER BY ann1.time_created DESC
 			LIMIT $max_limit";
-	
-	$result = get_data($sql);
 
-	$entities = array();
-	foreach($result as $entity) {
-		if(!$entities[$entity->guid]) {
-			$entities[$entity->guid] = get_entity($entity->guid);	
-		}
-		if(count($entities) >= $max) break;
+$result = get_data($sql);
+
+$entities = array();
+foreach ($result as $entity) {
+	if (!$entities[$entity->guid]) {
+		$entities[$entity->guid] = get_entity($entity->guid);
 	}
-	
-	$user = get_loggedin_user();
-	if( $user->guid == 9 ) {
-		echo "<pre>";
-		var_dump( $sql);
-//		var_dump( $result );
-		echo "</pre>";
+	if (count($entities) >= $max) {
+		break;
 	}
-	$title = elgg_echo("tidypics:recentlycommented");
-	$area2 = elgg_view_title($title);
-	
-	// grab the html to display the images
-	$images = tp_view_entity_list($entities, $max, 0, $max, false);
-	
-	// this view takes care of the title on the main column and the content wrapper
-	$area2 = elgg_view('tidypics/content_wrapper', array('title' => $title, 'content' => $images,));
-	if( empty( $area2 )) $area2 = $images;
-		
-	$body = elgg_view_layout('two_column_left_sidebar', '', $area2);
-	page_draw($title, $body);
-?>
\ No newline at end of file
+}
+
+$user = get_loggedin_user();
+$title = elgg_echo("tidypics:recentlycommented");
+$area2 = elgg_view_title($title);
+
+// grab the html to display the images
+$images = tp_view_entity_list($entities, $max, 0, $max, false);
+
+// this view takes care of the title on the main column and the content wrapper
+$area2 = elgg_view('tidypics/content_wrapper', array('title' => $title, 'content' => $images,));
+if (empty($area2)) {
+	$area2 = $images;
+}
+
+$body = elgg_view_layout('two_column_left_sidebar', '', $area2);
+page_draw($title, $body);
-- 
cgit v1.2.3