Cleaned up the log browser plugin

git-svn-id: http://code.elgg.org/elgg/trunk@7467 36083f99-b078-4883-b0ff-0f9b5a30f544

4 files changed, 114 insertions, 24 deletions

diff --git a/mod/logbrowser/views/default/logbrowser/adminlinks.php b/mod/logbrowser/views/default/logbrowser/adminlinks.php
index c52419c89..03e42fbb8 100644
--- a/mod/logbrowser/views/default/logbrowser/adminlinks.php
+++ b/mod/logbrowser/views/default/logbrowser/adminlinks.php
@@ -1,4 +1,10 @@
 <?php
-
+/**
+ * The dropdown link for exploring a user's log
+ *
+ * @package ElggLogBrowser
+ */
 ?>
-<a href="<?php echo elgg_get_site_url(); ?>mod/logbrowser/?user_guid=<?php echo $vars['entity']->guid; ?>"><?php echo elgg_echo("logbrowser:explore"); ?></a>
\ No newline at end of file
+<a href="<?php echo elgg_get_site_url(); ?>pg/admin/overview/logbrowser/?user_guid=<?php echo $vars['entity']->guid; ?>">
+	<?php echo elgg_echo("logbrowser:explore"); ?>
+</a>
\ No newline at end of file
diff --git a/mod/logbrowser/views/default/logbrowser/css.php b/mod/logbrowser/views/default/logbrowser/css.php
index 1612bff99..5e2f4e168 100644
--- a/mod/logbrowser/views/default/logbrowser/css.php
+++ b/mod/logbrowser/views/default/logbrowser/css.php
@@ -1,10 +1,9 @@
 <?php
-
-	/**
-	 * Elgg logbrowser CSS
-	 * 
-	 * @package logbrowser
-	 */
+/**
+ * Elgg logbrowser CSS
+ * 
+ * @package ElggLogBrowser
+ */
 
 ?>
 #logbrowser_search_area {
@@ -22,27 +21,30 @@
 	padding-bottom:7px;
 }
 .log_entry {
-	width: 710px;
-	font-size: 80%;
-	background:white;
-	margin:0 10px 5px 10px;
-	border:1px solid white;
+	margin: 5px;
 }
-.log_entry:hover {
-	background-color: #eeeeee;
-	border:1px solid #666666;
+.log_entry td, th {
+	padding: 4px 8px;
+	background: white;
+	border: 1px solid #cccccc;
 }
-.log_entry td {
+.log_entry th {
+	background-color: #dddddd;
 }
-.log_entry_user {
-	width: 120px;
+.log_entry .alt td {
+	background: #eeeeee;
 }
 .log_entry_time {
 	width: 210px;
-	padding:2px;
 }
-.log_entry_item {
-	
+.log_entry_user {
+	width: 120px;
+}
+.log_entry_guid {
+	width: 80px;
+}
+.log_entry_object {
+	width: 120px;
 }
 .log_entry_action {
 	width: 75px;
diff --git a/mod/logbrowser/views/default/logbrowser/form.php b/mod/logbrowser/views/default/logbrowser/form.php
index c87b67703..7f52d04c9 100644
--- a/mod/logbrowser/views/default/logbrowser/form.php
+++ b/mod/logbrowser/views/default/logbrowser/form.php
@@ -1,3 +1,10 @@
+<?php
+/**
+ * Log browser search form
+ *
+ * @package ElggLogBrowser
+ */
+?>
 
 <div id="logbrowser_search_area">
 <?php
@@ -46,12 +53,15 @@
 		$wrappedform = elgg_view('input/form', array(
 			'body' => $form,
 			'method' => 'get',
-			'action' => "pg/admin/logbrowser",
+			'action' => "pg/admin/overview/logbrowser/",
+			'disable_security' => true,
 		));
 ?>
 
 	<div id="log_browser_search_form" class="hidden radius8"><?php echo $wrappedform; ?></div>
 	<p>
-		<a class="link" onclick="elgg_slide_toggle(this,'#logbrowser_search_area','#log_browser_search_form');"><?php echo elgg_echo('logbrowser:search'); ?></a>
+		<a class="link" onclick="elgg_slide_toggle(this,'#logbrowser_search_area','#log_browser_search_form');">
+			<?php echo elgg_echo('logbrowser:search'); ?>
+		</a>
 	</p>
 </div>
\ No newline at end of file
diff --git a/mod/logbrowser/views/default/logbrowser/table.php b/mod/logbrowser/views/default/logbrowser/table.php
new file mode 100644
index 000000000..77de5dcaa
--- /dev/null
+++ b/mod/logbrowser/views/default/logbrowser/table.php
@@ -0,0 +1,72 @@
+<?php
+/**
+ * Log browser table
+ *
+ * @package ElggLogBrowser
+ */
+
+$log_entries = $vars['log_entries'];
+?>
+
+<table class="log_entry">
+	<tr>
+		<th><?php echo elgg_echo('logbrowser:date'); ?></th>
+		<th><?php echo elgg_echo('logbrowser:user:name'); ?></th>
+		<th><?php echo elgg_echo('logbrowser:user:guid'); ?></th>
+		<th><?php echo elgg_echo('logbrowser:object'); ?></th>
+		<th><?php echo elgg_echo('logbrowser:object:guid'); ?></th>
+		<th><?php echo elgg_echo('logbrowser:action'); ?></th>
+	</tr>
+<?php
+	$alt = '';
+	foreach ($log_entries as $entry) {
+		$user = get_entity($entry->performed_by_guid);
+		if ($user) {
+			$user_link = elgg_view('output/url', array(
+				'href' => $user->getURL(),
+				'text' => $user->name
+			));
+			$user_guid_link = elgg_view('output/url', array(
+				'href' => elgg_get_site_url() . "pg/admin/overview/logbrowser/?user_guid=$user->guid",
+				'text' => $user->getGUID()
+			));
+		} else {
+			$user_guid_link = $user_link = '&nbsp;';
+		}
+
+		$object = get_object_from_log_entry($entry->id);
+		if (is_callable(array($object, 'getURL'))) {
+			$object_link = elgg_view('output/url', array(
+				'href' => $object->getURL(),
+				'text' => $entry->object_class
+			));
+		} else {
+			$object_link = $entry->object_class;
+		}
+?>
+	<tr <?php echo $alt; ?>>
+		<td class="log_entry_time">
+			<?php echo date('r', $entry->time_created); ?>
+		</td>
+		<td class="log_entry_user">
+			<?php echo $user_link; ?>
+		</td>
+		<td class="log_entry_guid">
+			<?php echo $user_guid_link; ?>
+		</td>
+		<td class="log_entry_object">
+			<?php echo $object_link; ?>
+		</td>
+		<td class="log_entry_guid">
+			<?php echo $entry->object_id; ?>
+		</td>
+		<td class="log_entry_action">
+			<?php echo elgg_echo($entry->event); ?>
+		</td>
+	</tr>
+<?php
+
+		$alt = $alt ? '' : 'class="alt"';
+	}
+?>
+</table>
\ No newline at end of file