[eventlog] Simplify permission check

1 files changed, 21 insertions, 30 deletions

diff --git a/modules-available/eventlog/page.inc.php b/modules-available/eventlog/page.inc.php
index 320c3b07..c482eb93 100644
--- a/modules-available/eventlog/page.inc.php
+++ b/modules-available/eventlog/page.inc.php
@@ -6,43 +6,34 @@ class Page_EventLog extends Page
 	protected function doPreprocess()
 	{
 		User::load();
-		if (!User::isLoggedIn()) {
-			Message::addError('main.no-permission');
-			Util::redirect('?do=Main');
-		}
-		if (User::hasPermission("view")) {
-			User::setLastSeenEvent(Property::getLastWarningId());
-		}
+		User::assertPermission('view');
+		User::setLastSeenEvent(Property::getLastWarningId());
 	}
 
 	protected function doRender()
 	{
 		Render::addTemplate("heading");
-		if (User::hasPermission("view")) {
-			$today = date('d.m.Y');
-			$yesterday = date('d.m.Y', time() - 86400);
-			$lines = array();
-			$paginate = new Paginate("SELECT logid, dateline, logtypeid, description, extra FROM eventlog ORDER BY logid DESC", 50);
-			$res = $paginate->exec();
-			while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
-				$day = date('d.m.Y', $row['dateline']);
-				if ($day === $today) {
-					$day = Dictionary::translate('lang_today');
-				} elseif ($day === $yesterday) {
-					$day = Dictionary::translate('lang_yesterday');
-				}
-				$row['date'] = $day . date(' H:i', $row['dateline']);
-				$row['icon'] = $this->typeToIcon($row['logtypeid']);
-				$row['color'] = $this->typeToColor($row['logtypeid']);
-				$lines[] = $row;
+		$today = date('d.m.Y');
+		$yesterday = date('d.m.Y', time() - 86400);
+		$lines = array();
+		$paginate = new Paginate("SELECT logid, dateline, logtypeid, description, extra FROM eventlog ORDER BY logid DESC", 50);
+		$res = $paginate->exec();
+		while ($row = $res->fetch(PDO::FETCH_ASSOC)) {
+			$day = date('d.m.Y', $row['dateline']);
+			if ($day === $today) {
+				$day = Dictionary::translate('lang_today');
+			} elseif ($day === $yesterday) {
+				$day = Dictionary::translate('lang_yesterday');
 			}
-
-			$paginate->render('_page', array(
-				'list' => $lines
-			));
-		} else {
-			Message::addError('main.no-permission');
+			$row['date'] = $day . date(' H:i', $row['dateline']);
+			$row['icon'] = $this->typeToIcon($row['logtypeid']);
+			$row['color'] = $this->typeToColor($row['logtypeid']);
+			$lines[] = $row;
 		}
+
+		$paginate->render('_page', array(
+			'list' => $lines
+		));
 	}
 
 	private function typeToIcon($type)