simplified admin menu registration function in preparation for adding sections

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

2 files changed, 35 insertions, 28 deletions

diff --git a/engine/lib/admin.php b/engine/lib/admin.php
index 1e4b63cdd..be01a150c 100644
--- a/engine/lib/admin.php
+++ b/engine/lib/admin.php
@@ -113,32 +113,33 @@ function elgg_admin_notice_exists($id) {
  * Used in conjuction with http://elgg.org/admin/section_id/child_section style
  * page handler.
  *
- * @param string $section_id    The Unique ID of section
- * @param string $section_title Human readable section title.
- * @param string $parent_id     If a child section, the parent section id.
- * @param int    $weight        The menu item weight
+ * The text of the menu item is obtained from elgg_echo(admin:$parent_id:$menu_id)
+ *
+ * @param string $menu_id    The Unique ID of section
+ * @param string $parent_id  If a child section, the parent section id.
+ * @param int    $weight     The menu item weight
  *
  * @return bool
  * @since 1.8.0
  */
-function elgg_add_admin_menu_item($section_id, $section_title, $parent_id = NULL, $weight = 100) {
+function elgg_add_admin_menu_item($menu_id, $parent_id = NULL, $weight = 100) {
 
 	// in the admin section parents never have links
 	if ($parent_id) {
-		$href = "pg/admin/$parent_id/$section_id";
+		$href = "pg/admin/$parent_id/$menu_id";
 	} else {
 		$href = NULL;
 	}
 
-	$name = $section_id;
+	$name = $menu_id;
 	if ($parent_id) {
-		$name = "$name:$parent_id";
+		$name = "$parent_id:$name";
 	}
 
 	return elgg_register_menu_item('page', array(
 		'name' => $name,
 		'href' => $href,
-		'text' => $section_title,
+		'text' => elgg_echo("admin:$name"),
 		'context' => 'admin',
 		'parent_name' => $parent_id,
 		'weight' => $weight,
@@ -174,32 +175,32 @@ function admin_init() {
 	elgg_register_simplecache_view('js/admin');
 
 	// statistics
-	elgg_add_admin_menu_item('statistics', elgg_echo('admin:statistics'), null, 60);
-	elgg_add_admin_menu_item('overview', elgg_echo('admin:statistics:overview'), 'statistics');
+	elgg_add_admin_menu_item('statistics', null, 60);
+	elgg_add_admin_menu_item('overview', 'statistics');
 
 	// site
-	elgg_add_admin_menu_item('site', elgg_echo('admin:site'), null, 20);
-	elgg_add_admin_menu_item('basic', elgg_echo('admin:site:basic'), 'site', 10);
-	elgg_add_admin_menu_item('advanced', elgg_echo('admin:site:advanced'), 'site', 20);
+	elgg_add_admin_menu_item('site', null, 20);
+	elgg_add_admin_menu_item('basic', 'site', 10);
+	elgg_add_admin_menu_item('advanced', 'site', 20);
 
 	// appearance
-	elgg_add_admin_menu_item('appearance', elgg_echo('admin:appearance'), null, 30);
-	elgg_add_admin_menu_item('menu_items', elgg_echo('admin:appearance:menu_items'), 'appearance', 10);
-	elgg_add_admin_menu_item('profile_fields', elgg_echo('admin:appearance:profile_fields'), 'appearance', 20);
+	elgg_add_admin_menu_item('appearance', null, 30);
+	elgg_add_admin_menu_item('menu_items', 'appearance', 10);
+	elgg_add_admin_menu_item('profile_fields', 'appearance', 20);
 
 	// users
-	elgg_add_admin_menu_item('users', elgg_echo('admin:users'), null, 40);
-	elgg_add_admin_menu_item('add', elgg_echo('admin:users:add'), 'users', 10);
-	elgg_add_admin_menu_item('online', elgg_echo('admin:users:online'), 'users', 20);
-	elgg_add_admin_menu_item('newest', elgg_echo('admin:users:newest'), 'users', 30);
+	elgg_add_admin_menu_item('users', null, 40);
+	elgg_add_admin_menu_item('add', 'users', 10);
+	elgg_add_admin_menu_item('online', 'users', 20);
+	elgg_add_admin_menu_item('newest', 'users', 30);
 
 	// plugins
-	elgg_add_admin_menu_item('plugins', elgg_echo('admin:plugins'), null, 50);
-	elgg_add_admin_menu_item('simple', elgg_echo('admin:plugins:simple'), 'plugins', 10);
-	elgg_add_admin_menu_item('advanced', elgg_echo('admin:plugins:advanced'), 'plugins', 20);
+	elgg_add_admin_menu_item('plugins', null, 50);
+	elgg_add_admin_menu_item('simple', 'plugins', 10);
+	elgg_add_admin_menu_item('advanced', 'plugins', 20);
 
 	// utilities
-	elgg_add_admin_menu_item('utilities', elgg_echo('admin:utilities'), null, 70);
+	elgg_add_admin_menu_item('utilities', null, 70);
 
 	// dashboard
 	elgg_register_menu_item('page', array(
@@ -250,12 +251,18 @@ function elgg_admin_add_plugin_settings_menu() {
 		return FALSE;
 	}
 
-	elgg_add_admin_menu_item('plugin_settings', elgg_echo('admin:plugin_settings'), null, 51);
+	elgg_add_admin_menu_item('plugin_settings', null, 51);
 
 	foreach ($active_plugins as $plugin) {
 		$plugin_id = $plugin->getID();
 		if (elgg_view_exists("settings/$plugin_id/edit")) {
-			elgg_add_admin_menu_item($plugin_id, $plugin->manifest->getName(), 'plugin_settings');
+			elgg_register_menu_item('page', array(
+				'name' => $plugin_id,
+				'href' => "pg/admin/plugin_settings/$plugin_id",
+				'text' => $plugin->manifest->getName(),
+				'parent_name' => 'plugin_settings',
+				'context' => 'admin',
+		 ));
 		}
 	}
 }
diff --git a/engine/lib/widgets.php b/engine/lib/widgets.php
index 7c0867548..cd8b657f3 100644
--- a/engine/lib/widgets.php
+++ b/engine/lib/widgets.php
@@ -288,7 +288,7 @@ function elgg_default_widgets_init() {
 	$CONFIG->default_widget_info = $default_widgets;
 
 	if ($default_widgets) {
-		elgg_add_admin_menu_item('default_widgets', elgg_echo('admin:appearance:default_widgets'), 'appearance', 30);
+		elgg_add_admin_menu_item('default_widgets', 'appearance', 30);
 
 		// override permissions for creating widget on logged out / just created entities
 		elgg_register_plugin_hook_handler('container_permissions_check', 'object', 'elgg_default_widgets_permissions_override');