Changeset 27670 for trunk/src/wp-admin/includes/menu.php

Timestamp:

03/24/2014 02:34:14 AM (12 years ago)

Author:

DrewAPicture

Message:

Inline documentation for hooks in wp-admin/includes/menu.php.

Also fixes syntax for a few unrelated multi-line comments in the file.

Props kpdesign for some minor cleanup.
Fixes #27430.

File:

• trunk/src/wp-admin/includes/menu.php (modified) (8 diffs)

trunk/src/wp-admin/includes/menu.php

@@ -1 +1 @@
 <?php
-
 /**
  * Build Administration Menu.
@@ -8 +7 @@
  */
 
-if ( is_network_admin() )
-    do_action('_network_admin_menu');
-elseif ( is_user_admin() )
-    do_action('_user_admin_menu');
-else
-    do_action('_admin_menu');
+if ( is_network_admin() ) {
+
+    /**
+     * Fires before the administration menu loads in the Network Admin.
+     *
+     * The hook fires before menus and sub-menus are removed based on user privileges.
+     *
+     * @private
+     * @since 3.1.0
+     */
+    do_action( '_network_admin_menu' );
+} elseif ( is_user_admin() ) {
+
+    /**
+     * Fires before the administration menu loads in the User Admin.
+     *
+     * The hook fires before menus and sub-menus are removed based on user privileges.
+     *
+     * @private
+     * @since 3.1.0
+     */
+    do_action( '_user_admin_menu' );
+} else {
+
+    /**
+     * Fires before the administration menu loads in the admin.
+     *
+     * The hook fires before menus and sub-menus are removed based on user privileges.
+     *
+     * @private
+     * @since 2.2.0
+     */
+    do_action( '_admin_menu' );
+}
 
 // Create list of page plugin hook names.
@@ -62 +89 @@
 unset($sub_loop);
 
-// Loop over the top-level menu.
-// Menus for which the original parent is not accessible due to lack of privs will have the next
-// submenu in line be assigned as the new menu parent.
+/*
+ * Loop over the top-level menu.
+ * Menus for which the original parent is not accessible due to lack of privileges
+ * will have the next submenu in line be assigned as the new menu parent.
+ */
 foreach ( $menu as $id => $data ) {
     if ( empty($submenu[$data[2]]) )
@@ -90 +119 @@
 unset($id, $data, $subs, $first_sub, $old_parent, $new_parent);
 
-if ( is_network_admin() )
-    do_action('network_admin_menu', '');
-elseif ( is_user_admin() )
-    do_action('user_admin_menu', '');
-else
-    do_action('admin_menu', '');
-
-// Remove menus that have no accessible submenus and require privs that the user does not have.
-// Run re-parent loop again.
+if ( is_network_admin() ) {
+
+    /**
+     * Fires before the administration menu loads in the Network Admin.
+     *
+     * @since 3.1.0
+     *
+     * @param string $context Empty context.
+     */
+    do_action( 'network_admin_menu', '' );
+} elseif ( is_user_admin() ) {
+
+    /**
+     * Fires before the administration menu loads in the User Admin.
+     *
+     * @since 3.1.0
+     *
+     * @param string $context Empty context.
+     */
+    do_action( 'user_admin_menu', '' );
+} else {
+
+    /**
+     * Fires before the administration menu loads in the admin.
+     *
+     * @since 1.5.0
+     *
+     * @param string $context Empty context.
+     */
+    do_action( 'admin_menu', '' );
+}
+
+/*
+ * Remove menus that have no accessible submenus and require privileges
+ * that the user does not have. Run re-parent loop again.
+ */
 foreach ( $menu as $id => $data ) {
     if ( ! current_user_can($data[1]) )
         $_wp_menu_nopriv[$data[2]] = true;
 
-    // If there is only one submenu and it is has same destination as the parent,
-    // remove the submenu.
+    /*
+     * If there is only one submenu and it is has same destination as the parent,
+     * remove the submenu.
+     */
     if ( ! empty( $submenu[$data[2]] ) && 1 == count ( $submenu[$data[2]] ) ) {
         $subs = $submenu[$data[2]];
@@ -178 +236 @@
     }
 
+    /**
+     * Filter administration menus array with classes added for top-level items.
+     *
+     * @since 2.7.0
+     *
+     * @param array $menu Associative array of administration menu items.
+     */
     return apply_filters( 'add_menu_classes', $menu );
 }
@@ -183 +248 @@
 uksort($menu, "strnatcasecmp"); // make it all pretty
 
-if ( apply_filters('custom_menu_order', false) ) {
+/**
+ * Filter whether to enable custom ordering of the administration menu.
+ *
+ * See the 'menu_order' filter for reordering menu items.
+ *
+ * @since 2.8.0
+ *
+ * @param bool $custom Whether custom ordering is enabled. Default false.
+ */
+if ( apply_filters( 'custom_menu_order', false ) ) {
+    log_it( 'false is true' );
     $menu_order = array();
     foreach ( $menu as $menu_item ) {
@@ -190 +265 @@
     unset($menu_item);
     $default_menu_order = $menu_order;
-    $menu_order = apply_filters('menu_order', $menu_order);
+
+    /**
+     * Filter the order of administration menu items.
+     *
+     * A truthy value must first be passed to the 'custom_menu_order' filter
+     * for this filter to work. Use the following to enable custom menu ordering:
+     *
+     *     add_filter( 'custom_menu_order', '__return_true' );
+     *
+     * @since 2.8.0
+     *
+     * @param array $menu_order An ordered array of menu items.
+     */
+    $menu_order = apply_filters( 'menu_order', $menu_order );
     $menu_order = array_flip($menu_order);
     $default_menu_order = array_flip($default_menu_order);
@@ -223 +311 @@
 
 if ( !user_can_access_admin_page() ) {
-    do_action('admin_page_access_denied');
+
+    /**
+     * Fires when access to an admin page is denied.
+     *
+     * @since 2.5.0
+     */
+    do_action( 'admin_page_access_denied' );
+
     wp_die( __('You do not have sufficient permissions to access this page.') );
 }