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

diff --git a/engine/lib/admin.php b/engine/lib/admin.php
index 1e4b63cdd83fbfd17fb792b2d3a5925cff174923..be01a150cad7298b471b41f6878e08c91f05bca2 100644 (file)
--- 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 7c086754853637daba601604d381d29b9e0b37e2..cd8b657f317f159a91830d6565d36bf380523825 100644 (file)
--- 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');

diff --git a/mod/developers/start.php b/mod/developers/start.php
index dd74493f4fe193275faa3e1b80606a870cc05e8c..5dc4cb2b80f0fb9fdfc1e1b0949450fa7070307a 100644 (file)
--- a/mod/developers/start.php
+++ b/mod/developers/start.php
@@ -18,9 +18,9 @@ function developers_init() {
 
 function developers_setup_menu() {
        if (elgg_in_context('admin')) {
-               elgg_add_admin_menu_item('developers', elgg_echo('admin:developers'));
-               elgg_add_admin_menu_item('settings', elgg_echo('admin:developers:settings'), 'developers');
-               elgg_add_admin_menu_item('preview', elgg_echo('admin:developers:preview'), 'developers');
+               elgg_add_admin_menu_item('developers');
+               elgg_add_admin_menu_item('settings', 'developers');
+               elgg_add_admin_menu_item('preview', 'developers');
        }
 }
 

diff --git a/mod/diagnostics/start.php b/mod/diagnostics/start.php
index a79306b2509b47aab33fb95a2c898fe1169c7cac..21ee6207d8bd774123601bfb67594fdfde3ce474 100644 (file)
--- a/mod/diagnostics/start.php
+++ b/mod/diagnostics/start.php
@@ -16,7 +16,7 @@ function diagnostics_init() {
        elgg_register_page_handler('diagnostics','diagnostics_page_handler');
 
        // Add admin menu item
-       elgg_add_admin_menu_item('diagnostics', elgg_echo('diagnostics'), 'utilities');
+       elgg_add_admin_menu_item('diagnostics', 'utilities');
 
        // Register some actions
        $file = elgg_get_plugins_path() . "diagnostics/actions/download.php";

diff --git a/mod/externalpages/start.php b/mod/externalpages/start.php
index 41df8750d43294942eb7bded09ebb80dad6ba067..16f46326f83e82fa163be3143e3b5dae36448067 100644 (file)
--- a/mod/externalpages/start.php
+++ b/mod/externalpages/start.php
@@ -11,7 +11,7 @@ function expages_init() {
        elgg_register_page_handler('expages', 'expages_page_handler');
 
        // add a menu item for the admin edit page
-       elgg_add_admin_menu_item('expages', elgg_echo('expages'), 'site');
+       elgg_add_admin_menu_item('expages', 'site');
 
        // add footer links
        expages_setup_footer_menu();

diff --git a/mod/logbrowser/start.php b/mod/logbrowser/start.php
index dace1a68e47b21939533bb03863d3df720f62cf6..f0f4070db5e3fa6665c6224f2432385d7f5f0e98 100644 (file)
--- a/mod/logbrowser/start.php
+++ b/mod/logbrowser/start.php
@@ -14,7 +14,7 @@ function logbrowser_init() {
        
        elgg_register_plugin_hook_handler('register', 'menu:user_hover', 'logbrowser_user_hover_menu');
        
-       elgg_add_admin_menu_item('logbrowser', elgg_echo('logbrowser'), 'utilities');
+       elgg_add_admin_menu_item('logbrowser', 'utilities');
 }
 
 /**

diff --git a/mod/reportedcontent/start.php b/mod/reportedcontent/start.php
index f89b01f2b87ae679d1bd251b5703162c89eaf3dc..fe717395806b6ae2bc014b492edbf5f3377d2d55 100644 (file)
--- a/mod/reportedcontent/start.php
+++ b/mod/reportedcontent/start.php
@@ -38,7 +38,7 @@ function reportedcontent_init() {
        elgg_register_plugin_hook_handler('register', 'menu:user_hover', 'reportedcontent_user_hover_menu');
 
        // Add admin menu item
-       elgg_add_admin_menu_item('reportedcontent', elgg_echo('reportedcontent'), 'utilities');
+       elgg_add_admin_menu_item('reportedcontent', 'utilities');
 
        elgg_register_widget_type(
                        'reportedcontent',

diff --git a/mod/uservalidationbyemail/start.php b/mod/uservalidationbyemail/start.php
index 03f20451109922471a5e82ec89207da64e53ae6c..ce9a9cd2cda27fc9e864921c531d115141b05b7e 100644 (file)
--- a/mod/uservalidationbyemail/start.php
+++ b/mod/uservalidationbyemail/start.php
@@ -39,7 +39,7 @@ function uservalidationbyemail_init() {
        elgg_register_plugin_hook_handler('public_pages', 'walled_garden', 'uservalidationbyemail_public_pages');
 
        // admin interface to manually validate users
-       elgg_add_admin_menu_item('unvalidated', elgg_echo('uservalidationbyemail:admin:unvalidated'), 'users');
+       elgg_add_admin_menu_item('unvalidated', 'users');
 
        elgg_extend_view('js/elgg', 'uservalidationbyemail/js');