Skip to:
Content

bbPress.org

Changeset 6196


Ignore:
Timestamp:
12/28/2016 06:16:53 PM (10 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.