Merge pull request #98 from edfactor/patch-1

Only admins should see Administration topbar menu item
ewinslow · Jun 10, 2013 · 076886f · 076886f
2 parents 00d072d + b49274c
commit 076886f
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/start.php b/start.php
@@ -335,13 +335,15 @@ function facebook_theme_pagesetup_handler() {
 		));
 
 		elgg_unregister_menu_item('topbar', 'administration');
-		elgg_register_menu_item('topbar', array(
-			'href' => '/admin',
-			'name' => 'administration',
-			'parent_name' => 'account',
-			'section' => 'alt',
-			'text' => elgg_echo('admin'),
-		));
+		if (elgg_is_admin_logged_in()) { 
+                    elgg_register_menu_item('topbar', array(
+                            'href' => '/admin',
+                            'name' => 'administration',
+                            'parent_name' => 'account',
+                            'section' => 'alt',
+                            'text' => elgg_echo('admin'),
+                    ));
+                }
 
 		if (elgg_is_active_plugin('notifications')) {
 			elgg_register_menu_item('topbar', array(
@@ -677,4 +679,4 @@ function facebook_theme_profile_page_handler($page) {
 	return true;
 }
 
-elgg_register_event_handler('init', 'system', 'facebook_theme_init');
+elgg_register_event_handler('init', 'system', 'facebook_theme_init');