Skip to:
Content

bbPress.org

Changeset 5437


Ignore:
Timestamp:
07/09/2014 11:22:29 PM (7 years ago)
Author:
johnjamesjacoby
Message:

Bracketize common component.

Location:
trunk/src/includes/common
Files:
7 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/includes/common/ajax.php

    r5037 r5437  
    6363
    6464    // Bail if not an ajax request
    65     if ( ! bbp_is_ajax() )
     65    if ( ! bbp_is_ajax() ) {
    6666        return;
     67    }
    6768
    6869    // Set WordPress core ajax constant
     
    9495
    9596    // Set status to 200 if setting response as successful
    96     if ( ( true === $success ) && ( -1 === $status ) )
     97    if ( ( true === $success ) && ( -1 === $status ) ) {
    9798        $status = 200;
     99    }
    98100
    99101    // Setup the response array
  • trunk/src/includes/common/classes.php

    r5398 r5437  
    7272     */
    7373    public function __construct( $args = '' ) {
    74         if ( empty( $args ) )
     74        if ( empty( $args ) ) {
    7575            return;
     76        }
    7677
    7778        $this->setup_globals( $args );
     
    360361    public function display_element( $element = false, &$children_elements = array(), $max_depth = 0, $depth = 0, $args = array(), &$output = '' ) {
    361362
    362         if ( empty( $element ) )
     363        if ( empty( $element ) ) {
    363364            return;
     365        }
    364366
    365367        // Get element's id
  • trunk/src/includes/common/formatting.php

    r5220 r5437  
    353353
    354354            $url_clickable = '~
    355                 ([\\s(<.,;:!?])                                        # 1: Leading whitespace, or punctuation
    356                 (                                                      # 2: URL
    357                     [\\w]{1,20}+://                                # Scheme and hier-part prefix
    358                     (?=\S{1,2000}\s)                               # Limit to URLs less than about 2000 characters long
    359                     [\\w\\x80-\\xff#%\\~/@\\[\\]*(+=&$-]*+         # Non-punctuation URL character
    360                     (?:                                            # Unroll the Loop: Only allow puctuation URL character if followed by a non-punctuation URL character
     355                ([\\s(<.,;:!?])                                # 1: Leading whitespace, or punctuation
     356                (                                              # 2: URL
     357                    [\\w]{1,20}+://                            # Scheme and hier-part prefix
     358                    (?=\S{1,2000}\s)                           # Limit to URLs less than about 2000 characters long
     359                    [\\w\\x80-\\xff#%\\~/@\\[\\]*(+=&$-]*+     # Non-punctuation URL character
     360                    (?:                                        # Unroll the Loop: Only allow puctuation URL character if followed by a non-punctuation URL character
    361361                        [\'.,;:!?)]                            # Punctuation URL character
    362362                        [\\w\\x80-\\xff#%\\~/@\\[\\]*(+=&$-]++ # Non-punctuation URL character
    363363                    )*
    364364                )
    365                 (\)?)                                                  # 3: Trailing closing parenthesis (for parethesis balancing post processing)
    366             ~xS'; // The regex is a non-anchored pattern and does not have a single fixed starting character.
    367                   // Tell PCRE to spend more time optimizing since, when used on a page load, it will probably be used several times.
     365                (\)?)                                          # 3: Trailing closing parenthesis (for parethesis balancing post processing)
     366            ~xS';
     367
     368            // The regex is a non-anchored pattern and does not have a single fixed starting character.
     369            // Tell PCRE to spend more time optimizing since, when used on a page load, it will probably be used several times.
    368370
    369371            $ret = preg_replace_callback( $url_clickable, '_make_url_clickable_cb', $ret );
  • trunk/src/includes/common/functions.php

    r5420 r5437  
    3030
    3131    // If empty, set $number to (int) 0
    32     if ( ! is_numeric( $number ) )
     32    if ( ! is_numeric( $number ) ) {
    3333        $number = 0;
     34    }
    3435
    3536    return apply_filters( 'bbp_number_format', number_format( $number, $decimals, $dec_point, $thousands_sep ), $number, $decimals, $dec_point, $thousands_sep );
     
    5051
    5152    // If empty, set $number to (int) 0
    52     if ( ! is_numeric( $number ) )
     53    if ( ! is_numeric( $number ) ) {
    5354        $number = 0;
     55    }
    5456
    5557    return apply_filters( 'bbp_number_format_i18n', number_format_i18n( $number, $decimals ), $number, $decimals );
     
    319321
    320322    // Paged found
    321     if ( !empty( $paged ) )
     323    if ( !empty( $paged ) ) {
    322324        return (int) $paged;
     325    }
    323326
    324327    // Default to first page
     
    346349
    347350    // Post is not being updated or the post_author is already 0, return
    348     if ( empty( $postarr['ID'] ) || empty( $data['post_author'] ) )
     351    if ( empty( $postarr['ID'] ) || empty( $data['post_author'] ) ) {
    349352        return $data;
     353    }
    350354
    351355    // Post is not a topic or reply, return
    352     if ( !in_array( $data['post_type'], array( bbp_get_topic_post_type(), bbp_get_reply_post_type() ) ) )
     356    if ( !in_array( $data['post_type'], array( bbp_get_topic_post_type(), bbp_get_reply_post_type() ) ) ) {
    353357        return $data;
     358    }
    354359
    355360    // Is the post by an anonymous user?
    356361    if ( ( bbp_get_topic_post_type() === $data['post_type'] && !bbp_is_topic_anonymous( $postarr['ID'] ) ) ||
    357          ( bbp_get_reply_post_type() === $data['post_type'] && !bbp_is_reply_anonymous( $postarr['ID'] ) ) )
     362         ( bbp_get_reply_post_type() === $data['post_type'] && !bbp_is_reply_anonymous( $postarr['ID'] ) ) ) {
    358363        return $data;
     364    }
    359365
    360366    // The post is being updated. It is a topic or a reply and is written by an anonymous user.
     
    622628    // Filter variables and add errors if necessary
    623629    $r['bbp_anonymous_name'] = apply_filters( 'bbp_pre_anonymous_post_author_name',  $r['bbp_anonymous_name']  );
    624     if ( empty( $r['bbp_anonymous_name'] ) )
     630    if ( empty( $r['bbp_anonymous_name'] ) ) {
    625631        bbp_add_error( 'bbp_anonymous_name',  __( '<strong>ERROR</strong>: Invalid author name submitted!',   'bbpress' ) );
     632    }
    626633
    627634    $r['bbp_anonymous_email'] = apply_filters( 'bbp_pre_anonymous_post_author_email', $r['bbp_anonymous_email'] );
    628     if ( empty( $r['bbp_anonymous_email'] ) )
     635    if ( empty( $r['bbp_anonymous_email'] ) ) {
    629636        bbp_add_error( 'bbp_anonymous_email', __( '<strong>ERROR</strong>: Invalid email address submitted!', 'bbpress' ) );
     637    }
    630638
    631639    // Website is optional
     
    660668
    661669    // No duplicate checks for those who can throttle
    662     if ( current_user_can( 'throttle' ) )
     670    if ( current_user_can( 'throttle' ) ) {
    663671        return true;
     672    }
    664673
    665674    // Define global to use get_meta_sql() and get_var() methods
     
    735744    // Option disabled. No flood checks.
    736745    $throttle_time = get_option( '_bbp_throttle_time' );
    737     if ( empty( $throttle_time ) )
     746    if ( empty( $throttle_time ) ) {
    738747        return true;
     748    }
    739749
    740750    // User is anonymous, so check a transient based on the IP
     
    778788
    779789    // Allow for moderation check to be skipped
    780     if ( apply_filters( 'bbp_bypass_check_for_moderation', false, $anonymous_data, $author_id, $title, $content ) )
     790    if ( apply_filters( 'bbp_bypass_check_for_moderation', false, $anonymous_data, $author_id, $title, $content ) ) {
    781791        return true;
     792    }
    782793
    783794    // Bail if keymaster is author
    784     if ( !empty( $author_id ) && bbp_is_user_keymaster( $author_id ) )
     795    if ( !empty( $author_id ) && bbp_is_user_keymaster( $author_id ) ) {
    785796        return true;
     797    }
    786798
    787799    // Define local variable(s)
     
    795807
    796808    // Bail if blacklist is empty
    797     if ( empty( $blacklist ) )
     809    if ( empty( $blacklist ) ) {
    798810        return true;
     811    }
    799812
    800813    /** User Data *************************************************************/
     
    897910
    898911    // Allow for blacklist check to be skipped
    899     if ( apply_filters( 'bbp_bypass_check_for_blacklist', false, $anonymous_data, $author_id, $title, $content ) )
     912    if ( apply_filters( 'bbp_bypass_check_for_blacklist', false, $anonymous_data, $author_id, $title, $content ) ) {
    900913        return true;
     914    }
    901915
    902916    // Bail if keymaster is author
    903     if ( !empty( $author_id ) && bbp_is_user_keymaster( $author_id ) )
     917    if ( !empty( $author_id ) && bbp_is_user_keymaster( $author_id ) ) {
    904918        return true;
     919    }
    905920
    906921    // Define local variable
     
    913928
    914929    // Bail if blacklist is empty
    915     if ( empty( $blacklist ) )
     930    if ( empty( $blacklist ) ) {
    916931        return true;
     932    }
    917933
    918934    /** User Data *************************************************************/
     
    14281444
    14291445    // Noop if WP core supports this already
    1430     if ( in_array( 'post_parent__in', $wp->private_query_vars ) )
     1446    if ( in_array( 'post_parent__in', $wp->private_query_vars ) ) {
    14311447        return $where;
     1448    }
    14321449
    14331450    // Bail if no object passed
    1434     if ( empty( $object ) )
     1451    if ( empty( $object ) ) {
    14351452        return $where;
     1453    }
    14361454
    14371455    // Only 1 post_parent so return $where
    1438     if ( is_numeric( $object->query_vars['post_parent'] ) )
     1456    if ( is_numeric( $object->query_vars['post_parent'] ) ) {
    14391457        return $where;
     1458    }
    14401459
    14411460    // Including specific post_parent's
     
    14721491
    14731492    // Bail if nothing passed
    1474     if ( empty( $parent_id ) )
     1493    if ( empty( $parent_id ) ) {
    14751494        return false;
     1495    }
    14761496
    14771497    // The ID of the cached query
     
    15171537
    15181538    // Bail if nothing passed
    1519     if ( empty( $parent_id ) )
     1539    if ( empty( $parent_id ) ) {
    15201540        return false;
     1541    }
    15211542
    15221543    // The ID of the cached query
     
    15621583
    15631584    // Bail if nothing passed
    1564     if ( empty( $parent_id ) )
     1585    if ( empty( $parent_id ) ) {
    15651586        return false;
     1587    }
    15661588
    15671589    // The ID of the cached query
     
    16071629
    16081630    // Bail if nothing passed
    1609     if ( empty( $parent_id ) )
     1631    if ( empty( $parent_id ) ) {
    16101632        return false;
     1633    }
    16111634
    16121635    // The ID of the cached query
  • trunk/src/includes/common/shortcodes.php

    r5163 r5437  
    224224
    225225        // Sanity check required info
    226         if ( !empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) )
    227             return $content;
     226        if ( !empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) ) {
     227            return $content;
     228        }
    228229
    229230        // Set passed attribute to $forum_id for clarity
     
    231232
    232233        // Bail if ID passed is not a forum
    233         if ( !bbp_is_forum( $forum_id ) )
    234             return $content;
     234        if ( !bbp_is_forum( $forum_id ) ) {
     235            return $content;
     236        }
    235237
    236238        // Start output buffer
     
    318320
    319321        // Sanity check required info
    320         if ( !empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) )
    321             return $content;
     322        if ( !empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) ) {
     323            return $content;
     324        }
    322325
    323326        // Unset globals
     
    329332
    330333        // Bail if ID passed is not a topic
    331         if ( !bbp_is_topic( $topic_id ) )
    332             return $content;
     334        if ( !bbp_is_topic( $topic_id ) ) {
     335            return $content;
     336        }
    333337
    334338        // Reset the queries if not in theme compat
     
    382386
    383387        // Sanity check supplied info
    384         if ( !empty( $content ) || ( !empty( $attr['forum_id'] ) && ( !is_numeric( $attr['forum_id'] ) || !bbp_is_forum( $attr['forum_id'] ) ) ) )
    385             return $content;
     388        if ( !empty( $content ) || ( !empty( $attr['forum_id'] ) && ( !is_numeric( $attr['forum_id'] ) || !bbp_is_forum( $attr['forum_id'] ) ) ) ) {
     389            return $content;
     390        }
    386391
    387392        // Unset globals
     
    436441
    437442        // Sanity check required info
    438         if ( !empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) )
    439             return $content;
     443        if ( !empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) ) {
     444            return $content;
     445        }
    440446
    441447        // Unset globals
     
    447453
    448454        // Bail if ID passed is not a reply
    449         if ( !bbp_is_reply( $reply_id ) )
    450             return $content;
     455        if ( !bbp_is_reply( $reply_id ) ) {
     456            return $content;
     457        }
    451458
    452459        // Reset the queries if not in theme compat
     
    546553
    547554        // Sanity check required info
    548         if ( !empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) )
    549             return $content;
     555        if ( !empty( $content ) || ( empty( $attr['id'] ) || !is_numeric( $attr['id'] ) ) ) {
     556            return $content;
     557        }
    550558
    551559        // Unset globals
     
    613621
    614622        // Sanity check required info
    615         if ( empty( $attr['id'] ) )
    616             return $content;
     623        if ( empty( $attr['id'] ) ) {
     624            return $content;
     625        }
    617626
    618627        // Set passed attribute to $view_id for clarity
     
    730739
    731740        // Output templates
    732         if ( !is_user_logged_in() )
     741        if ( !is_user_logged_in() ) {
    733742            bbp_get_template_part( 'form',     'user-login' );
    734         else
     743        } else {
    735744            bbp_get_template_part( 'feedback', 'logged-in'  );
     745        }
    736746
    737747        // Return contents of output buffer
     
    755765
    756766        // Output templates
    757         if ( !is_user_logged_in() )
     767        if ( !is_user_logged_in() ) {
    758768            bbp_get_template_part( 'form',     'user-register' );
    759         else
     769        } else {
    760770            bbp_get_template_part( 'feedback', 'logged-in'     );
     771        }
    761772
    762773        // Return contents of output buffer
     
    780791
    781792        // Output templates
    782         if ( !is_user_logged_in() )
     793        if ( !is_user_logged_in() ) {
    783794            bbp_get_template_part( 'form',     'user-lost-pass' );
    784         else
     795        } else {
    785796            bbp_get_template_part( 'feedback', 'logged-in'      );
     797        }
    786798
    787799        // Return contents of output buffer
  • trunk/src/includes/common/template.php

    r5388 r5437  
    105105
    106106    // Get the current site ID
    107     if ( empty( $site_id ) )
     107    if ( empty( $site_id ) ) {
    108108        $site_id = get_current_blog_id();
     109    }
    109110
    110111    // Get the site visibility setting
     
    129130
    130131    // Supplied ID is a forum
    131     if ( !empty( $post_id ) && ( bbp_get_forum_post_type() === get_post_type( $post_id ) ) )
    132         $retval = true;
     132    if ( !empty( $post_id ) && ( bbp_get_forum_post_type() === get_post_type( $post_id ) ) ) {
     133        $retval = true;
     134    }
    133135
    134136    return (bool) apply_filters( 'bbp_is_forum', $retval, $post_id );
     
    152154
    153155    // In forum archive
    154     if ( is_post_type_archive( bbp_get_forum_post_type() ) || bbp_is_query_name( 'bbp_forum_archive' ) || !empty( $wp_query->bbp_show_topics_on_root ) )
    155         $retval = true;
     156    if ( is_post_type_archive( bbp_get_forum_post_type() ) || bbp_is_query_name( 'bbp_forum_archive' ) || !empty( $wp_query->bbp_show_topics_on_root ) ) {
     157        $retval = true;
     158    }
    156159
    157160    return (bool) apply_filters( 'bbp_is_forum_archive', $retval );
     
    180183
    181184    // Single and a match
    182     if ( is_singular( bbp_get_forum_post_type() ) || bbp_is_query_name( 'bbp_single_forum' ) )
    183         $retval = true;
     185    if ( is_singular( bbp_get_forum_post_type() ) || bbp_is_query_name( 'bbp_single_forum' ) ) {
     186        $retval = true;
     187    }
    184188
    185189    return (bool) apply_filters( 'bbp_is_single_forum', $retval );
     
    201205
    202206    // Check query
    203     if ( !empty( $wp_query->bbp_is_forum_edit ) && ( $wp_query->bbp_is_forum_edit === true ) )
     207    if ( !empty( $wp_query->bbp_is_forum_edit ) && ( $wp_query->bbp_is_forum_edit === true ) ) {
    204208        $retval = true;
    205209
    206210    // Editing in admin
    207     elseif ( is_admin() && ( 'post.php' === $pagenow ) && ( get_post_type() === bbp_get_forum_post_type() ) && ( !empty( $_GET['action'] ) && ( 'edit' === $_GET['action'] ) ) )
    208         $retval = true;
     211    } elseif ( is_admin() && ( 'post.php' === $pagenow ) && ( get_post_type() === bbp_get_forum_post_type() ) && ( !empty( $_GET['action'] ) && ( 'edit' === $_GET['action'] ) ) ) {
     212        $retval = true;
     213    }
    209214
    210215    return (bool) apply_filters( 'bbp_is_forum_edit', $retval );
     
    227232
    228233    // Supplied ID is a topic
    229     if ( !empty( $post_id ) && ( bbp_get_topic_post_type() === get_post_type( $post_id ) ) )
    230         $retval = true;
     234    if ( !empty( $post_id ) && ( bbp_get_topic_post_type() === get_post_type( $post_id ) ) ) {
     235        $retval = true;
     236    }
    231237
    232238    return (bool) apply_filters( 'bbp_is_topic', $retval, $post_id );
     
    251257
    252258    // Edit is not a single topic
    253     if ( bbp_is_topic_edit() )
     259    if ( bbp_is_topic_edit() ) {
    254260        return false;
     261    }
    255262
    256263    // Single and a match
    257     if ( is_singular( bbp_get_topic_post_type() ) || bbp_is_query_name( 'bbp_single_topic' ) )
    258         $retval = true;
     264    if ( is_singular( bbp_get_topic_post_type() ) || bbp_is_query_name( 'bbp_single_topic' ) ) {
     265        $retval = true;
     266    }
    259267
    260268    return (bool) apply_filters( 'bbp_is_single_topic', $retval );
     
    277285
    278286    // In topic archive
    279     if ( is_post_type_archive( bbp_get_topic_post_type() ) || bbp_is_query_name( 'bbp_topic_archive' ) )
    280         $retval = true;
     287    if ( is_post_type_archive( bbp_get_topic_post_type() ) || bbp_is_query_name( 'bbp_topic_archive' ) ) {
     288        $retval = true;
     289    }
    281290
    282291    return (bool) apply_filters( 'bbp_is_topic_archive', $retval );
     
    298307
    299308    // Check query
    300     if ( !empty( $wp_query->bbp_is_topic_edit ) && ( $wp_query->bbp_is_topic_edit === true ) )
     309    if ( !empty( $wp_query->bbp_is_topic_edit ) && ( $wp_query->bbp_is_topic_edit === true ) ) {
    301310        $retval = true;
    302311
    303312    // Editing in admin
    304     elseif ( is_admin() && ( 'post.php' === $pagenow ) && ( get_post_type() === bbp_get_topic_post_type() ) && ( !empty( $_GET['action'] ) && ( 'edit' === $_GET['action'] ) ) )
    305         $retval = true;
     313    } elseif ( is_admin() && ( 'post.php' === $pagenow ) && ( get_post_type() === bbp_get_topic_post_type() ) && ( !empty( $_GET['action'] ) && ( 'edit' === $_GET['action'] ) ) ) {
     314        $retval = true;
     315    }
    306316
    307317    return (bool) apply_filters( 'bbp_is_topic_edit', $retval );
     
    322332
    323333    // Check topic edit and GET params
    324     if ( bbp_is_topic_edit() && !empty( $_GET['action'] ) && ( 'merge' === $_GET['action'] ) )
     334    if ( bbp_is_topic_edit() && !empty( $_GET['action'] ) && ( 'merge' === $_GET['action'] ) ) {
    325335        return true;
     336    }
    326337
    327338    return (bool) apply_filters( 'bbp_is_topic_merge', $retval );
     
    342353
    343354    // Check topic edit and GET params
    344     if ( bbp_is_topic_edit() && !empty( $_GET['action'] ) && ( 'split' === $_GET['action'] ) )
    345         $retval = true;
     355    if ( bbp_is_topic_edit() && !empty( $_GET['action'] ) && ( 'split' === $_GET['action'] ) ) {
     356        $retval = true;
     357    }
    346358
    347359    return (bool) apply_filters( 'bbp_is_topic_split', $retval );
     
    358370
    359371    // Bail if topic-tags are off
    360     if ( ! bbp_allow_topic_tags() )
     372    if ( ! bbp_allow_topic_tags() ) {
    361373        return false;
     374    }
    362375
    363376    // Return false if editing a topic tag
    364     if ( bbp_is_topic_tag_edit() )
     377    if ( bbp_is_topic_tag_edit() ) {
    365378        return false;
     379    }
    366380
    367381    // Assume false
     
    369383
    370384    // Check tax and query vars
    371     if ( is_tax( bbp_get_topic_tag_tax_id() ) || !empty( bbpress()->topic_query->is_tax ) || get_query_var( 'bbp_topic_tag' ) )
    372         $retval = true;
     385    if ( is_tax( bbp_get_topic_tag_tax_id() ) || !empty( bbpress()->topic_query->is_tax ) || get_query_var( 'bbp_topic_tag' ) ) {
     386        $retval = true;
     387    }
    373388
    374389    return (bool) apply_filters( 'bbp_is_topic_tag', $retval );
     
    387402
    388403    // Bail if topic-tags are off
    389     if ( ! bbp_allow_topic_tags() )
     404    if ( ! bbp_allow_topic_tags() ) {
    390405        return false;
     406    }
    391407
    392408    // Assume false
     
    394410
    395411    // Check query
    396     if ( !empty( $wp_query->bbp_is_topic_tag_edit ) && ( true === $wp_query->bbp_is_topic_tag_edit ) )
     412    if ( !empty( $wp_query->bbp_is_topic_tag_edit ) && ( true === $wp_query->bbp_is_topic_tag_edit ) ) {
    397413        $retval = true;
    398414
    399415    // Editing in admin
    400     elseif ( is_admin() && ( 'edit-tags.php' === $pagenow ) && ( bbp_get_topic_tag_tax_id() === $taxnow ) && ( !empty( $_GET['action'] ) && ( 'edit' === $_GET['action'] ) ) )
    401         $retval = true;
     416    } elseif ( is_admin() && ( 'edit-tags.php' === $pagenow ) && ( bbp_get_topic_tag_tax_id() === $taxnow ) && ( !empty( $_GET['action'] ) && ( 'edit' === $_GET['action'] ) ) ) {
     417        $retval = true;
     418    }
    402419
    403420    return (bool) apply_filters( 'bbp_is_topic_tag_edit', $retval );
     
    427444        bbp_get_topic_post_type(),
    428445        bbp_get_reply_post_type()
    429     ) ) )
    430         $retval = true;
     446    ) ) ) {
     447        $retval = true;
     448    }
    431449
    432450    return (bool) apply_filters( 'bbp_is_custom_post_type', $retval, $the_post );
     
    449467
    450468    // Supplied ID is a reply
    451     if ( !empty( $post_id ) && ( bbp_get_reply_post_type() === get_post_type( $post_id ) ) )
    452         $retval = true;
     469    if ( !empty( $post_id ) && ( bbp_get_reply_post_type() === get_post_type( $post_id ) ) ) {
     470        $retval = true;
     471    }
    453472
    454473    return (bool) apply_filters( 'bbp_is_reply', $retval, $post_id );
     
    470489
    471490    // Check query
    472     if ( !empty( $wp_query->bbp_is_reply_edit ) && ( true === $wp_query->bbp_is_reply_edit ) )
     491    if ( !empty( $wp_query->bbp_is_reply_edit ) && ( true === $wp_query->bbp_is_reply_edit ) ) {
    473492        $retval = true;
    474493
    475494    // Editing in admin
    476     elseif ( is_admin() && ( 'post.php' === $pagenow ) && ( get_post_type() === bbp_get_reply_post_type() ) && ( !empty( $_GET['action'] ) && ( 'edit' === $_GET['action'] ) ) )
    477         $retval = true;
     495    } elseif ( is_admin() && ( 'post.php' === $pagenow ) && ( get_post_type() === bbp_get_reply_post_type() ) && ( !empty( $_GET['action'] ) && ( 'edit' === $_GET['action'] ) ) ) {
     496        $retval = true;
     497    }
    478498
    479499    return (bool) apply_filters( 'bbp_is_reply_edit', $retval );
     
    492512
    493513    // Check reply edit and GET params
    494     if ( bbp_is_reply_edit() && !empty( $_GET['action'] ) && ( 'move' === $_GET['action'] ) )
    495         $retval = true;
     514    if ( bbp_is_reply_edit() && !empty( $_GET['action'] ) && ( 'move' === $_GET['action'] ) ) {
     515        $retval = true;
     516    }
    496517
    497518    return (bool) apply_filters( 'bbp_is_reply_move', $retval );
     
    516537
    517538    // Edit is not a single reply
    518     if ( bbp_is_reply_edit() )
     539    if ( bbp_is_reply_edit() ) {
    519540        return false;
     541    }
    520542
    521543    // Single and a match
    522     if ( is_singular( bbp_get_reply_post_type() ) || ( bbp_is_query_name( 'bbp_single_reply' ) ) )
    523         $retval = true;
     544    if ( is_singular( bbp_get_reply_post_type() ) || ( bbp_is_query_name( 'bbp_single_reply' ) ) ) {
     545        $retval = true;
     546    }
    524547
    525548    return (bool) apply_filters( 'bbp_is_single_reply', $retval );
     
    540563
    541564    // Check query
    542     if ( !empty( $wp_query->bbp_is_single_user_favs ) && ( true === $wp_query->bbp_is_single_user_favs ) )
    543         $retval = true;
     565    if ( !empty( $wp_query->bbp_is_single_user_favs ) && ( true === $wp_query->bbp_is_single_user_favs ) ) {
     566        $retval = true;
     567    }
    544568
    545569    return (bool) apply_filters( 'bbp_is_favorites', $retval );
     
    560584
    561585    // Check query
    562     if ( !empty( $wp_query->bbp_is_single_user_subs ) && ( true === $wp_query->bbp_is_single_user_subs ) )
    563         $retval = true;
     586    if ( !empty( $wp_query->bbp_is_single_user_subs ) && ( true === $wp_query->bbp_is_single_user_subs ) ) {
     587        $retval = true;
     588    }
    564589
    565590    return (bool) apply_filters( 'bbp_is_subscriptions', $retval );
     
    581606
    582607    // Check query
    583     if ( !empty( $wp_query->bbp_is_single_user_topics ) && ( true === $wp_query->bbp_is_single_user_topics ) )
    584         $retval = true;
     608    if ( !empty( $wp_query->bbp_is_single_user_topics ) && ( true === $wp_query->bbp_is_single_user_topics ) ) {
     609        $retval = true;
     610    }
    585611
    586612    return (bool) apply_filters( 'bbp_is_topics_created', $retval );
     
    602628
    603629    // Check query
    604     if ( !empty( $wp_query->bbp_is_single_user_replies ) && ( true === $wp_query->bbp_is_single_user_replies ) )
    605         $retval = true;
     630    if ( !empty( $wp_query->bbp_is_single_user_replies ) && ( true === $wp_query->bbp_is_single_user_replies ) ) {
     631        $retval = true;
     632    }
    606633
    607634    return (bool) apply_filters( 'bbp_is_replies_created', $retval );
     
    625652
    626653    // Check query
    627     if ( !empty( $wp_query->bbp_is_single_user_home ) && ( true === $wp_query->bbp_is_single_user_home ) )
    628         $retval = true;
     654    if ( !empty( $wp_query->bbp_is_single_user_home ) && ( true === $wp_query->bbp_is_single_user_home ) ) {
     655        $retval = true;
     656    }
    629657
    630658    return (bool) apply_filters( 'bbp_is_user_home', $retval );
     
    646674    $retval = false;
    647675
    648     if ( bbp_is_user_home() && bbp_is_single_user_edit() )
    649         $retval = true;
     676    if ( bbp_is_user_home() && bbp_is_single_user_edit() ) {
     677        $retval = true;
     678    }
    650679
    651680    return (bool) apply_filters( 'bbp_is_user_home_edit', $retval );
     
    667696
    668697    // Check query
    669     if ( !empty( $wp_query->bbp_is_single_user ) && ( true === $wp_query->bbp_is_single_user ) )
    670         $retval = true;
     698    if ( !empty( $wp_query->bbp_is_single_user ) && ( true === $wp_query->bbp_is_single_user ) ) {
     699        $retval = true;
     700    }
    671701
    672702    return (bool) apply_filters( 'bbp_is_single_user', $retval );
     
    688718
    689719    // Check query
    690     if ( !empty( $wp_query->bbp_is_single_user_edit ) && ( true === $wp_query->bbp_is_single_user_edit ) )
    691         $retval = true;
     720    if ( !empty( $wp_query->bbp_is_single_user_edit ) && ( true === $wp_query->bbp_is_single_user_edit ) ) {
     721        $retval = true;
     722    }
    692723
    693724    return (bool) apply_filters( 'bbp_is_single_user_edit', $retval );
     
    709740
    710741    // Check query
    711     if ( !empty( $wp_query->bbp_is_single_user_profile ) && ( true === $wp_query->bbp_is_single_user_profile ) )
    712         $retval = true;
     742    if ( !empty( $wp_query->bbp_is_single_user_profile ) && ( true === $wp_query->bbp_is_single_user_profile ) ) {
     743        $retval = true;
     744    }
    713745
    714746    return (bool) apply_filters( 'bbp_is_single_user_profile', $retval );
     
    730762
    731763    // Check query
    732     if ( !empty( $wp_query->bbp_is_single_user_topics ) && ( true === $wp_query->bbp_is_single_user_topics ) )
    733         $retval = true;
     764    if ( !empty( $wp_query->bbp_is_single_user_topics ) && ( true === $wp_query->bbp_is_single_user_topics ) ) {
     765        $retval = true;
     766    }
    734767
    735768    return (bool) apply_filters( 'bbp_is_single_user_topics', $retval );
     
    751784
    752785    // Check query
    753     if ( !empty( $wp_query->bbp_is_single_user_replies ) && ( true === $wp_query->bbp_is_single_user_replies ) )
    754         $retval = true;
     786    if ( !empty( $wp_query->bbp_is_single_user_replies ) && ( true === $wp_query->bbp_is_single_user_replies ) ) {
     787        $retval = true;
     788    }
    755789
    756790    return (bool) apply_filters( 'bbp_is_single_user_replies', $retval );
     
    773807
    774808    // Check query
    775     if ( !empty( $wp_query->bbp_is_view ) && ( true === $wp_query->bbp_is_view ) )
    776         $retval = true;
     809    if ( !empty( $wp_query->bbp_is_view ) && ( true === $wp_query->bbp_is_view ) ) {
     810        $retval = true;
     811    }
    777812
    778813    // Check query name
    779     if ( empty( $retval ) && bbp_is_query_name( 'bbp_single_view' ) )
    780         $retval = true;
     814    if ( empty( $retval ) && bbp_is_query_name( 'bbp_single_view' ) ) {
     815        $retval = true;
     816    }
    781817
    782818    return (bool) apply_filters( 'bbp_is_single_view', $retval );
     
    796832
    797833    // Bail if search is disabled
    798     if ( ! bbp_allow_search() )
     834    if ( ! bbp_allow_search() ) {
    799835        return false;
     836    }
    800837
    801838    // Assume false
     
    803840
    804841    // Check query
    805     if ( !empty( $wp_query->bbp_is_search ) && ( true === $wp_query->bbp_is_search ) )
    806         $retval = true;
     842    if ( !empty( $wp_query->bbp_is_search ) && ( true === $wp_query->bbp_is_search ) ) {
     843        $retval = true;
     844    }
    807845
    808846    // Check query name
    809     if ( empty( $retval ) && bbp_is_query_name( bbp_get_search_rewrite_id() ) )
    810         $retval = true;
     847    if ( empty( $retval ) && bbp_is_query_name( bbp_get_search_rewrite_id() ) ) {
     848        $retval = true;
     849    }
    811850
    812851    // Check $_GET
    813     if ( empty( $retval ) && isset( $_REQUEST[ bbp_get_search_rewrite_id() ] ) && empty( $_REQUEST[ bbp_get_search_rewrite_id() ] ) )
    814         $retval = true;
     852    if ( empty( $retval ) && isset( $_REQUEST[ bbp_get_search_rewrite_id() ] ) && empty( $_REQUEST[ bbp_get_search_rewrite_id() ] ) ) {
     853        $retval = true;
     854    }
    815855
    816856    return (bool) apply_filters( 'bbp_is_search', $retval );
     
    830870
    831871    // Bail if search is disabled
    832     if ( ! bbp_allow_search() )
     872    if ( ! bbp_allow_search() ) {
    833873        return false;
     874    }
    834875
    835876    // Assume false
     
    837878
    838879    // Check query
    839     if ( !empty( $wp_query->bbp_search_terms ) )
    840         $retval = true;
     880    if ( !empty( $wp_query->bbp_search_terms ) ) {
     881        $retval = true;
     882    }
    841883
    842884    // Check query name
    843     if ( empty( $retval ) && bbp_is_query_name( 'bbp_search_results' ) )
    844         $retval = true;
     885    if ( empty( $retval ) && bbp_is_query_name( 'bbp_search_results' ) ) {
     886        $retval = true;
     887    }
    845888
    846889    // Check $_REQUEST
    847     if ( empty( $retval ) && !empty( $_REQUEST[ bbp_get_search_rewrite_id() ] ) )
    848         $retval = true;
     890    if ( empty( $retval ) && !empty( $_REQUEST[ bbp_get_search_rewrite_id() ] ) ) {
     891        $retval = true;
     892    }
    849893
    850894    return (bool) apply_filters( 'bbp_is_search_results', $retval );
     
    866910
    867911    // Check query
    868     if ( !empty( $wp_query->bbp_is_edit ) && ( $wp_query->bbp_is_edit === true ) )
    869         $retval = true;
     912    if ( !empty( $wp_query->bbp_is_edit ) && ( $wp_query->bbp_is_edit === true ) ) {
     913        $retval = true;
     914    }
    870915
    871916    return (bool) apply_filters( 'bbp_is_edit', $retval );
     
    11451190 * @since bbPress (r2815)
    11461191 *
    1147  * @param string $url Pass a URL to redirect to
     1192 * @param mixed $args This function supports these arguments:
     1193 *  - action: The action being taken
     1194 *  - context: The context the action is being taken from
    11481195 * @uses add_query_arg() To add a arg to the url
    11491196 * @uses site_url() Toget the site url
     
    12391286
    12401287        // Check that requested
    1241         if ( empty( $_REQUEST[$request] ) )
     1288        if ( empty( $_REQUEST[ $request ] ) ) {
    12421289            return false;
     1290        }
    12431291
    12441292        // Set request varaible
     
    12971345        $bbp = bbpress();
    12981346
    1299         if ( true === $auto_increment )
     1347        if ( true === $auto_increment ) {
    13001348            ++$bbp->tab_index;
     1349        }
    13011350
    13021351        return apply_filters( 'bbp_get_tab_index', (int) $bbp->tab_index );
     
    15071556        <input type="hidden" name="bbp_forum_id" id="bbp_forum_id"    value="<?php bbp_forum_id(); ?>" />
    15081557
    1509         <?php
    1510 
    1511         if ( current_user_can( 'unfiltered_html' ) )
     1558        <?php if ( current_user_can( 'unfiltered_html' ) ) :
    15121559            wp_nonce_field( 'bbp-unfiltered-html-forum_' . bbp_get_forum_id(), '_bbp_unfiltered_html_forum', false );
    1513 
    1514         ?>
     1560        endif; ?>
    15151561
    15161562        <?php wp_nonce_field( 'bbp-edit-forum_' . bbp_get_forum_id() );
     
    15261572        <input type="hidden" name="action" id="bbp_post_action" value="bbp-new-forum" />
    15271573
    1528         <?php
    1529 
    1530         if ( current_user_can( 'unfiltered_html' ) )
     1574        <?php if ( current_user_can( 'unfiltered_html' ) ) :
    15311575            wp_nonce_field( 'bbp-unfiltered-html-forum_new', '_bbp_unfiltered_html_forum', false );
    1532 
    1533         ?>
     1576        endif; ?>
    15341577
    15351578        <?php wp_nonce_field( 'bbp-new-forum' );
     
    15561599        <input type="hidden" name="bbp_topic_id" id="bbp_topic_id"    value="<?php bbp_topic_id(); ?>" />
    15571600
    1558         <?php
    1559 
    1560         if ( current_user_can( 'unfiltered_html' ) )
     1601        <?php if ( current_user_can( 'unfiltered_html' ) ) :
    15611602            wp_nonce_field( 'bbp-unfiltered-html-topic_' . bbp_get_topic_id(), '_bbp_unfiltered_html_topic', false );
    1562 
    1563         ?>
     1603        endif; ?>
    15641604
    15651605        <?php wp_nonce_field( 'bbp-edit-topic_' . bbp_get_topic_id() );
     
    15751615        <input type="hidden" name="action" id="bbp_post_action" value="bbp-new-topic" />
    15761616
    1577         <?php if ( current_user_can( 'unfiltered_html' ) )
    1578             wp_nonce_field( 'bbp-unfiltered-html-topic_new', '_bbp_unfiltered_html_topic', false ); ?>
     1617        <?php if ( current_user_can( 'unfiltered_html' ) ) :
     1618            wp_nonce_field( 'bbp-unfiltered-html-topic_new', '_bbp_unfiltered_html_topic', false );
     1619        endif; ?>
    15791620
    15801621        <?php wp_nonce_field( 'bbp-new-topic' );
     
    16011642        <input type="hidden" name="action"          id="bbp_post_action" value="bbp-edit-reply" />
    16021643
    1603         <?php if ( current_user_can( 'unfiltered_html' ) )
    1604             wp_nonce_field( 'bbp-unfiltered-html-reply_' . bbp_get_reply_id(), '_bbp_unfiltered_html_reply', false ); ?>
     1644        <?php if ( current_user_can( 'unfiltered_html' ) ) :
     1645            wp_nonce_field( 'bbp-unfiltered-html-reply_' . bbp_get_reply_id(), '_bbp_unfiltered_html_reply', false );
     1646        endif; ?>
    16051647
    16061648        <?php wp_nonce_field( 'bbp-edit-reply_' . bbp_get_reply_id() );
     
    16121654        <input type="hidden" name="action"          id="bbp_post_action" value="bbp-new-reply" />
    16131655
    1614         <?php if ( current_user_can( 'unfiltered_html' ) )
    1615             wp_nonce_field( 'bbp-unfiltered-html-reply_' . bbp_get_topic_id(), '_bbp_unfiltered_html_reply', false ); ?>
     1656        <?php if ( current_user_can( 'unfiltered_html' ) ) :
     1657            wp_nonce_field( 'bbp-unfiltered-html-reply_' . bbp_get_topic_id(), '_bbp_unfiltered_html_reply', false );
     1658        endif; ?>
    16161659
    16171660        <?php wp_nonce_field( 'bbp-new-reply' );
     
    16201663        if ( bbp_is_query_name( 'bbp_single_topic' ) ) :
    16211664            bbp_redirect_to_field( get_permalink() );
    1622 
    16231665        endif;
    16241666    endif;
     
    18651907 * @since bbPress (r4606)
    18661908 *
    1867  * @param array $buttons
     1909 * @param array $settings
    18681910 * @see quicktags_settings
    18691911 * @return array Quicktags settings
     
    19602002
    19612003        $view = bbp_get_view_id( $view );
    1962         if ( empty( $view ) )
     2004        if ( empty( $view ) ) {
    19632005            return false;
    1964 
    1965         return $bbp->views[$view]['title'];
     2006        }
     2007
     2008        return $bbp->views[ $view ]['title'];
    19662009    }
    19672010
     
    19942037
    19952038        $view = bbp_get_view_id( $view );
    1996         if ( empty( $view ) )
     2039        if ( empty( $view ) ) {
    19972040            return home_url();
     2041        }
    19982042
    19992043        // Pretty permalinks
     
    21152159
    21162160        // Turn off breadcrumbs
    2117         if ( apply_filters( 'bbp_no_breadcrumb', is_front_page() ) )
     2161        if ( apply_filters( 'bbp_no_breadcrumb', is_front_page() ) ) {
    21182162            return;
     2163        }
    21192164
    21202165        // Define variables
     
    22952340
    22962341                // Skip parent if empty or error
    2297                 if ( empty( $parent ) || is_wp_error( $parent ) )
     2342                if ( empty( $parent ) || is_wp_error( $parent ) ) {
    22982343                    continue;
     2344                }
    22992345
    23002346                // Switch through post_type to ensure correct filters are applied
     
    24272473
    24282474    // Bail if no notices or errors
    2429     if ( !bbp_has_errors() )
     2475    if ( !bbp_has_errors() ) {
    24302476        return;
     2477    }
    24312478
    24322479    // Define local variable(s)
     
    26702717
    26712718    // Compare new title with original title
    2672     if ( $new_title === $title )
     2719    if ( $new_title === $title ) {
    26732720        return $title;
     2721    }
    26742722
    26752723    // Temporary separator, for accurate flipping, if necessary
     
    26772725    $prefix = '';
    26782726
    2679     if ( !empty( $new_title ) )
     2727    if ( !empty( $new_title ) ) {
    26802728        $prefix = " $sep ";
     2729    }
    26812730
    26822731    // sep on right, so reverse the order
  • trunk/src/includes/common/widgets.php

    r5297 r5437  
    410410
    411411        // Bail if search is disabled
    412         if ( ! bbp_allow_search() )
     412        if ( ! bbp_allow_search() ) {
    413413            return;
     414        }
    414415
    415416        // Get widget settings
Note: See TracChangeset for help on using the changeset viewer.