first go at brining tidypics up to code standards

1 files changed, 61 insertions, 60 deletions

diff --git a/pages/lists/mostviewedimages.php b/pages/lists/mostviewedimages.php
index d945680f1..eb87bc17c 100644
--- a/pages/lists/mostviewedimages.php
+++ b/pages/lists/mostviewedimages.php
@@ -1,33 +1,33 @@
 <?php
 
-	/**
-	 * Most viewed images - either for a user or all site
-	 * 
-	 */
+/**
+ * Most viewed images - either for a user or all site
+ *
+ */
 
-	// 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";
 
-	global $CONFIG;
-	$prefix = $CONFIG->dbprefix;
-	$max = 24;
-	
-	$owner_guid = page_owner();
-	
-	//$start = microtime(true);
-	$photos = tp_get_entities_from_annotations_calculate_x(	
-													'count',
-													'object', 
-													'image', 
-													'tp_view', 
-													'', 
-													'', 
-													$owner_guid, 
-													$max);
-	//error_log("elgg query is " . (float)(microtime(true) - $start));
-		
-	//this works but is wildly inefficient
-	//$annotations = get_annotations(0, "object", "image", "tp_view", "", "", 5000);
+global $CONFIG;
+$prefix = $CONFIG->dbprefix;
+$max = 24;
+
+$owner_guid = page_owner();
+
+//$start = microtime(true);
+$photos = tp_get_entities_from_annotations_calculate_x(	
+		'count',
+		'object',
+		'image',
+		'tp_view',
+		'',
+		'',
+		$owner_guid,
+		$max);
+//error_log("elgg query is " . (float)(microtime(true) - $start));
+
+//this works but is wildly inefficient
+//$annotations = get_annotations(0, "object", "image", "tp_view", "", "", 5000);
 /*
 	$start = microtime(true);
 	$sql = "SELECT ent.guid, count( * ) AS views
@@ -48,39 +48,40 @@
 		$entities[] = get_entity($entity->guid);
 	}
 */
-	//error_log("custom query is " . (float)(microtime(true) - $start));
-	
-	// 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' );
-	}
-													
-	if ($owner_guid) {
-		if ($owner_guid == get_loggedin_userid()) {
-			$title = elgg_echo("tidypics:yourmostviewed");	
-		} else {
-			$title = sprintf(elgg_echo("tidypics:friendmostviewed"), page_owner_entity()->name);
-		}
+//error_log("custom query is " . (float)(microtime(true) - $start));
+
+// 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' );
+}
+
+if ($owner_guid) {
+	if ($owner_guid == get_loggedin_userid()) {
+		$title = elgg_echo("tidypics:yourmostviewed");
 	} else {
-		// world view - set page owner to logged in user
-		if (isloggedin()) {
-			set_page_owner(get_loggedin_userid());
-		}
-		
-		$title = elgg_echo("tidypics:mostviewed");
+		$title = sprintf(elgg_echo("tidypics:friendmostviewed"), page_owner_entity()->name);
 	}
-	$area2 = elgg_view_title($title);
-	
-	// grab the html to display the images
-	$content = tp_view_entity_list($photos, $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' => $content,));
-	if( empty( $area2 )) $area2 = $content;	
-	
-	$body = elgg_view_layout('two_column_left_sidebar', '', $area2);
-	page_draw($title, $body);
-?>
\ No newline at end of file
+} else {
+	// world view - set page owner to logged in user
+	if (isloggedin()) {
+		set_page_owner(get_loggedin_userid());
+	}
+
+	$title = elgg_echo("tidypics:mostviewed");
+}
+$area2 = elgg_view_title($title);
+
+// grab the html to display the images
+$content = tp_view_entity_list($photos, $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' => $content,));
+if (empty($area2)) {
+	$area2 = $content;
+}
+
+$body = elgg_view_layout('two_column_left_sidebar', '', $area2);
+page_draw($title, $body);