Changeset 18788 for trunk/wp-includes/admin-bar.php

Timestamp:

09/27/2011 04:04:10 AM (13 years ago)

Author:

nacin

Message:

Cleanup in wp_admin_bar_new_content_menu. current() instead of array_shift(), remove unused index 1 from the arrays, remove ?post_type=post from the Post link. see #18197.

File:

• trunk/wp-includes/admin-bar.php (modified) (4 diffs)

trunk/wp-includes/admin-bar.php

@@ -430 +430 @@
  */
 function wp_admin_bar_new_content_menu( $wp_admin_bar ) {
-    $primary   = array();
-    $secondary = array();
+    $primary = $secondary = array();
 
     foreach ( (array) get_post_types( array( 'show_in_admin_bar' => true ), 'objects' ) as $ptype_obj ) {
@@ -437 +436 @@
             continue;
 
-        $primary[ 'post-new.php?post_type=' . $ptype_obj->name ] = array( $ptype_obj->labels->name_admin_bar, $ptype_obj->cap->edit_posts, 'new-' . $ptype_obj->name );
+        $key = 'post' == $ptype_obj->name ? 'post-new.php' : 'post-new.php?post_type=' . $ptype_obj->name;
+        $primary[ $key ] = array( $ptype_obj->labels->name_admin_bar, 'new-' . $ptype_obj->name );
     }
 
     if ( current_user_can( 'upload_files' ) )
-        $primary[ 'media-new.php' ] = array( _x( 'Media', 'add new from admin bar' ), 'upload_files', 'new-media' );
+        $primary[ 'media-new.php' ] = array( _x( 'Media', 'add new from admin bar' ), 'new-media' );
 
     if ( current_user_can( 'manage_links' ) )
-        $primary[ 'link-add.php' ] = array( _x( 'Link', 'add new from admin bar' ), 'manage_links', 'new-link' );
+        $primary[ 'link-add.php' ] = array( _x( 'Link', 'add new from admin bar' ), 'new-link' );
 
     if ( current_user_can( 'create_users' ) || current_user_can( 'promote_users' ) )
-        $secondary[ 'user-new.php' ] = array( _x( 'User', 'add new from admin bar' ), 'create_users', 'new-user' );
+        $secondary[ 'user-new.php' ] = array( _x( 'User', 'add new from admin bar' ), 'new-user' );
 
     if ( ! is_multisite() && current_user_can( 'install_themes' ) )
-        $secondary[ 'theme-install.php' ] = array( _x( 'Theme', 'add new from admin bar' ), 'install_themes', 'new-theme' );
+        $secondary[ 'theme-install.php' ] = array( _x( 'Theme', 'add new from admin bar' ), 'new-theme' );
 
     if ( ! is_multisite() && current_user_can( 'install_plugins' ) )
-        $secondary[ 'plugin-install.php' ] = array( _x( 'Plugin', 'add new from admin bar' ), 'install_plugins', 'new-plugin' );
-
-    if ( empty( $primary ) && empty( $secondary ) )
+        $secondary[ 'plugin-install.php' ] = array( _x( 'Plugin', 'add new from admin bar' ), 'new-plugin' );
+
+    if ( ! $primary && ! $secondary )
         return;
 
@@ -461 +461 @@
         'id'    => 'new-content',
         'title' => _x( 'Add New', 'admin bar menu group label' ),
-        'href'  => admin_url( array_shift( array_keys( $primary ) ) ),
+        'href'  => admin_url( current( array_keys( $primary ) ) ),
     ) );
 
@@ -485 +485 @@
             $wp_admin_bar->add_menu( array(
                 'parent' => $parent,
-                'id'     => $action[2],
+                'id'     => $action[1],
                 'title'  => $action[0],
                 'href'   => admin_url( $link )