Skip to:
Content

bbPress.org


Ignore:
Timestamp:
07/11/2014 09:54:15 PM (12 years ago)
Author:
johnjamesjacoby
Message:

Breathing room for ! usages in Users component.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/includes/users/functions.php

    r5436 r5442  
    2828
    2929    // Raw redirect_to was passed, so use it
    30     if ( !empty( $raw_url ) ) {
     30    if ( ! empty( $raw_url ) ) {
    3131        $url = $raw_url;
    3232
    … …  
    104104        $bbp_current_poster = wp_get_current_commenter();
    105105
    106         if ( !empty( $key ) && in_array( $key, array_keys( $cookie_names ) ) ) {
     106        if ( ! empty( $key ) && in_array( $key, array_keys( $cookie_names ) ) ) {
    107107            return $bbp_current_poster[$cookie_names[$key]];
    108108        }
    … …  
    125125 */
    126126function bbp_set_current_anonymous_user_data( $anonymous_data = array() ) {
    127     if ( empty( $anonymous_data ) || !is_array( $anonymous_data ) ) {
     127    if ( empty( $anonymous_data ) || ! is_array( $anonymous_data ) ) {
    128128        return;
    129129    }
    … …  
    157157 */
    158158function bbp_current_author_ua() {
    159     $retval = !empty( $_SERVER['HTTP_USER_AGENT'] ) ? substr( $_SERVER['HTTP_USER_AGENT'], 0, 254 ) : '';
     159    $retval = ! empty( $_SERVER['HTTP_USER_AGENT'] ) ? substr( $_SERVER['HTTP_USER_AGENT'], 0, 254 ) : '';
    160160
    161161    return apply_filters( 'bbp_current_author_ua', $retval );
    … …  
    213213    // If user has favorites, load them
    214214    $favorites = bbp_get_user_favorites_topic_ids( $user_id );
    215     if ( !empty( $favorites ) ) {
     215    if ( ! empty( $favorites ) ) {
    216216        $query = bbp_has_topics( array( 'post__in' => $favorites ) );
    217217    } else {
    … …  
    271271    $favorites = bbp_get_user_favorites_topic_ids( $user_id );
    272272
    273     if ( !empty( $favorites ) ) {
     273    if ( ! empty( $favorites ) ) {
    274274
    275275        // Checking a specific topic id
    276         if ( !empty( $topic_id ) ) {
     276        if ( ! empty( $topic_id ) ) {
    277277            $topic    = bbp_get_topic( $topic_id );
    278             $topic_id = !empty( $topic ) ? $topic->ID : 0;
     278            $topic_id = ! empty( $topic ) ? $topic->ID : 0;
    279279
    280280        // Using the global topic id
    … …  
    283283
    284284        // Use the current post id
    285         } elseif ( !bbp_get_topic_id() ) {
     285        } elseif ( ! bbp_get_topic_id() ) {
    286286            $topic_id = get_the_ID();
    287287        }
    288288
    289289        // Is topic_id in the user's favorites
    290         if ( !empty( $topic_id ) ) {
     290        if ( ! empty( $topic_id ) ) {
    291291            $retval = in_array( $topic_id, $favorites );
    292292        }
    … …  
    319319
    320320    $favorites = bbp_get_user_favorites_topic_ids( $user_id );
    321     if ( !in_array( $topic_id, $favorites ) ) {
     321    if ( ! in_array( $topic_id, $favorites ) ) {
    322322        $favorites[] = $topic_id;
    323323        $favorites   = implode( ',', wp_parse_id_list( array_filter( $favorites ) ) );
    … …  
    359359        $favorites = array_filter( $favorites );
    360360
    361         if ( !empty( $favorites ) ) {
     361        if ( ! empty( $favorites ) ) {
    362362            $favorites = implode( ',', wp_parse_id_list( $favorites ) );
    363363            update_user_option( $user_id, '_bbp_favorites', $favorites );
    … …  
    392392function bbp_favorites_handler( $action = '' ) {
    393393
    394     if ( !bbp_is_favorites_active() ) {
     394    if ( ! bbp_is_favorites_active() ) {
    395395        return false;
    396396    }
    … …  
    408408
    409409    // Bail if actions aren't meant for this function
    410     if ( !in_array( $action, $possible_actions ) ) {
     410    if ( ! in_array( $action, $possible_actions ) ) {
    411411        return;
    412412    }
    … …  
    425425
    426426    // Check current user's ability to edit the user
    427     } elseif ( !current_user_can( 'edit_user', $user_id ) ) {
     427    } elseif ( ! current_user_can( 'edit_user', $user_id ) ) {
    428428        bbp_add_error( 'bbp_favorite_permissions', __( '<strong>ERROR</strong>: You don\'t have the permission to edit favorites of that user!', 'bbpress' ) );
    429429    }
    … …  
    574574    // If user has subscriptions, load them
    575575    $subscriptions = bbp_get_user_subscribed_topic_ids( $user_id );
    576     if ( !empty( $subscriptions ) ) {
     576    if ( ! empty( $subscriptions ) ) {
    577577        $query = bbp_has_topics( array( 'post__in' => $subscriptions ) );
    578578    } else {
    … …  
    605605    // If user has subscriptions, load them
    606606    $subscriptions = bbp_get_user_subscribed_forum_ids( $user_id );
    607     if ( !empty( $subscriptions ) ) {
     607    if ( ! empty( $subscriptions ) ) {
    608608        $query = bbp_has_forums( array( 'post__in' => $subscriptions ) );
    609609    } else {
    … …  
    796796    // Validate user
    797797    $user_id = bbp_get_user_id( $user_id, true, true );
    798     if ( !empty( $user_id ) ) {
     798    if ( ! empty( $user_id ) ) {
    799799
    800800        // Get subscription ID's if none passed
    … …  
    816816
    817817            // Use the current post id
    818             } elseif ( !bbp_get_topic_id() ) {
     818            } elseif ( ! bbp_get_topic_id() ) {
    819819                $topic_id = get_the_ID();
    820820            }
    … …  
    900900
    901901    $subscriptions = (array) bbp_get_user_subscribed_forum_ids( $user_id );
    902     if ( !in_array( $forum_id, $subscriptions ) ) {
     902    if ( ! in_array( $forum_id, $subscriptions ) ) {
    903903        $subscriptions[] = $forum_id;
    904904        $subscriptions   = implode( ',', wp_parse_id_list( array_filter( $subscriptions ) ) );
    … …  
    937937
    938938    $subscriptions = (array) bbp_get_user_subscribed_topic_ids( $user_id );
    939     if ( !in_array( $topic_id, $subscriptions ) ) {
     939    if ( ! in_array( $topic_id, $subscriptions ) ) {
    940940        $subscriptions[] = $topic_id;
    941941        $subscriptions   = implode( ',', wp_parse_id_list( array_filter( $subscriptions ) ) );
    … …  
    10291029    $subscriptions = array_filter( $subscriptions );
    10301030
    1031     if ( !empty( $subscriptions ) ) {
     1031    if ( ! empty( $subscriptions ) ) {
    10321032        $subscriptions = implode( ',', wp_parse_id_list( $subscriptions ) );
    10331033        update_user_option( $user_id, '_bbp_forum_subscriptions', $subscriptions );
    … …  
    10761076    $subscriptions = array_filter( $subscriptions );
    10771077
    1078     if ( !empty( $subscriptions ) ) {
     1078    if ( ! empty( $subscriptions ) ) {
    10791079        $subscriptions = implode( ',', wp_parse_id_list( $subscriptions ) );
    10801080        update_user_option( $user_id, '_bbp_subscriptions', $subscriptions );
    … …  
    11461146
    11471147    // Check current user's ability to edit the user
    1148     } elseif ( !current_user_can( 'edit_user', $user_id ) ) {
     1148    } elseif ( ! current_user_can( 'edit_user', $user_id ) ) {
    11491149        bbp_add_error( 'bbp_subscription_permissions', __( '<strong>ERROR</strong>: You don\'t have the permission to edit favorites of that user!', 'bbpress' ) );
    11501150    }
    … …  
    12191219function bbp_subscriptions_handler( $action = '' ) {
    12201220
    1221     if ( !bbp_is_subscriptions_active() ) {
     1221    if ( ! bbp_is_subscriptions_active() ) {
    12221222        return false;
    12231223    }
    … …  
    12351235
    12361236    // Bail if actions aren't meant for this function
    1237     if ( !in_array( $action, $possible_actions ) ) {
     1237    if ( ! in_array( $action, $possible_actions ) ) {
    12381238        return;
    12391239    }
    … …  
    12521252
    12531253    // Check current user's ability to edit the user
    1254     } elseif ( !current_user_can( 'edit_user', $user_id ) ) {
     1254    } elseif ( ! current_user_can( 'edit_user', $user_id ) ) {
    12551255        bbp_add_error( 'bbp_subscription_permissions', __( '<strong>ERROR</strong>: You don\'t have the permission to edit favorites of that user!', 'bbpress' ) );
    12561256    }
    … …  
    13691369
    13701370    // Delete new email address from user options
    1371     } elseif ( is_multisite() && bbp_is_user_home_edit() && !empty( $_GET['dismiss'] ) && ( $user_id . '_new_email' === $_GET['dismiss'] ) ) {
     1371    } elseif ( is_multisite() && bbp_is_user_home_edit() && ! empty( $_GET['dismiss'] ) && ( $user_id . '_new_email' === $_GET['dismiss'] ) ) {
    13721372        delete_option( $user_id . '_new_email' );
    13731373        wp_safe_redirect( add_query_arg( array( 'updated' => 'true' ), bbp_get_user_profile_edit_url( $user_id ) ) );
    … …  
    13921392
    13931393    // Prevent edit_user() from wiping out the user's Toolbar on front setting
    1394     if ( !isset( $_POST['admin_bar_front'] ) && _get_admin_bar_pref( 'front', $user_id ) ) {
     1394    if ( ! isset( $_POST['admin_bar_front'] ) && _get_admin_bar_pref( 'front', $user_id ) ) {
    13951395        $_POST['admin_bar_front'] = 1;
    13961396    }
    … …  
    18341834
    18351835    // Bail if no username
    1836     $username = !empty( $_POST['log'] ) ? $_POST['log'] : '';
     1836    $username = ! empty( $_POST['log'] ) ? $_POST['log'] : '';
    18371837    if ( empty( $username ) ) {
    18381838        return;
Note: See TracChangeset for help on using the changeset viewer.