Skip to:
Content

bbPress.org

Ticket #2874: 2874.1.diff

File 2874.1.diff, 2.0 KB (added by tharsheblows, 4 years ago)

add in version_compare

  • src/includes/admin/users.php

     
    4242         */
    4343        function setup_actions() {
    4444
     45                global $wp_version;
     46
    4547                // Bail if in network admin
    4648                if ( is_network_admin() ) {
    4749                        return;
     
    5254
    5355                // WordPress user screen
    5456                add_action( 'restrict_manage_users',      array( $this, 'user_role_bulk_dropdown' )        );
    55                 add_filter( 'manage_users_columns',       array( $this, 'user_role_column'        )        );
    56                 add_filter( 'manage_users_custom_column', array( $this, 'user_role_row'           ), 10, 3 );
    5757
     58                // In WordPress >= 4.4, all roles are listed in the table so we don't need the additional columns.
     59                $wp_major_version = substr( $wp_version, 0, 3 );
     60                if( version_compare( $wp_major_version, 4.4, '<' ) ){
     61                        add_filter( 'manage_users_columns',       array( $this, 'user_role_column'        )        );
     62                        add_filter( 'manage_users_custom_column', array( $this, 'user_role_row'           ), 10, 3 );
     63                }
     64
    5865                // Process bulk role change
    5966                add_action( 'load-users.php',             array( $this, 'user_role_bulk_change'   )        );
    6067        }
     
    127134         */
    128135        public static function user_role_bulk_dropdown() {
    129136
    130                 // Bail if current user cannot promote users 
     137                // Bail if current user cannot promote users
    131138                if ( ! current_user_can( 'promote_users' ) ) {
    132139                        return;
    133140                }
     
    182189                // Bail if nonce check fails
    183190                check_admin_referer( 'bbp-bulk-users', 'bbp-bulk-users-nonce' );
    184191
    185                 // Bail if current user cannot promote users 
     192                // Bail if current user cannot promote users
    186193                if ( ! current_user_can( 'promote_users' ) ) {
    187194                        return;
    188195                }
     
    200207                        }
    201208
    202209                        // Set up user and role data
    203                         $user_role = bbp_get_user_role( $user_id );                     
     210                        $user_role = bbp_get_user_role( $user_id );
    204211                        $new_role  = sanitize_text_field( $_REQUEST['bbp-new-role'] );
    205212
    206213                        // Only keymasters can set other keymasters