WordPress 4.3
[autoinstalls/wordpress.git] / wp-includes / capabilities.php
index 4bdc28cc955c3844152745b7c7279445736c78c4..42133f6febe2732465c234941f4daa6d1ddb0c9e 100644 (file)
  * the name in value of the 'name' key. The capabilities are stored as an array
  * in the value of the 'capability' key.
  *
- * <code>
- * array (
- *             'rolename' => array (
- *                     'name' => 'rolename',
- *                     'capabilities' => array()
- *             )
- * )
- * </code>
+ *     array (
+ *             'rolename' => array (
+ *                     'name' => 'rolename',
+ *                     'capabilities' => array()
+ *             )
+ *     )
  *
  * @since 2.0.0
  * @package WordPress
@@ -34,7 +32,7 @@ class WP_Roles {
         * @access public
         * @var array
         */
-       var $roles;
+       public $roles;
 
        /**
         * List of the role objects.
@@ -43,7 +41,7 @@ class WP_Roles {
         * @access public
         * @var array
         */
-       var $role_objects = array();
+       public $role_objects = array();
 
        /**
         * List of role names.
@@ -52,7 +50,7 @@ class WP_Roles {
         * @access public
         * @var array
         */
-       var $role_names = array();
+       public $role_names = array();
 
        /**
         * Option name for storing role list.
@@ -61,7 +59,7 @@ class WP_Roles {
         * @access public
         * @var string
         */
-       var $role_key;
+       public $role_key;
 
        /**
         * Whether to use the database for retrieval and storage.
@@ -70,17 +68,34 @@ class WP_Roles {
         * @access public
         * @var bool
         */
-       var $use_db = true;
+       public $use_db = true;
 
        /**
         * Constructor
         *
         * @since 2.0.0
         */
-       function __construct() {
+       public function __construct() {
                $this->_init();
        }
 
+       /**
+        * Make private/protected methods readable for backwards compatibility.
+        *
+        * @since 4.0.0
+        * @access public
+        *
+        * @param callable $name      Method to call.
+        * @param array    $arguments Arguments to pass when calling.
+        * @return mixed|false Return value of the callback, false otherwise.
+        */
+       public function __call( $name, $arguments ) {
+               if ( '_init' === $name ) {
+                       return call_user_func_array( array( $this, $name ), $arguments );
+               }
+               return false;
+       }
+
        /**
         * Set up the object properties.
         *
@@ -90,10 +105,11 @@ class WP_Roles {
         *
         * @since 2.1.0
         * @access protected
-        * @uses $wpdb Used to get the database prefix.
+        *
+        * @global wpdb  $wpdb          WordPress database abstraction object.
         * @global array $wp_user_roles Used to set the 'roles' property value.
         */
-       function _init () {
+       protected function _init() {
                global $wpdb, $wp_user_roles;
                $this->role_key = $wpdb->get_blog_prefix() . 'user_roles';
                if ( ! empty( $wp_user_roles ) ) {
@@ -122,13 +138,15 @@ class WP_Roles {
         *
         * @since 3.5.0
         * @access public
+        *
+        * @global wpdb $wpdb
         */
-       function reinit() {
+       public function reinit() {
                // There is no need to reinit if using the wp_user_roles global.
                if ( ! $this->use_db )
                        return;
 
-               global $wpdb, $wp_user_roles;
+               global $wpdb;
 
                // Duplicated from _init() to avoid an extra function call.
                $this->role_key = $wpdb->get_blog_prefix() . 'user_roles';
@@ -158,9 +176,9 @@ class WP_Roles {
         * @param string $role Role name.
         * @param string $display_name Role display name.
         * @param array $capabilities List of role capabilities in the above format.
-        * @return WP_Role|null WP_Role object if role is added, null if already exists.
+        * @return WP_Role|void WP_Role object, if role is added.
         */
-       function add_role( $role, $display_name, $capabilities = array() ) {
+       public function add_role( $role, $display_name, $capabilities = array() ) {
                if ( isset( $this->roles[$role] ) )
                        return;
 
@@ -183,7 +201,7 @@ class WP_Roles {
         *
         * @param string $role Role name.
         */
-       function remove_role( $role ) {
+       public function remove_role( $role ) {
                if ( ! isset( $this->role_objects[$role] ) )
                        return;
 
@@ -208,7 +226,7 @@ class WP_Roles {
         * @param string $cap Capability name.
         * @param bool $grant Optional, default is true. Whether role is capable of performing capability.
         */
-       function add_cap( $role, $cap, $grant = true ) {
+       public function add_cap( $role, $cap, $grant = true ) {
                if ( ! isset( $this->roles[$role] ) )
                        return;
 
@@ -226,7 +244,7 @@ class WP_Roles {
         * @param string $role Role name.
         * @param string $cap Capability name.
         */
-       function remove_cap( $role, $cap ) {
+       public function remove_cap( $role, $cap ) {
                if ( ! isset( $this->roles[$role] ) )
                        return;
 
@@ -244,7 +262,7 @@ class WP_Roles {
         * @param string $role Role name.
         * @return WP_Role|null WP_Role object if found, null if the role does not exist.
         */
-       function get_role( $role ) {
+       public function get_role( $role ) {
                if ( isset( $this->role_objects[$role] ) )
                        return $this->role_objects[$role];
                else
@@ -259,7 +277,7 @@ class WP_Roles {
         *
         * @return array List of role names.
         */
-       function get_names() {
+       public function get_names() {
                return $this->role_names;
        }
 
@@ -272,7 +290,7 @@ class WP_Roles {
         * @param string $role Role name to look up.
         * @return bool
         */
-       function is_role( $role ) {
+       public function is_role( $role ) {
                return isset( $this->role_names[$role] );
        }
 }
@@ -292,7 +310,7 @@ class WP_Role {
         * @access public
         * @var string
         */
-       var $name;
+       public $name;
 
        /**
         * List of capabilities the role contains.
@@ -301,7 +319,7 @@ class WP_Role {
         * @access public
         * @var array
         */
-       var $capabilities;
+       public $capabilities;
 
        /**
         * Constructor - Set up object properties.
@@ -315,7 +333,7 @@ class WP_Role {
         * @param string $role Role name.
         * @param array $capabilities List of capabilities.
         */
-       function __construct( $role, $capabilities ) {
+       public function __construct( $role, $capabilities ) {
                $this->name = $role;
                $this->capabilities = $capabilities;
        }
@@ -323,21 +341,15 @@ class WP_Role {
        /**
         * Assign role a capability.
         *
-        * @see WP_Roles::add_cap() Method uses implementation for role.
         * @since 2.0.0
         * @access public
         *
         * @param string $cap Capability name.
         * @param bool $grant Whether role has capability privilege.
         */
-       function add_cap( $cap, $grant = true ) {
-               global $wp_roles;
-
-               if ( ! isset( $wp_roles ) )
-                       $wp_roles = new WP_Roles();
-
+       public function add_cap( $cap, $grant = true ) {
                $this->capabilities[$cap] = $grant;
-               $wp_roles->add_cap( $this->name, $cap, $grant );
+               wp_roles()->add_cap( $this->name, $cap, $grant );
        }
 
        /**
@@ -353,14 +365,9 @@ class WP_Role {
         *
         * @param string $cap Capability name.
         */
-       function remove_cap( $cap ) {
-               global $wp_roles;
-
-               if ( ! isset( $wp_roles ) )
-                       $wp_roles = new WP_Roles();
-
+       public function remove_cap( $cap ) {
                unset( $this->capabilities[$cap] );
-               $wp_roles->remove_cap( $this->name, $cap );
+               wp_roles()->remove_cap( $this->name, $cap );
        }
 
        /**
@@ -377,7 +384,7 @@ class WP_Role {
         * @param string $cap Capability name.
         * @return bool True, if user has capability. False, if doesn't have capability.
         */
-       function has_cap( $cap ) {
+       public function has_cap( $cap ) {
                /**
                 * Filter which capabilities a role has.
                 *
@@ -402,16 +409,31 @@ class WP_Role {
  * @since 2.0.0
  * @package WordPress
  * @subpackage User
+ *
+ * @property string $nickname
+ * @property string $user_description
+ * @property string $user_firstname
+ * @property string $user_lastname
+ * @property string $user_login
+ * @property string $user_pass
+ * @property string $user_nicename
+ * @property string $user_email
+ * @property string $user_url
+ * @property string $user_registered
+ * @property string $user_activation_key
+ * @property string $user_status
+ * @property string $display_name
+ * @property string $spam
+ * @property string $deleted
  */
 class WP_User {
        /**
         * User data container.
         *
         * @since 2.0.0
-        * @access private
-        * @var array
+        * @var object
         */
-       var $data;
+       public $data;
 
        /**
         * The user's ID.
@@ -420,7 +442,7 @@ class WP_User {
         * @access public
         * @var int
         */
-       var $ID = 0;
+       public $ID = 0;
 
        /**
         * The individual capabilities the user has been given.
@@ -429,7 +451,7 @@ class WP_User {
         * @access public
         * @var array
         */
-       var $caps = array();
+       public $caps = array();
 
        /**
         * User metadata option name.
@@ -438,7 +460,7 @@ class WP_User {
         * @access public
         * @var string
         */
-       var $cap_key;
+       public $cap_key;
 
        /**
         * The roles the user is part of.
@@ -447,7 +469,7 @@ class WP_User {
         * @access public
         * @var array
         */
-       var $roles = array();
+       public $roles = array();
 
        /**
         * All capabilities the user has, including individual and role based.
@@ -456,7 +478,7 @@ class WP_User {
         * @access public
         * @var array
         */
-       var $allcaps = array();
+       public $allcaps = array();
 
        /**
         * The filter context applied to user data fields.
@@ -467,6 +489,11 @@ class WP_User {
         */
        var $filter = null;
 
+       /**
+        * @static
+        * @access private
+        * @var array
+        */
        private static $back_compat_keys;
 
        /**
@@ -477,12 +504,13 @@ class WP_User {
         * @since 2.0.0
         * @access public
         *
+        * @global wpdb $wpdb
+        *
         * @param int|string|stdClass|WP_User $id User's ID, a WP_User object, or a user object from the DB.
         * @param string $name Optional. User's username
         * @param int $blog_id Optional Blog ID, defaults to current blog.
-        * @return WP_User
         */
-       function __construct( $id = 0, $name = '', $blog_id = '' ) {
+       public function __construct( $id = 0, $name = '', $blog_id = '' ) {
                if ( ! isset( self::$back_compat_keys ) ) {
                        $prefix = $GLOBALS['wpdb']->prefix;
                        self::$back_compat_keys = array(
@@ -495,7 +523,7 @@ class WP_User {
                        );
                }
 
-               if ( is_a( $id, 'WP_User' ) ) {
+               if ( $id instanceof WP_User ) {
                        $this->init( $id->data, $blog_id );
                        return;
                } elseif ( is_object( $id ) ) {
@@ -508,13 +536,17 @@ class WP_User {
                        $id = 0;
                }
 
-               if ( $id )
+               if ( $id ) {
                        $data = self::get_data_by( 'id', $id );
-               else
+               } else {
                        $data = self::get_data_by( 'login', $name );
+               }
 
-               if ( $data )
+               if ( $data ) {
                        $this->init( $data, $blog_id );
+               } else {
+                       $this->data = new stdClass;
+               }
        }
 
        /**
@@ -523,7 +555,7 @@ class WP_User {
         * @param object $data User DB row object
         * @param int $blog_id Optional. The blog id to initialize for
         */
-       function init( $data, $blog_id = '' ) {
+       public function init( $data, $blog_id = '' ) {
                $this->data = $data;
                $this->ID = (int) $data->ID;
 
@@ -535,11 +567,15 @@ class WP_User {
         *
         * @since 3.3.0
         *
+        * @static
+        *
+        * @global wpdb $wpdb
+        *
         * @param string $field The field to query against: 'id', 'slug', 'email' or 'login'
         * @param string|int $value The field value
-        * @return object Raw user object
+        * @return object|false Raw user object
         */
-       static function get_data_by( $field, $value ) {
+       public static function get_data_by( $field, $value ) {
                global $wpdb;
 
                if ( 'id' == $field ) {
@@ -594,12 +630,31 @@ class WP_User {
                return $user;
        }
 
+       /**
+        * Makes private/protected methods readable for backwards compatibility.
+        *
+        * @since 4.3.0
+        * @access public
+        *
+        * @param callable $name      Method to call.
+        * @param array    $arguments Arguments to pass when calling.
+        * @return mixed|false Return value of the callback, false otherwise.
+        */
+       public function __call( $name, $arguments ) {
+               if ( '_init_caps' === $name ) {
+                       return call_user_func_array( array( $this, $name ), $arguments );
+               }
+               return false;
+       }
+
        /**
         * Magic method for checking the existence of a certain custom field
         *
         * @since 3.3.0
+        * @param string $key
+        * @return bool
         */
-       function __isset( $key ) {
+       public function __isset( $key ) {
                if ( 'id' == $key ) {
                        _deprecated_argument( 'WP_User->id', '2.1', __( 'Use <code>WP_User->ID</code> instead.' ) );
                        $key = 'ID';
@@ -618,8 +673,10 @@ class WP_User {
         * Magic method for accessing custom fields
         *
         * @since 3.3.0
+        * @param string $key
+        * @return mixed
         */
-       function __get( $key ) {
+       public function __get( $key ) {
                if ( 'id' == $key ) {
                        _deprecated_argument( 'WP_User->id', '2.1', __( 'Use <code>WP_User->ID</code> instead.' ) );
                        return $this->ID;
@@ -645,7 +702,7 @@ class WP_User {
         *
         * @since 3.3.0
         */
-       function __set( $key, $value ) {
+       public function __set( $key, $value ) {
                if ( 'id' == $key ) {
                        _deprecated_argument( 'WP_User->id', '2.1', __( 'Use <code>WP_User->ID</code> instead.' ) );
                        $this->ID = $value;
@@ -663,7 +720,7 @@ class WP_User {
         *
         * @return bool True if user exists in the database, false if not.
         */
-       function exists() {
+       public function exists() {
                return ! empty( $this->ID );
        }
 
@@ -675,8 +732,9 @@ class WP_User {
         * @since 3.3.0
         *
         * @param string $key Property
+        * @return mixed
         */
-       function get( $key ) {
+       public function get( $key ) {
                return $this->__get( $key );
        }
 
@@ -688,8 +746,9 @@ class WP_User {
         * @since 3.3.0
         *
         * @param string $key Property
+        * @return bool
         */
-       function has_prop( $key ) {
+       public function has_prop( $key ) {
                return $this->__isset( $key );
        }
 
@@ -700,7 +759,7 @@ class WP_User {
         *
         * @return array Array representation.
         */
-       function to_array() {
+       public function to_array() {
                return get_object_vars( $this->data );
        }
 
@@ -715,9 +774,11 @@ class WP_User {
         * @access protected
         * @since 2.1.0
         *
+        * @global wpdb $wpdb
+        *
         * @param string $cap_key Optional capability key
         */
-       function _init_caps( $cap_key = '' ) {
+       protected function _init_caps( $cap_key = '' ) {
                global $wpdb;
 
                if ( empty($cap_key) )
@@ -742,16 +803,12 @@ class WP_User {
         * granted permission to.
         *
         * @since 2.0.0
-        * @uses $wp_roles
         * @access public
         *
         * @return array List of all capabilities for the user.
         */
-       function get_role_caps() {
-               global $wp_roles;
-
-               if ( ! isset( $wp_roles ) )
-                       $wp_roles = new WP_Roles();
+       public function get_role_caps() {
+               $wp_roles = wp_roles();
 
                //Filter out caps that are not role names and assign to $this->roles
                if ( is_array( $this->caps ) )
@@ -778,11 +835,21 @@ class WP_User {
         *
         * @param string $role Role name.
         */
-       function add_role( $role ) {
+       public function add_role( $role ) {
                $this->caps[$role] = true;
                update_user_meta( $this->ID, $this->cap_key, $this->caps );
                $this->get_role_caps();
                $this->update_user_level_from_caps();
+
+               /**
+                * Fires immediately after the user has been given a new role.
+                *
+                * @since 4.3.0
+                *
+                * @param int    $user_id The user ID.
+                * @param string $role    The new role.
+                */
+               do_action( 'add_user_role', $this->ID, $role );
        }
 
        /**
@@ -793,13 +860,23 @@ class WP_User {
         *
         * @param string $role Role name.
         */
-       function remove_role( $role ) {
+       public function remove_role( $role ) {
                if ( !in_array($role, $this->roles) )
                        return;
                unset( $this->caps[$role] );
                update_user_meta( $this->ID, $this->cap_key, $this->caps );
                $this->get_role_caps();
                $this->update_user_level_from_caps();
+
+               /**
+                * Fires immediately after a role as been removed from a user.
+                *
+                * @since 4.3.0
+                *
+                * @param int    $user_id The user ID.
+                * @param string $role    The removed role.
+                */
+               do_action( 'remove_user_role', $this->ID, $role );
        }
 
        /**
@@ -814,7 +891,7 @@ class WP_User {
         *
         * @param string $role Role name.
         */
-       function set_role( $role ) {
+       public function set_role( $role ) {
                if ( 1 == count( $this->roles ) && $role == current( $this->roles ) )
                        return;
 
@@ -864,7 +941,7 @@ class WP_User {
         * @param string $item Level capability name.
         * @return int Max Level.
         */
-       function level_reduction( $max, $item ) {
+       public function level_reduction( $max, $item ) {
                if ( preg_match( '/^level_(10|[0-9])$/i', $item, $matches ) ) {
                        $level = intval( $matches[1] );
                        return max( $max, $level );
@@ -882,8 +959,10 @@ class WP_User {
         *
         * @since 2.0.0
         * @access public
+        *
+        * @global wpdb $wpdb
         */
-       function update_user_level_from_caps() {
+       public function update_user_level_from_caps() {
                global $wpdb;
                $this->user_level = array_reduce( array_keys( $this->allcaps ), array( $this, 'level_reduction' ), 0 );
                update_user_meta( $this->ID, $wpdb->get_blog_prefix() . 'user_level', $this->user_level );
@@ -898,9 +977,11 @@ class WP_User {
         * @param string $cap Capability name.
         * @param bool $grant Whether to grant capability to user.
         */
-       function add_cap( $cap, $grant = true ) {
+       public function add_cap( $cap, $grant = true ) {
                $this->caps[$cap] = $grant;
                update_user_meta( $this->ID, $this->cap_key, $this->caps );
+               $this->get_role_caps();
+               $this->update_user_level_from_caps();
        }
 
        /**
@@ -911,11 +992,14 @@ class WP_User {
         *
         * @param string $cap Capability name.
         */
-       function remove_cap( $cap ) {
-               if ( ! isset( $this->caps[$cap] ) )
+       public function remove_cap( $cap ) {
+               if ( ! isset( $this->caps[ $cap ] ) ) {
                        return;
-               unset( $this->caps[$cap] );
+               }
+               unset( $this->caps[ $cap ] );
                update_user_meta( $this->ID, $this->cap_key, $this->caps );
+               $this->get_role_caps();
+               $this->update_user_level_from_caps();
        }
 
        /**
@@ -923,8 +1007,10 @@ class WP_User {
         *
         * @since 2.1.0
         * @access public
+        *
+        * @global wpdb $wpdb
         */
-       function remove_all_caps() {
+       public function remove_all_caps() {
                global $wpdb;
                $this->caps = array();
                delete_user_meta( $this->ID, $this->cap_key );
@@ -945,7 +1031,7 @@ class WP_User {
         * @param string|int $cap Capability or role name to search.
         * @return bool True, if user has capability; false, if user does not have capability.
         */
-       function has_cap( $cap ) {
+       public function has_cap( $cap ) {
                if ( is_numeric( $cap ) ) {
                        _deprecated_argument( __FUNCTION__, '2.0', __('Usage of user levels by plugins and themes is deprecated. Use roles and capabilities instead.') );
                        $cap = $this->translate_level_to_cap( $cap );
@@ -968,7 +1054,7 @@ class WP_User {
                 * @since 2.0.0
                 * @since 3.7.0 Added the user object.
                 *
-                * @param array   $allcaps An array of all the role's capabilities.
+                * @param array   $allcaps An array of all the user's capabilities.
                 * @param array   $caps    Actual capabilities for meta capability.
                 * @param array   $args    Optional parameters passed to has_cap(), typically object ID.
                 * @param WP_User $user    The user object.
@@ -995,7 +1081,7 @@ class WP_User {
         * @param int $level Level number, 1 to 10.
         * @return string
         */
-       function translate_level_to_cap( $level ) {
+       public function translate_level_to_cap( $level ) {
                return 'level_' . $level;
        }
 
@@ -1004,9 +1090,11 @@ class WP_User {
         *
         * @since 3.0.0
         *
+        * @global wpdb $wpdb
+        *
         * @param int $blog_id Optional Blog ID, defaults to current blog.
         */
-       function for_blog( $blog_id = '' ) {
+       public function for_blog( $blog_id = '' ) {
                global $wpdb;
                if ( ! empty( $blog_id ) )
                        $cap_key = $wpdb->get_blog_prefix( $blog_id ) . 'capabilities';
@@ -1071,20 +1159,15 @@ function map_meta_cap( $cap, $user_id ) {
                        break;
                }
 
-               $post_author_id = $post->post_author;
-
-               // If no author set yet, default to current user for cap checks.
-               if ( ! $post_author_id )
-                       $post_author_id = $user_id;
-
-               // If the user is the author...
-               if ( $user_id == $post_author_id ) {
+               // If the post author is set and the user is the author...
+               if ( $post->post_author && $user_id == $post->post_author ) {
                        // If the post is published...
                        if ( 'publish' == $post->post_status ) {
                                $caps[] = $post_type->cap->delete_published_posts;
                        } elseif ( 'trash' == $post->post_status ) {
-                               if ('publish' == get_post_meta($post->ID, '_wp_trash_meta_status', true) )
+                               if ( 'publish' == get_post_meta( $post->ID, '_wp_trash_meta_status', true ) ) {
                                        $caps[] = $post_type->cap->delete_published_posts;
+                               }
                        } else {
                                // If the post is draft...
                                $caps[] = $post_type->cap->delete_posts;
@@ -1093,10 +1176,11 @@ function map_meta_cap( $cap, $user_id ) {
                        // The user is trying to edit someone else's post.
                        $caps[] = $post_type->cap->delete_others_posts;
                        // The post is published, extra cap required.
-                       if ( 'publish' == $post->post_status )
+                       if ( 'publish' == $post->post_status ) {
                                $caps[] = $post_type->cap->delete_published_posts;
-                       elseif ( 'private' == $post->post_status )
+                       } elseif ( 'private' == $post->post_status ) {
                                $caps[] = $post_type->cap->delete_private_posts;
+                       }
                }
                break;
                // edit_post breaks down to edit_posts, edit_published_posts, or
@@ -1104,8 +1188,10 @@ function map_meta_cap( $cap, $user_id ) {
        case 'edit_post':
        case 'edit_page':
                $post = get_post( $args[0] );
-               if ( empty( $post ) )
+               if ( empty( $post ) ) {
+                       $caps[] = 'do_not_allow';
                        break;
+               }
 
                if ( 'revision' == $post->post_type ) {
                        $post = get_post( $post->post_parent );
@@ -1121,20 +1207,15 @@ function map_meta_cap( $cap, $user_id ) {
                        break;
                }
 
-               $post_author_id = $post->post_author;
-
-               // If no author set yet, default to current user for cap checks.
-               if ( ! $post_author_id )
-                       $post_author_id = $user_id;
-
-               // If the user is the author...
-               if ( $user_id == $post_author_id ) {
+               // If the post author is set and the user is the author...
+               if ( $post->post_author && $user_id == $post->post_author ) {
                        // If the post is published...
                        if ( 'publish' == $post->post_status ) {
                                $caps[] = $post_type->cap->edit_published_posts;
                        } elseif ( 'trash' == $post->post_status ) {
-                               if ('publish' == get_post_meta($post->ID, '_wp_trash_meta_status', true) )
+                               if ( 'publish' == get_post_meta( $post->ID, '_wp_trash_meta_status', true ) ) {
                                        $caps[] = $post_type->cap->edit_published_posts;
+                               }
                        } else {
                                // If the post is draft...
                                $caps[] = $post_type->cap->edit_posts;
@@ -1143,10 +1224,11 @@ function map_meta_cap( $cap, $user_id ) {
                        // The user is trying to edit someone else's post.
                        $caps[] = $post_type->cap->edit_others_posts;
                        // The post is published, extra cap required.
-                       if ( 'publish' == $post->post_status )
+                       if ( 'publish' == $post->post_status ) {
                                $caps[] = $post_type->cap->edit_published_posts;
-                       elseif ( 'private' == $post->post_status )
+                       } elseif ( 'private' == $post->post_status ) {
                                $caps[] = $post_type->cap->edit_private_posts;
+                       }
                }
                break;
        case 'read_post':
@@ -1173,18 +1255,13 @@ function map_meta_cap( $cap, $user_id ) {
                        break;
                }
 
-               $post_author_id = $post->post_author;
-
-               // If no author set yet, default to current user for cap checks.
-               if ( ! $post_author_id )
-                       $post_author_id = $user_id;
-
-               if ( $user_id == $post_author_id )
+               if ( $post->post_author && $user_id == $post->post_author ) {
                        $caps[] = $post_type->cap->read;
-               elseif ( $status_obj->private )
+               } elseif ( $status_obj->private ) {
                        $caps[] = $post_type->cap->read_private_posts;
-               else
+               } else {
                        $caps = map_meta_cap( 'edit_post', $user_id, $post->ID );
+               }
                break;
        case 'publish_post':
                $post = get_post( $args[0] );
@@ -1204,8 +1281,8 @@ function map_meta_cap( $cap, $user_id ) {
                        /**
                         * Filter whether the user is allowed to add post meta to a post.
                         *
-                        * The dynamic portion of the hook name, $meta_key, refers to the
-                        * meta key passed to map_meta_cap().
+                        * The dynamic portion of the hook name, `$meta_key`, refers to the
+                        * meta key passed to {@see map_meta_cap()}.
                         *
                         * @since 3.3.0
                         *
@@ -1228,7 +1305,16 @@ function map_meta_cap( $cap, $user_id ) {
                if ( empty( $comment ) )
                        break;
                $post = get_post( $comment->comment_post_ID );
-               $caps = map_meta_cap( 'edit_post', $user_id, $post->ID );
+
+               /*
+                * If the post doesn't exist, we have an orphaned comment.
+                * Fall back to the edit_posts capability, instead.
+                */
+               if ( $post ) {
+                       $caps = map_meta_cap( 'edit_post', $user_id, $post->ID );
+               } else {
+                       $caps = map_meta_cap( 'edit_posts', $user_id );
+               }
                break;
        case 'unfiltered_upload':
                if ( defined('ALLOW_UNFILTERED_UPLOADS') && ALLOW_UNFILTERED_UPLOADS && ( !is_multisite() || is_super_admin( $user_id ) )  )
@@ -1261,18 +1347,25 @@ function map_meta_cap( $cap, $user_id ) {
        case 'update_plugins':
        case 'delete_plugins':
        case 'install_plugins':
+       case 'upload_plugins':
        case 'update_themes':
        case 'delete_themes':
        case 'install_themes':
+       case 'upload_themes':
        case 'update_core':
                // Disallow anything that creates, deletes, or updates core, plugin, or theme files.
                // Files in uploads are excepted.
-               if ( defined( 'DISALLOW_FILE_MODS' ) && DISALLOW_FILE_MODS )
+               if ( defined( 'DISALLOW_FILE_MODS' ) && DISALLOW_FILE_MODS ) {
                        $caps[] = 'do_not_allow';
-               elseif ( is_multisite() && ! is_super_admin( $user_id ) )
+               } elseif ( is_multisite() && ! is_super_admin( $user_id ) ) {
                        $caps[] = 'do_not_allow';
-               else
+               } elseif ( 'upload_themes' === $cap ) {
+                       $caps[] = 'install_themes';
+               } elseif ( 'upload_plugins' === $cap ) {
+                       $caps[] = 'install_plugins';
+               } else {
                        $caps[] = $cap;
+               }
                break;
        case 'activate_plugins':
                $caps[] = $cap;
@@ -1294,7 +1387,7 @@ function map_meta_cap( $cap, $user_id ) {
        case 'create_users':
                if ( !is_multisite() )
                        $caps[] = $cap;
-               elseif ( is_super_admin() || get_site_option( 'add_new_users' ) )
+               elseif ( is_super_admin( $user_id ) || get_site_option( 'add_new_users' ) )
                        $caps[] = $cap;
                else
                        $caps[] = 'do_not_allow';
@@ -1305,6 +1398,12 @@ function map_meta_cap( $cap, $user_id ) {
                else
                        $caps[] = 'do_not_allow';
                break;
+       case 'customize' :
+               $caps[] = 'edit_theme_options';
+               break;
+       case 'delete_site':
+               $caps[] = 'manage_options';
+               break;
        default:
                // Handle meta capabilities for custom post types.
                $post_type_meta_caps = _post_type_meta_capabilities();
@@ -1360,21 +1459,25 @@ function current_user_can( $capability ) {
  * @return bool
  */
 function current_user_can_for_blog( $blog_id, $capability ) {
-       if ( is_multisite() )
-               switch_to_blog( $blog_id );
+       $switched = is_multisite() ? switch_to_blog( $blog_id ) : false;
 
        $current_user = wp_get_current_user();
 
-       if ( empty( $current_user ) )
+       if ( empty( $current_user ) ) {
+               if ( $switched ) {
+                       restore_current_blog();
+               }
                return false;
+       }
 
        $args = array_slice( func_get_args(), 2 );
        $args = array_merge( array( $capability ), $args );
 
        $can = call_user_func_array( array( $current_user, 'has_cap' ), $args );
 
-       if ( is_multisite() )
+       if ( $switched ) {
                restore_current_blog();
+       }
 
        return $can;
 }
@@ -1425,28 +1528,39 @@ function user_can( $user, $capability ) {
        return call_user_func_array( array( $user, 'has_cap' ), $args );
 }
 
+/**
+ * Retrieves the global WP_Roles instance and instantiates it if necessary.
+ *
+ * @since 4.3.0
+ *
+ * @global WP_Roles $wp_roles WP_Roles global instance.
+ *
+ * @return WP_Roles WP_Roles global instance if not already instantiated.
+ */
+function wp_roles() {
+       global $wp_roles;
+
+       if ( ! isset( $wp_roles ) ) {
+               $wp_roles = new WP_Roles();
+       }
+       return $wp_roles;
+}
+
 /**
  * Retrieve role object.
  *
- * @see WP_Roles::get_role() Uses method to retrieve role object.
  * @since 2.0.0
  *
  * @param string $role Role name.
  * @return WP_Role|null WP_Role object if found, null if the role does not exist.
  */
 function get_role( $role ) {
-       global $wp_roles;
-
-       if ( ! isset( $wp_roles ) )
-               $wp_roles = new WP_Roles();
-
-       return $wp_roles->get_role( $role );
+       return wp_roles()->get_role( $role );
 }
 
 /**
  * Add role, if it does not exist.
  *
- * @see WP_Roles::add_role() Uses method to add role.
  * @since 2.0.0
  *
  * @param string $role Role name.
@@ -1455,29 +1569,18 @@ function get_role( $role ) {
  * @return WP_Role|null WP_Role object if role is added, null if already exists.
  */
 function add_role( $role, $display_name, $capabilities = array() ) {
-       global $wp_roles;
-
-       if ( ! isset( $wp_roles ) )
-               $wp_roles = new WP_Roles();
-
-       return $wp_roles->add_role( $role, $display_name, $capabilities );
+       return wp_roles()->add_role( $role, $display_name, $capabilities );
 }
 
 /**
  * Remove role, if it exists.
  *
- * @see WP_Roles::remove_role() Uses method to remove role.
  * @since 2.0.0
  *
  * @param string $role Role name.
  */
 function remove_role( $role ) {
-       global $wp_roles;
-
-       if ( ! isset( $wp_roles ) )
-               $wp_roles = new WP_Roles();
-
-       $wp_roles->remove_role( $role );
+       wp_roles()->remove_role( $role );
 }
 
 /**
@@ -1485,7 +1588,7 @@ function remove_role( $role ) {
  *
  * @since 3.0.0
  *
- * @uses $super_admins Super admins global variable, if set.
+ * @global array $super_admins
  *
  * @return array List of super admin logins
  */