diff --git a/contrib/log_db/main.php b/contrib/log_db/main.php
index 6049b278..86dc9ad8 100644
--- a/contrib/log_db/main.php
+++ b/contrib/log_db/main.php
@@ -43,14 +43,17 @@ class LogDatabase extends SimpleExtension {
global $database, $user;
if($event->page_matches("log/view")) {
if($user->is_admin()) {
- $events = $database->get_all("SELECT * FROM score_log");
+ $events = $database->get_all("SELECT * FROM score_log ORDER BY id DESC LIMIT 50");
$this->theme->display_events($events);
}
}
}
public function onUserBlockBuilding($event) {
- $event->add_link("Event Log", make_link("log/view"));
+ global $user;
+ if($user->is_admin()) {
+ $event->add_link("Event Log", make_link("log/view"));
+ }
}
public function onLog($event) {
diff --git a/contrib/log_db/theme.php b/contrib/log_db/theme.php
index 4104fcd8..580818a3 100644
--- a/contrib/log_db/theme.php
+++ b/contrib/log_db/theme.php
@@ -3,17 +3,23 @@
class LogDatabaseTheme extends Themelet {
public function display_events($events) {
$table = "
";
- $table .= "Time | Module / Priority | Username / Address | Message | ";
- $table .= "";
+ $table .= "Time | Module | User | Message | ";
+ $table .= "\n";
$n = 0;
foreach($events as $event) {
$oe = ($n++ % 2 == 0) ? "even" : "odd";
- $table .= "";
- $table .= "".$event['date_sent']." | ";
- $table .= "".$event['section']." / ".$event['priority']." | ";
- $table .= "".html_escape($event['username'])." / ".$event['address']." | ";
+ $c = $this->pri_to_col($event['priority']);
+ $table .= "
";
+ $table .= "".str_replace(" ", " ", $event['date_sent'])." | ";
+ $table .= "".$event['section']." | ";
+ if($event['username'] == "Anonymous") {
+ $table .= "".$event['address']." | ";
+ }
+ else {
+ $table .= "".html_escape($event['username'])." | ";
+ }
$table .= "".html_escape($event['message'])." | ";
- $table .= "
";
+ $table .= "\n";
}
$table .= "
";
@@ -23,5 +29,16 @@ class LogDatabaseTheme extends Themelet {
$page->add_block(new NavBlock());
$page->add_block(new Block("Events", $table));
}
+
+ protected function pri_to_col($pri) {
+ switch($pri) {
+ case SCORE_LOG_DEBUG: return "#999";
+ case SCORE_LOG_INFO: return "#000";
+ case SCORE_LOG_WARNING: return "#800";
+ case SCORE_LOG_ERROR: return "#C00";
+ case SCORE_LOG_CRITICAL: return "#F00";
+ default: return "";
+ }
+ }
}
?>
diff --git a/core/util.inc.php b/core/util.inc.php
index 6c7d1d20..8f714782 100644
--- a/core/util.inc.php
+++ b/core/util.inc.php
@@ -477,19 +477,11 @@ function log_msg($section, $priority, $message) {
send_event(new LogEvent($section, $priority, $message));
}
-/**
- * A shorthand way to send a LogEvent
- */
-function log_info($section, $message) {
- log_msg($section, SCORE_LOG_INFO, $message);
-}
-
-/**
- * A shorthand way to send a LogEvent
- */
-function log_error($section, $message) {
- log_msg($section, SCORE_LOG_ERROR, $message);
-}
+function log_debug($section, $message) {log_msg($section, SCORE_LOG_DEBUG, $message);}
+function log_info($section, $message) {log_msg($section, SCORE_LOG_INFO, $message);}
+function log_warning($section, $message) {log_msg($section, SCORE_LOG_WARNING, $message);}
+function log_error($section, $message) {log_msg($section, SCORE_LOG_ERROR, $message);}
+function log_critical($section, $message) {log_msg($section, SCORE_LOG_CRITICAL, $message);}
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *\
diff --git a/ext/handle_404/main.php b/ext/handle_404/main.php
index 4830ff3d..88b48163 100644
--- a/ext/handle_404/main.php
+++ b/ext/handle_404/main.php
@@ -15,7 +15,7 @@ class Handle404 implements Extension {
if($page->mode == "page" && (!isset($page->blocks) || $this->count_main($page->blocks) == 0)) {
$h_pagename = html_escape(implode('/', $event->args));
header("HTTP/1.0 404 Page Not Found");
- log_info("handle_404", "Hit 404: $h_pagename");
+ log_debug("handle_404", "Hit 404: $h_pagename");
$page->set_title("404");
$page->set_heading("404 - No Handler Found");
$page->add_block(new NavBlock());
diff --git a/ext/setup/main.php b/ext/setup/main.php
index d96d36f5..57970646 100644
--- a/ext/setup/main.php
+++ b/ext/setup/main.php
@@ -287,7 +287,7 @@ class Setup extends SimpleExtension {
}
}
}
- log_info("setup", "Configuration updated");
+ log_warning("setup", "Configuration updated");
}
public function onUserBlockBuilding($event) {
diff --git a/ext/user/main.php b/ext/user/main.php
index e7fd3b6a..8338c198 100644
--- a/ext/user/main.php
+++ b/ext/user/main.php
@@ -242,7 +242,12 @@ class UserPage extends SimpleExtension {
if(!is_null($duser)) {
$user = $duser;
$this->set_login_cookie($name, $pass);
- log_info("user", "Logged in");
+ if($user->is_admin()) {
+ log_warning("user", "Admin logged in");
+ }
+ else {
+ log_info("user", "User logged in");
+ }
$page->set_mode("redirect");
$page->set_redirect(make_link("user"));
}