Skip to:
Content

bbPress.org

Changeset 6196


Ignore:
Timestamp:
12/28/2016 06:16:53 PM (9 years ago)
Author:
johnjamesjacoby
Message:

Breathing room for all !is_ usages.

Location:
trunk/src/includes
Files:
15 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/includes/admin/converters/e107v1.php

    r6191 r6196  
    582582            }
    583583
    584             if ( !is_null( $row ) ) {
     584            if ( ! is_null( $row ) ) {
    585585                $this->map_userid[ $field ] = $row->value_id;
    586586            } else {

  • trunk/src/includes/admin/tools.php

    r6193 r6196  
    15541554    $values = array();
    15551555    foreach ( $users as $user ) {
    1556         if ( empty( $user->favorites ) || !is_string( $user->favorites ) ) {
     1556        if ( empty( $user->favorites ) || ! is_string( $user->favorites ) ) {
    15571557            continue;
    15581558        }
    15591559
    15601560        $favorites = array_intersect( $topics, explode( ',', $user->favorites ) );
    1561         if ( empty( $favorites ) || !is_array( $favorites ) ) {
     1561        if ( empty( $favorites ) || ! is_array( $favorites ) ) {
    15621562            continue;
    15631563        }
    … …  
    16231623    $values = array();
    16241624    foreach ( $users as $user ) {
    1625         if ( empty( $user->subscriptions ) || !is_string( $user->subscriptions ) ) {
     1625        if ( empty( $user->subscriptions ) || ! is_string( $user->subscriptions ) ) {
    16261626            continue;
    16271627        }
    16281628
    16291629        $subscriptions = array_intersect( $topics, explode( ',', $user->subscriptions ) );
    1630         if ( empty( $subscriptions ) || !is_array( $subscriptions ) ) {
     1630        if ( empty( $subscriptions ) || ! is_array( $subscriptions ) ) {
    16311631            continue;
    16321632        }
    … …  
    16921692    $values = array();
    16931693    foreach ( $users as $user ) {
    1694         if ( empty( $user->subscriptions ) || !is_string( $user->subscriptions ) ) {
     1694        if ( empty( $user->subscriptions ) || ! is_string( $user->subscriptions ) ) {
    16951695            continue;
    16961696        }
    16971697
    16981698        $subscriptions = array_intersect( $forums, explode( ',', $user->subscriptions ) );
    1699         if ( empty( $subscriptions ) || !is_array( $subscriptions ) ) {
     1699        if ( empty( $subscriptions ) || ! is_array( $subscriptions ) ) {
    17001700            continue;
    17011701        }

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

    r6191 r6196  
    126126        );
    127127
    128         if ( ! empty( $older_date ) && !is_numeric( $older_date ) ) {
     128        if ( ! empty( $older_date ) && ! is_numeric( $older_date ) ) {
    129129            $time_chunks = explode( ':', str_replace( ' ', ':', $older_date ) );
    130130            $date_chunks = explode( '-', str_replace( ' ', '-', $older_date ) );

  • trunk/src/includes/common/shortcodes.php

    r5957 r6196  
    223223
    224224        // Sanity check required info
    225         if ( ! empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) ) {
     225        if ( ! empty( $content ) || ( empty( $attr['id'] ) || ! is_numeric( $attr['id'] ) ) ) {
    226226            return $content;
    227227        }
    … …  
    317317
    318318        // Sanity check required info
    319         if ( ! empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) ) {
     319        if ( ! empty( $content ) || ( empty( $attr['id'] ) || ! is_numeric( $attr['id'] ) ) ) {
    320320            return $content;
    321321        }
    … …  
    383383
    384384        // Sanity check supplied info
    385         if ( ! empty( $content ) || ( ! empty( $attr['forum_id'] ) && ( !is_numeric( $attr['forum_id'] ) || ! bbp_is_forum( $attr['forum_id'] ) ) ) ) {
     385        if ( ! empty( $content ) || ( ! empty( $attr['forum_id'] ) && ( ! is_numeric( $attr['forum_id'] ) || ! bbp_is_forum( $attr['forum_id'] ) ) ) ) {
    386386            return $content;
    387387        }
    … …  
    438438
    439439        // Sanity check required info
    440         if ( ! empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) ) {
     440        if ( ! empty( $content ) || ( empty( $attr['id'] ) || ! is_numeric( $attr['id'] ) ) ) {
    441441            return $content;
    442442        }
    … …  
    550550
    551551        // Sanity check required info
    552         if ( ! empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) ) {
     552        if ( ! empty( $content ) || ( empty( $attr['id'] ) || ! is_numeric( $attr['id'] ) ) ) {
    553553            return $content;
    554554        }
    … …  
    734734
    735735        // Output templates
    736         if ( !is_user_logged_in() ) {
     736        if ( ! is_user_logged_in() ) {
    737737            bbp_get_template_part( 'form',     'user-login' );
    738738        } else {
    … …  
    760760
    761761        // Output templates
    762         if ( !is_user_logged_in() ) {
     762        if ( ! is_user_logged_in() ) {
    763763            bbp_get_template_part( 'form',     'user-register' );
    764764        } else {
    … …  
    786786
    787787        // Output templates
    788         if ( !is_user_logged_in() ) {
     788        if ( ! is_user_logged_in() ) {
    789789            bbp_get_template_part( 'form',     'user-lost-pass' );
    790790        } else {

  • trunk/src/includes/common/template.php

    r6097 r6196  
    14731473
    14741474        // Force array
    1475         if ( ! empty( $r['exclude'] ) && !is_array( $r['exclude'] ) ) {
     1475        if ( ! empty( $r['exclude'] ) && ! is_array( $r['exclude'] ) ) {
    14761476            $r['exclude'] = explode( ',', $r['exclude'] );
    14771477        }

  • trunk/src/includes/common/widgets.php

    r6038 r6196  
    8383        }
    8484
    85         if ( !is_user_logged_in() ) : ?>
     85        if ( ! is_user_logged_in() ) : ?>
    8686
    8787            <form method="post" action="<?php bbp_wp_login_action( array( 'context' => 'login_post' ) ); ?>" class="bbp-login-form">
    … …  
    615615
    616616        // Force to any
    617         if ( ! empty( $instance['parent_forum'] ) && !is_numeric( $instance['parent_forum'] ) ) {
     617        if ( ! empty( $instance['parent_forum'] ) && ! is_numeric( $instance['parent_forum'] ) ) {
    618618            $instance['parent_forum'] = 'any';
    619619        }
    … …  
    860860
    861861        // Force to any
    862         if ( ! empty( $instance['parent_forum'] ) && !is_numeric( $instance['parent_forum'] ) ) {
     862        if ( ! empty( $instance['parent_forum'] ) && ! is_numeric( $instance['parent_forum'] ) ) {
    863863            $instance['parent_forum'] = 'any';
    864864        }

  • trunk/src/includes/core/theme-compat.php

    r5951 r6196  
    486486
    487487    // Bail if shortcodes are unset somehow
    488     if ( !is_a( $bbp_shortcodes, 'BBP_Shortcodes' ) ) {
     488    if ( ! is_a( $bbp_shortcodes, 'BBP_Shortcodes' ) ) {
    489489        return $template;
    490490    }

  • trunk/src/includes/extend/buddypress/functions.php

    r6180 r6196  
    310310
    311311    // Make sure result is an array
    312     if ( !is_array( $forum_ids ) ) {
     312    if ( ! is_array( $forum_ids ) ) {
    313313        $forum_ids = (array) $forum_ids;
    314314    }
    … …  
    342342
    343343    // Make sure result is an array
    344     if ( !is_array( $group_ids ) ) {
     344    if ( ! is_array( $group_ids ) ) {
    345345        $group_ids = (array) $group_ids;
    346346    }

  • trunk/src/includes/extend/buddypress/groups.php

    r6115 r6196  
    304304            <?php endif; ?>
    305305
    306             <?php if ( !is_admin() ) : ?>
     306            <?php if ( ! is_admin() ) : ?>
    307307                <input type="submit" value="<?php esc_attr_e( 'Save Settings', 'bbpress' ); ?>" />
    308308            <?php endif; ?>
    … …  
    426426
    427427        // Redirect after save when not in admin
    428         if ( !is_admin() ) {
     428        if ( ! is_admin() ) {
    429429            bp_core_redirect( trailingslashit( bp_get_group_permalink( buddypress()->groups->current_group ) . '/admin/' . $this->slug ) );
    430430        }

  • trunk/src/includes/extend/buddypress/loader.php

    r5951 r6196  
    168168
    169169        // Stop if there is no user displayed or logged in
    170         if ( !is_user_logged_in() && !bp_displayed_user_id() ) {
     170        if ( ! is_user_logged_in() && !bp_displayed_user_id() ) {
    171171            return;
    172172        }

  • trunk/src/includes/forums/capabilities.php

    r6087 r6196  
    138138
    139139                // User is author so allow edit if not in admin
    140                 } elseif ( !is_admin() && ( (int) $user_id === (int) $_post->post_author ) ) {
     140                } elseif ( ! is_admin() && ( (int) $user_id === (int) $_post->post_author ) ) {
    141141                    $caps[] = $post_type->cap->edit_posts;
    142142

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

    r6141 r6196  
    276276    /** No Errors *************************************************************/
    277277
    278     if ( ! empty( $forum_id ) && !is_wp_error( $forum_id ) ) {
     278    if ( ! empty( $forum_id ) && ! is_wp_error( $forum_id ) ) {
    279279
    280280        /** Trash Check *******************************************************/
    … …  
    549549    /** No Errors *************************************************************/
    550550
    551     if ( ! empty( $forum_id ) && !is_wp_error( $forum_id ) ) {
     551    if ( ! empty( $forum_id ) && ! is_wp_error( $forum_id ) ) {
    552552
    553553        // Update counts, etc...
    … …  
    21732173
    21742174    // Bail if $posts_query is not an object or of incorrect class
    2175     if ( !is_object( $posts_query ) || !is_a( $posts_query, 'WP_Query' ) ) {
     2175    if ( ! is_object( $posts_query ) || ! is_a( $posts_query, 'WP_Query' ) ) {
    21762176        return;
    21772177    }
    … …  
    23582358
    23592359    // Bail if not viewing a single item or if user has caps
    2360     if ( !is_singular() || bbp_is_user_keymaster() || current_user_can( 'read_hidden_forums' ) ) {
     2360    if ( ! is_singular() || bbp_is_user_keymaster() || current_user_can( 'read_hidden_forums' ) ) {
    23612361        return;
    23622362    }
    … …  
    24132413
    24142414    // Bail if not viewing a single item or if user has caps
    2415     if ( !is_singular() || bbp_is_user_keymaster() || current_user_can( 'read_private_forums' ) ) {
     2415    if ( ! is_singular() || bbp_is_user_keymaster() || current_user_can( 'read_private_forums' ) ) {
    24162416        return;
    24172417    }

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

    r6146 r6196  
    410410
    411411    // Check for missing reply_id or error
    412     if ( ! empty( $reply_id ) && !is_wp_error( $reply_id ) ) {
     412    if ( ! empty( $reply_id ) && ! is_wp_error( $reply_id ) ) {
    413413
    414414        /** Topic Tags ********************************************************/
    … …  
    772772    /** No Errors *************************************************************/
    773773
    774     if ( ! empty( $reply_id ) && !is_wp_error( $reply_id ) ) {
     774    if ( ! empty( $reply_id ) && ! is_wp_error( $reply_id ) ) {
    775775
    776776        // Update counts, etc...

  • trunk/src/includes/replies/template.php

    r6185 r6196  
    785785
    786786        // Check reply and revision log exist
    787         if ( empty( $reply_id ) || empty( $revision_log ) || !is_array( $revision_log ) ) {
     787        if ( empty( $reply_id ) || empty( $revision_log ) || ! is_array( $revision_log ) ) {
    788788            return false;
    789789        }

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

    r6192 r6196  
    5555 */
    5656function bbp_is_anonymous() {
    57     if ( !is_user_logged_in() && bbp_allow_anonymous() ) {
     57    if ( ! is_user_logged_in() && bbp_allow_anonymous() ) {
    5858        $is_anonymous = true;
    5959    } else {
Note: See TracChangeset for help on using the changeset viewer.