Changeset 21375 for trunk/wp-includes/capabilities.php

Timestamp:

07/31/2012 05:39:14 PM (14 years ago)

Author:

nacin

Message:

Don't call get_userdata() every time for the current user in map_meta_cap()'s read_post, edit_post, and delete_post branches. see #21120.

File:

• trunk/wp-includes/capabilities.php (modified) (6 diffs)

trunk/wp-includes/capabilities.php

@@ -961 +961 @@
         case 'delete_post':
         case 'delete_page':
-                $author_data = get_userdata( $user_id );
                 $post = get_post( $args[0] );
 
@@ -982 +981 @@
                 } else {
                         // No author set yet, so default to current user for cap checks.
-                        $post_author_data = $author_data;
+                        $post_author_data = get_userdata( $user_id );
                 }
 
@@ -1011 +1010 @@
         case 'edit_post':
         case 'edit_page':
-                $author_data = get_userdata( $user_id );
                 $post = get_post( $args[0] );
 
@@ -1032 +1030 @@
                 } else {
                         // No author set yet, so default to current user for cap checks.
-                        $post_author_data = $author_data;
+                        $post_author_data = get_userdata( $user_id );
                 }
 
@@ -1060 +1058 @@
         case 'read_post':
         case 'read_page':
-                $author_data = get_userdata( $user_id );
                 $post = get_post( $args[0] );
 
@@ -1087 +1084 @@
                 } else {
                         // No author set yet, so default to current user for cap checks.
-                        $post_author_data = $author_data;
+                        $post_author_data = get_userdata( $user_id );
                 }