diff --git a/ext/alias_editor/main.php b/ext/alias_editor/main.php index a459093e..ed1a09c6 100644 --- a/ext/alias_editor/main.php +++ b/ext/alias_editor/main.php @@ -15,34 +15,37 @@ class AliasEditor extends Extension { public function receive_event($event) { if(is_a($event, 'PageRequestEvent') && ($event->page == "alias")) { global $user; - if($user->is_admin()) { - if($event->get_arg(0) == "add") { + if($event->get_arg(0) == "add") { + if($user->is_admin()) { if(isset($_POST['oldtag']) && isset($_POST['newtag'])) { send_event(new AddAliasEvent($_POST['oldtag'], $_POST['newtag'])); } } - else if($event->get_arg(0) == "remove") { + } + else if($event->get_arg(0) == "remove") { + if($user->is_admin()) { if(isset($_POST['oldtag'])) { global $database; $database->Execute("DELETE FROM aliases WHERE oldtag=?", array($_POST['oldtag'])); - + global $page; $page->set_mode("redirect"); $page->set_redirect(make_link("admin")); } } - else if($event->get_arg(0) == "list") { - global $page; - $page->set_title("Alias List"); - $page->set_heading("Alias List"); - $page->add_main_block(new Block("Aliases", $this->build_aliases())); - } - else if($event->get_arg(0) == "export") { - global $page; - $page->set_mode("data"); - $page->set_type("text/plain"); - $page->set_data($this->get_alias_csv()); - } + } + else if($event->get_arg(0) == "list") { + global $page; + $page->set_title("Alias List"); + $page->set_heading("Alias List"); + $page->add_side_block(new NavBlock()); + $page->add_main_block(new Block("Aliases", $this->build_aliases())); + } + else if($event->get_arg(0) == "export") { + global $page; + $page->set_mode("data"); + $page->set_type("text/plain"); + $page->set_data($this->get_alias_csv()); } } @@ -65,28 +68,11 @@ class AliasEditor extends Extension { // admin page HTML {{{ private function build_aliases() { global $database; - $h_aliases = ""; - $aliases = $database->db->GetAssoc("SELECT oldtag, newtag FROM aliases"); - foreach($aliases as $old => $new) { - $h_old = html_escape($old); - $h_new = html_escape($new); - $h_aliases .= " -
From | To | Action | - $h_aliases + + global $user; + if($user->is_admin()) { + $action = "Action | "; + $add = "
$h_old | $h_new | "; + if($user->is_admin()) { + $h_aliases .= " ++ + | + "; + } + $h_aliases .= "
From | To | $action + $h_aliases + $add
Export +
"; return $html; }