Skip to:
Content

bbPress.org

Changeset 7379


Ignore:
Timestamp:
11/24/2025 07:23:06 PM (4 weeks ago)
Author:
johnjamesjacoby
Message:

Tools - Code Improvement: stop ignoring these PHPCS sniffs:

  • PEAR.Files.IncludingFile.UseRequire
  • PSR2.ControlStructures.SwitchDeclaration.BodyOnNextLineDEFAULT
  • PSR2.Methods.FunctionClosingBrace.SpacingBeforeClose
  • Squiz.ControlStructures.ControlSignature.NewlineAfterOpenBrace
  • Squiz.PHP.DisallowMultipleAssignments.FoundInControlStructure
  • Squiz.PHP.EmbeddedPhp.MultipleStatements
  • Squiz.PHP.NonExecutableCode.Unreachable
  • Universal.Arrays.DuplicateArrayKey.Found
  • Universal.ControlStructures.DisallowLonelyIf.Found
  • WordPress.PHP.NoSilencedErrors.Discouraged

This commit includes code formatting changes for the above sniffs.

See #3658.

Location:
trunk
Files:
34 edited

Legend:

Unmodified
Added
Removed

  • trunk/phpcs.xml.dist

    r7378 r7379  
    218218    </rule>
    219219
     220    <rule ref="PEAR.Files.IncludingFile.UseRequire">
     221        <exclude-pattern>/bbpress\.php$</exclude-pattern>
     222        <exclude-pattern>/src/*</exclude-pattern>
     223    </rule>
     224
    220225    <rule ref="Squiz.PHP.EmbeddedPhp.ContentBeforeEnd">
    221226        <exclude-pattern>/src/*</exclude-pattern>
    … …  
    224229    <rule ref="Squiz.PHP.EmbeddedPhp.ContentBeforeOpen">
    225230        <exclude-pattern>/src/*</exclude-pattern>
     231    </rule>
     232
     233    <rule ref="Squiz.ControlStructures.ControlSignature.NewlineAfterOpenBrace">
     234        <exclude-pattern>/src/templates/*</exclude-pattern>
    226235    </rule>
    227236
    … …  
    261270    </rule>
    262271
    263     <rule ref="PEAR.Files.IncludingFile.UseRequire">
    264         <exclude-pattern>/bbpress\.php$</exclude-pattern>
    265         <exclude-pattern>/src/*</exclude-pattern>
    266     </rule>
    267 
    268272    <rule ref="PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket">
    269273        <exclude-pattern>/src/*</exclude-pattern>
    … …  
    294298    </rule>
    295299
    296     <rule ref="PSR2.ControlStructures.SwitchDeclaration.BodyOnNextLineDEFAULT">
    297         <exclude-pattern>/src/*</exclude-pattern>
    298     </rule>
    299 
    300300    <rule ref="PSR2.ControlStructures.SwitchDeclaration.SpaceBeforeColonCASE">
    301301        <exclude-pattern>/src/*</exclude-pattern>
    … …  
    306306    </rule>
    307307
    308     <rule ref="PSR2.Methods.FunctionClosingBrace.SpacingBeforeClose">
    309         <exclude-pattern>/src/*</exclude-pattern>
    310     </rule>
    311 
    312     <rule ref="Squiz.ControlStructures.ControlSignature.NewlineAfterOpenBrace">
    313         <exclude-pattern>/src/*</exclude-pattern>
    314     </rule>
    315 
    316308    <rule ref="Squiz.PHP.CommentedOutCode.Found">
    317309        <exclude-pattern>/src/*</exclude-pattern>
    … …  
    322314    </rule>
    323315
    324     <rule ref="Squiz.PHP.DisallowMultipleAssignments.FoundInControlStructure">
    325         <exclude-pattern>/src/*</exclude-pattern>
    326     </rule>
    327 
    328316    <rule ref="Squiz.PHP.EmbeddedPhp.ContentAfterEnd">
    329317        <exclude-pattern>/src/*</exclude-pattern>
    … …  
    334322    </rule>
    335323
    336     <rule ref="Squiz.PHP.EmbeddedPhp.MultipleStatements">
    337         <exclude-pattern>/src/*</exclude-pattern>
    338     </rule>
    339 
    340324    <rule ref="Squiz.PHP.EmbeddedPhp.SpacingAfterOpen">
    341325        <exclude-pattern>/src/*</exclude-pattern>
    342326    </rule>
    343327
    344     <rule ref="Squiz.PHP.NonExecutableCode.Unreachable">
    345         <exclude-pattern>/src/*</exclude-pattern>
    346     </rule>
    347 
    348328    <rule ref="Squiz.Strings.ConcatenationSpacing.PaddingFound">
    349329        <exclude-pattern>/src/*</exclude-pattern>
    350330    </rule>
    351331
    352     <rule ref="Universal.Arrays.DuplicateArrayKey.Found">
    353         <exclude-pattern>/src/*</exclude-pattern>
    354     </rule>
    355 
    356     <rule ref="Universal.ControlStructures.DisallowLonelyIf.Found">
    357         <exclude-pattern>/src/*</exclude-pattern>
    358     </rule>
    359 
    360332    <rule ref="Universal.Files.SeparateFunctionsFromOO.Mixed">
    361333        <exclude-pattern>/src/*</exclude-pattern>
    … …  
    398370    </rule>
    399371
    400     <rule ref="WordPress.PHP.NoSilencedErrors.Discouraged">
    401         <exclude-pattern>/src/*</exclude-pattern>
    402     </rule>
    403 
    404372    <rule ref="WordPress.Security.EscapeOutput.OutputNotEscaped">
    405373        <exclude-pattern>/src/*</exclude-pattern>
    406374    </rule>
    407375
    408     <rule ref="WordPress.Security.EscapeOutput.UnsafePrintingFunction">
    409         <exclude-pattern>/src/*</exclude-pattern>
    410     </rule>
    411 
    412376    <rule ref="WordPress.Security.NonceVerification.Missing">
    413377        <exclude-pattern>/src/*</exclude-pattern>

  • trunk/src/includes/admin/classes/class-bbp-admin.php

    r7378 r7379  
    14341434
    14351435            case 'show' :
    1436             default : ?>
     1436            default :
     1437            ?>
    14371438
    14381439                <p><?php esc_html_e( 'You can update your forum through this page. Hit the link below to update.', 'bbpress' ); ?></p>
    … …  
    15631564
    15641565            case 'show' :
    1565             default : ?>
    1566 
     1566            default :
     1567            ?>
    15671568                <p><?php esc_html_e( 'You can update all the forums on your network through this page. It works by calling the update script of each site automatically. Hit the link below to update.', 'bbpress' ); ?></p>
    15681569                <p><a class="button" href="update-core.php?page=bbpress-update&amp;action=bbpress-update"><?php esc_html_e( 'Update Forums', 'bbpress' ); ?></a></p>

  • trunk/src/includes/admin/classes/class-bbp-converter.php

    r7378 r7379  
    270270
    271271        // Get disabled PHP functions (to avoid using them)
    272         $disabled = explode( ',', @ini_get( 'disable_functions' ) );
     272        $disabled = explode( ',', @ini_get( 'disable_functions' ) ); // phpcs:ignore
    273273
    274274        // Maybe avoid terminating when the client goes away (if function is not disabled)
    275275        if ( ! in_array( 'ignore_user_abort', $disabled, true ) ) {
    276             @ignore_user_abort( true );
     276            @ignore_user_abort( true ); // phpcs:ignore
    277277        }
    278278
    … …  
    280280        if ( ! in_array( 'ini_set', $disabled, true ) ) {
    281281            foreach ( $r as $key => $value ) {
    282                 // phpcs:ignore WordPress.PHP.IniSet.Risky
    283                 @ini_set( $key, $value );
     282                @ini_set( $key, $value ); // phpcs:ignore
    284283            }
    285284        }

  • trunk/src/includes/admin/classes/class-bbp-topic-replies-list-table.php

    r7378 r7379  
    8989     */
    9090    public function get_bulk_actions() {
    91         return array();
    92 
    93         // @todo cap checks
    94         return array(
     91
     92        // Default bulk actions
     93        $retval = array(
    9594            'unapprove' => esc_html__( 'Unapprove', 'bbpress' ),
    9695            'spam'      => esc_html__( 'Spam',      'bbpress' ),
    9796            'trash'     => esc_html__( 'Trash',     'bbpress' )
    9897        );
     98
     99        // Override to empty
     100        $retval = array();
     101
     102        // Return
     103        return $retval;
    99104    }
    100105

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

    r7378 r7379  
    554554            'to_fieldname'   => '_bbp_aef_user_private_text'
    555555        );
    556 
    557556    }
    558557

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

    r7378 r7379  
    576576            'to_fieldname'   => '_bbp_fluxbb_user_admin_note'
    577577        );
    578 
    579578    }
    580579

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

    r7378 r7379  
    460460            'to_fieldname'   => 'display_name'
    461461        );
    462 
    463462    }
    464463

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

    r7378 r7379  
    500500            'callback_method' => 'callback_html'
    501501        );
    502 
    503502    }
    504503

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

    r7378 r7379  
    490490            'to_fieldname'   => 'display_name'
    491491        );
    492 
    493492    }
    494493

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

    r7378 r7379  
    502502            'callback_method' => 'callback_html'
    503503        );
    504 
    505504    }
    506505

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

    r7378 r7379  
    609609            'to_fieldname'   => '_bbp_xmb_user_mood'
    610610        );
    611 
    612611    }
    613612

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

    r7378 r7379  
    586586            if ( ! is_null( $row ) ) {
    587587                $this->map_userid[ $field ] = $row->value_id;
     588            } elseif ( true === $this->convert_users ) {
     589                $this->map_userid[ $field ] = 0;
    588590            } else {
    589                 if ( true === $this->convert_users ) {
    590                     $this->map_userid[ $field ] = 0;
    591                 } else {
    592                     $this->map_userid[ $field ] = $field;
    593                 }
     591                $this->map_userid[ $field ] = $field;
    594592            }
    595593        }

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

    r7378 r7379  
    987987
    988988        // Bail if not viewing spam
    989         if ( empty( $_GET['post_status'] ) || ( bbp_get_spam_status_id() !== $_GET['post_status'] ) && current_user_can( 'moderate' ) ) {
     989        if (
     990            empty( $_GET['post_status'] )
     991            ||
     992            (
     993                ( bbp_get_spam_status_id() !== $_GET['post_status'] )
     994                &&
     995                current_user_can( 'moderate' )
     996            )
     997        ) {
    990998            return;
    991999        }

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

    r7378 r7379  
    16021602?>
    16031603
    1604     <p><?php _e( 'Information about the database for your previous forums so they can be converted.', 'bbpress' ); ?></p>
     1604    <p><?php esc_html_e( 'Information about the database for your previous forums so they can be converted.', 'bbpress' ); ?></p>
    16051605
    16061606<?php

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

    r7378 r7379  
    9696
    9797        // Try to include the converter
     98        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    9899        @include_once $converter_file;
    99100

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

    r7378 r7379  
    11811181
    11821182        // Bail if not viewing spam
    1183         if ( empty( $_GET['post_status'] ) || ( bbp_get_spam_status_id() !== $_GET['post_status'] ) && current_user_can( 'moderate' ) ) {
     1183        if (
     1184            empty( $_GET['post_status'] )
     1185            ||
     1186            (
     1187                ( bbp_get_spam_status_id() !== $_GET['post_status'] )
     1188                &&
     1189                current_user_can( 'moderate' )
     1190            )
     1191        ) {
    11841192            return;
    11851193        }

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

    r7361 r7379  
    122122
    123123    // Set the header content type
    124     @header( 'Content-Type: ' . get_option( 'html_type' ) . '; charset=' . get_option( 'blog_charset' ) );
    125     @header( 'X-Robots-Tag: noindex' );
     124    header( 'Content-Type: ' . get_option( 'html_type' ) . '; charset=' . get_option( 'blog_charset' ) );
     125    header( 'X-Robots-Tag: noindex' );
    126126
    127127    // Disable content sniffing in browsers that support it
    … …  
    161161
    162162    // Send back the JSON
    163     @header( 'Content-type: application/json' );
     163    header( 'Content-type: application/json' );
    164164    echo json_encode( $response );
    165165    die();

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

    r7378 r7379  
    5959     */
    6060    public function add_user_to_object( $object_id = 0, $user_id = 0, $meta_key = '', $meta_type = 'post', $unique = false ) {
    61 
     61        // Intentionally empty
    6262    }
    6363
    … …  
    7575     */
    7676    public function remove_user_from_object( $object_id = 0, $user_id = 0, $meta_key = '', $meta_type = 'post' ) {
    77 
     77        // Intentionally empty
    7878    }
    7979
    … …  
    9090     */
    9191    public function remove_user_from_all_objects( $user_id = 0, $meta_key = '', $meta_type = 'post' ) {
    92 
     92        // Intentionally empty
    9393    }
    9494
    … …  
    106106     */
    107107    public function remove_object_from_all_users( $object_id = 0, $meta_key = '', $meta_type = 'post' ) {
    108 
     108        // Intentionally empty
    109109    }
    110110
    … …  
    120120     */
    121121    public function remove_all_users_from_all_objects( $meta_key = '', $meta_type = 'post' ) {
    122 
     122        // Intentionally empty
    123123    }
    124124
    … …  
    135135     */
    136136    public function get_users_for_object( $object_id = 0, $meta_key = '', $meta_type = 'post' ) {
    137 
     137        // Intentionally empty
    138138    }
    139139
    … …  
    150150     */
    151151    public function get_query( $args = array(), $context_key = '', $meta_key = '', $meta_type = 'post' ) {
    152 
     152        // Intentionally empty
    153153    }
    154154}

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

    r7378 r7379  
    399399            $ret = apply_filters( 'bbp_make_clickable', $ret, $text );
    400400            $ret = substr( $ret, 1, -1 ); // Remove our whitespace padding.
    401             $r .= $ret;
     401            $r  .= $ret;
    402402        }
    403403    }

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

    r7378 r7379  
    17171717 * @global WP $wp
    17181718 * @param string $where
    1719  * @param WP_Query $object
     1719 * @param WP_Query $query
    17201720 * @return string
    17211721 */
    1722 function bbp_query_post_parent__in( $where, $object = '' ) {
     1722function bbp_query_post_parent__in( $where, $query = '' ) {
    17231723    global $wp;
    17241724
    … …  
    17291729
    17301730    // Bail if no object passed
    1731     if ( empty( $object ) ) {
     1731    if ( empty( $query ) ) {
    17321732        return $where;
    17331733    }
    17341734
    17351735    // Only 1 post_parent so return $where
    1736     if ( is_numeric( $object->query_vars['post_parent'] ) ) {
     1736    if ( is_numeric( $query->query_vars['post_parent'] ) ) {
    17371737        return $where;
    17381738    }
    … …  
    17421742
    17431743    // Including specific post_parent's
    1744     if ( ! empty( $object->query_vars['post_parent__in'] ) ) {
    1745         $ids    = implode( ',', wp_parse_id_list( $object->query_vars['post_parent__in'] ) );
     1744    if ( ! empty( $query->query_vars['post_parent__in'] ) ) {
     1745        $ids    = implode( ',', wp_parse_id_list( $query->query_vars['post_parent__in'] ) );
    17461746        $where .= " AND {$bbp_db->posts}.post_parent IN ($ids)";
    17471747
    17481748    // Excluding specific post_parent's
    1749     } elseif ( ! empty( $object->query_vars['post_parent__not_in'] ) ) {
    1750         $ids    = implode( ',', wp_parse_id_list( $object->query_vars['post_parent__not_in'] ) );
     1749    } elseif ( ! empty( $query->query_vars['post_parent__not_in'] ) ) {
     1750        $ids    = implode( ',', wp_parse_id_list( $query->query_vars['post_parent__not_in'] ) );
    17511751        $where .= " AND {$bbp_db->posts}.post_parent NOT IN ($ids)";
    17521752    }

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

    r7378 r7379  
    2424
    2525    // Bail if no post
    26     if ( ! $post = get_post( $post_id ) ) {
     26    $post = get_post( $post_id );
     27    if ( empty( $post ) ) {
    2728        return false;
    2829    }
    2930
    3031    // Bail if no lock
    31     if ( ! $lock = get_post_meta( $post->ID, '_edit_lock', true ) ) {
     32    $lock = get_post_meta( $post->ID, '_edit_lock', true );
     33    if ( empty( $lock ) ) {
    3234        return false;
    3335    }
    … …  
    6365
    6466    // Bail if no post
    65     if ( ! $post = get_post( $post_id ) ) {
     67    $post = get_post( $post_id );
     68    if ( empty( $post ) ) {
    6669        return false;
    6770    }
    6871
    6972    // Bail if no user
    70     if ( 0 === ( $user_id = get_current_user_id() ) ) {
     73    $user_id = get_current_user_id();
     74    if ( empty( $user_id ) ) {
    7175        return false;
    7276    }

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

    r7378 r7379  
    27892789
    27902790        // User "home"
     2791        // phpcs:ignore Universal.ControlStructures.DisallowLonelyIf.Found
    27912792        } else {
    27922793            if ( true === $is_user_home ) {

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

    r7378 r7379  
    571571        <ul class="bbp-forums-widget">
    572572
    573             <?php while ( $widget_query->have_posts() ) : $widget_query->the_post(); ?>
     573            <?php while ( $widget_query->have_posts() ) :
     574
     575                $widget_query->the_post(); ?>
    574576
    575577                <li <?php echo ( bbp_get_forum_id() === $widget_query->post->ID ? ' class="bbp-forum-widget-current-forum"' : '' ); ?>>
    … …  
    11861188        <ul class="bbp-replies-widget">
    11871189
    1188             <?php while ( $widget_query->have_posts() ) : $widget_query->the_post(); ?>
     1190            <?php while ( $widget_query->have_posts() ) :
     1191
     1192                $widget_query->the_post(); ?>
    11891193
    11901194                <li>

  • trunk/src/includes/core/template-loader.php

    r7378 r7379  
    2929function bbp_template_include_theme_supports( $template = '' ) {
    3030
    31     // phpcs:disable Generic.CodeAnalysis.EmptyStatement.DetectedElseif
     31    // phpcs:disable
    3232
    3333    // Editing a user

  • trunk/src/includes/extend/akismet.php

    r7378 r7379  
    863863
    864864        // Maybe HTTPS if not disabled
    865         if ( empty( $ssl_disabled_time ) && ( $is_ssl = wp_http_supports( array( 'ssl' ) ) ) ) {
    866             $akismet_url = set_url_scheme( $akismet_url, 'https' );
     865        if ( empty( $ssl_disabled_time ) ) {
     866            $is_ssl = wp_http_supports( array( 'ssl' ) );
     867
     868            // Use SSL
     869            if ( ! empty( $is_ssl ) ) {
     870                $akismet_url = set_url_scheme( $akismet_url, 'https' );
     871            }
    867872        }
    868873

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

    r7378 r7379  
    834834
    835835    // General component (bbpress/forums/other)
     836    // phpcs:ignore Universal.ControlStructures.DisallowLonelyIf.Found
    836837    } else {
    837838        if ( 'topic' === $type ) {

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

    r7378 r7379  
    17321732     * @since 2.6.14
    17331733     *
    1734      * @param  object object  Verified object
    1735      * @param  string $type   Type of variable to check with `is_a()`
    1736      * @return mixed  $object Verified object if valid, Default or null if invalid
    1737      */
    1738     public function rewrite_pagination( $object, $type = '' ) {
     1734     * @param  object $query  Verified query object
     1735     * @param  string $type   Type of variable to compare to
     1736     * @return mixed  Verified object if valid, Default or null if invalid
     1737     */
     1738    public function rewrite_pagination( $query, $type = '' ) {
    17391739
    17401740        // Bail if wrong global
    17411741        if ( 'wp_query' !== $type ) {
    1742             return $object;
     1742            return $query;
    17431743        }
    17441744
    17451745        // Bail if not inside a BuddyPress Group
    17461746        if ( ! bp_is_group() ) {
    1747             return $object;
     1747            return $query;
    17481748        }
    17491749
    17501750        // Bail if not inside a BuddyPress Group Forum
    17511751        if ( ! bp_is_current_action( 'forum' ) ) {
    1752             return $object;
     1752            return $query;
    17531753        }
    17541754
    … …  
    17671767            }
    17681768
    1769         // Single Forum
    1770         } else {
    1771 
    1772             // Get the page number from 1st position
    1773             if ( bp_is_action_variable( 'page', 0 ) ) {
    1774                 $page_number = bp_action_variable( 1 );
    1775             }
     1769        // Default (Single Forum)
     1770        } elseif ( bp_is_action_variable( 'page', 0 ) ) {
     1771            $page_number = bp_action_variable( 1 );
    17761772        }
    17771773
    17781774        // Bail if no page number
    17791775        if ( empty( $page_number ) ) {
    1780             return $object;
     1776            return $query;
    17811777        }
    17821778
    17831779        // Set the 'paged' WP_Query var to the new action-based value
    1784         $object->set( 'paged', $page_number );
     1780        $query->set( 'paged', $page_number );
    17851781
    17861782        // Return the filtered/modified object
    1787         return $object;
     1783        return $query;
    17881784    }
    17891785

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

    r7378 r7379  
    9595        // Require files if they exist
    9696        foreach ( $includes as $file ) {
     97
     98            // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
    9799            if ( @is_file( $this->path . $file ) ) {
    98100                require $this->path . $file;

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

    r7378 r7379  
    10421042    $hidden_forums = new WP_Query(
    10431043        array(
    1044             'fields'           => 'ids',
    1045             'suppress_filters' => true,
    1046             'post_type'        => bbp_get_forum_post_type(),
    1047             'post_status'      => bbp_get_hidden_status_id(),
    1048             'posts_per_page'   => -1,
     1044            'fields'         => 'ids',
     1045            'post_type'      => bbp_get_forum_post_type(),
     1046            'post_status'    => bbp_get_hidden_status_id(),
     1047            'posts_per_page' => -1,
    10491048
    10501049            // Performance

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

    r7378 r7379  
    24982498            $r['selected'] = sanitize_key( $_POST[ $r['select_id'] ] );
    24992499
    2500         // No Post value was passed
     2500        // Edit topic
     2501        } elseif ( bbp_is_forum_edit() ) {
     2502            $r['forum_id'] = bbp_get_forum_id( $r['forum_id'] );
     2503            $r['selected'] = bbp_get_forum_type( $r['forum_id'] );
     2504
     2505        // New topic
    25012506        } else {
    2502 
    2503             // Edit topic
    2504             if ( bbp_is_forum_edit() ) {
    2505                 $r['forum_id'] = bbp_get_forum_id( $r['forum_id'] );
    2506                 $r['selected'] = bbp_get_forum_type( $r['forum_id'] );
    2507 
    2508             // New topic
    2509             } else {
    2510                 $r['selected'] = bbp_get_public_status_id();
    2511             }
     2507            $r['selected'] = bbp_get_public_status_id();
    25122508        }
    25132509    }
    … …  
    25892585            $r['selected'] = sanitize_key( $_POST[ $r['select_id'] ] );
    25902586
    2591         // No Post value was passed
     2587        // Edit topic
     2588        } elseif ( bbp_is_forum_edit() ) {
     2589            $r['forum_id'] = bbp_get_forum_id( $r['forum_id'] );
     2590            $r['selected'] = bbp_get_forum_status( $r['forum_id'] );
     2591
     2592        // New topic
    25922593        } else {
    2593 
    2594             // Edit topic
    2595             if ( bbp_is_forum_edit() ) {
    2596                 $r['forum_id'] = bbp_get_forum_id( $r['forum_id'] );
    2597                 $r['selected'] = bbp_get_forum_status( $r['forum_id'] );
    2598 
    2599             // New topic
    2600             } else {
    2601                 $r['selected'] = bbp_get_public_status_id();
    2602             }
     2594            $r['selected'] = bbp_get_public_status_id();
    26032595        }
    26042596    }
    … …  
    26802672            $r['selected'] = sanitize_key( $_POST[ $r['select_id'] ] );
    26812673
    2682         // No Post value was passed
     2674        // Edit topic
     2675        } elseif ( bbp_is_forum_edit() ) {
     2676            $r['forum_id'] = bbp_get_forum_id( $r['forum_id'] );
     2677            $r['selected'] = bbp_get_forum_visibility( $r['forum_id'] );
     2678
     2679        // New topic
    26832680        } else {
    2684 
    2685             // Edit topic
    2686             if ( bbp_is_forum_edit() ) {
    2687                 $r['forum_id'] = bbp_get_forum_id( $r['forum_id'] );
    2688                 $r['selected'] = bbp_get_forum_visibility( $r['forum_id'] );
    2689 
    2690             // New topic
    2691             } else {
    2692                 $r['selected'] = bbp_get_public_status_id();
    2693             }
     2681            $r['selected'] = bbp_get_public_status_id();
    26942682        }
    26952683    }

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

    r7378 r7379  
    571571        return;
    572572
    573     // Reply exists
     573    // Check users ability to create new reply
     574    } elseif ( ! bbp_is_reply_anonymous( $reply_id ) ) {
     575
     576        // User cannot edit this reply
     577        if ( ! current_user_can( 'edit_reply', $reply_id ) ) {
     578            bbp_add_error( 'bbp_edit_reply_permission', __( '<strong>Error</strong>: You do not have permission to edit that reply.', 'bbpress' ) );
     579            return;
     580        }
     581
     582        // Set reply author
     583        $reply_author = bbp_get_reply_author_id( $reply_id );
     584
     585    // It is an anonymous post
    574586    } else {
    575587
    576         // Check users ability to create new reply
    577         if ( ! bbp_is_reply_anonymous( $reply_id ) ) {
    578 
    579             // User cannot edit this reply
    580             if ( ! current_user_can( 'edit_reply', $reply_id ) ) {
    581                 bbp_add_error( 'bbp_edit_reply_permission', __( '<strong>Error</strong>: You do not have permission to edit that reply.', 'bbpress' ) );
    582                 return;
    583             }
    584 
    585             // Set reply author
    586             $reply_author = bbp_get_reply_author_id( $reply_id );
    587 
    588         // It is an anonymous post
    589         } else {
    590 
    591             // Filter anonymous data
    592             $anonymous_data = bbp_filter_anonymous_post_data();
    593         }
     588        // Filter anonymous data
     589        $anonymous_data = bbp_filter_anonymous_post_data();
    594590    }
    595591
    … …  
    14001396            case 'topic' :
    14011397            default :
    1402 
    14031398                // User needs to be able to publish topics
    14041399                if ( current_user_can( 'publish_topics' ) ) {
    … …  
    23202315
    23212316        <?php if ( bbp_has_replies( $replies_query ) ) : ?>
    2322             <?php while ( bbp_replies() ) : bbp_the_reply(); ?>
     2317            <?php while ( bbp_replies() ) :
     2318
     2319                bbp_the_reply(); ?>
    23232320
    23242321                <item>

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

    r7378 r7379  
    497497
    498498    // Include pagination
     499    // phpcs:ignore Universal.ControlStructures.DisallowLonelyIf.Found
    499500    } else {
    500501
    … …  
    25462547
    25472548    // We are including the lead topic
     2549    // phpcs:ignore Universal.ControlStructures.DisallowLonelyIf.Found
    25482550    } else {
    25492551
    … …  
    28662868            $r['selected'] = sanitize_key( $_POST[ $r['select_id'] ] );
    28672869
    2868         // No Post value was passed
     2870        // Edit reply
     2871        } elseif ( bbp_is_reply_edit() ) {
     2872            $r['reply_id'] = bbp_get_reply_id( $r['reply_id'] );
     2873            $r['selected'] = bbp_get_reply_status( $r['reply_id'] );
     2874
     2875        // New reply
    28692876        } else {
    2870 
    2871             // Edit reply
    2872             if ( bbp_is_reply_edit() ) {
    2873                 $r['reply_id'] = bbp_get_reply_id( $r['reply_id'] );
    2874                 $r['selected'] = bbp_get_reply_status( $r['reply_id'] );
    2875 
    2876             // New reply
    2877             } else {
    2878                 $r['selected'] = bbp_get_public_status_id();
    2879             }
     2877            $r['selected'] = bbp_get_public_status_id();
    28802878        }
    28812879    }

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

    r7378 r7379  
    469469        return;
    470470
    471     // Topic exists
     471    // Check users ability to create new topic
     472    } elseif ( ! bbp_is_topic_anonymous( $topic_id ) ) {
     473
     474        // User cannot edit this topic
     475        if ( ! current_user_can( 'edit_topic', $topic_id ) ) {
     476            bbp_add_error( 'bbp_edit_topic_permission', __( '<strong>Error</strong>: You do not have permission to edit that topic.', 'bbpress' ) );
     477        }
     478
     479        // Set topic author
     480        $topic_author = bbp_get_topic_author_id( $topic_id );
     481
     482    // It is an anonymous post
    472483    } else {
    473484
    474         // Check users ability to create new topic
    475         if ( ! bbp_is_topic_anonymous( $topic_id ) ) {
    476 
    477             // User cannot edit this topic
    478             if ( ! current_user_can( 'edit_topic', $topic_id ) ) {
    479                 bbp_add_error( 'bbp_edit_topic_permission', __( '<strong>Error</strong>: You do not have permission to edit that topic.', 'bbpress' ) );
    480             }
    481 
    482             // Set topic author
    483             $topic_author = bbp_get_topic_author_id( $topic_id );
    484 
    485         // It is an anonymous post
    486         } else {
    487 
    488             // Filter anonymous data
    489             $anonymous_data = bbp_filter_anonymous_post_data();
    490         }
     485        // Filter anonymous data
     486        $anonymous_data = bbp_filter_anonymous_post_data();
    491487    }
    492488
    … …  
    11751171        bbp_add_error( 'bbp_merge_topic_nonce', __( '<strong>Error</strong>: Are you sure you wanted to do that?', 'bbpress' ) );
    11761172        return;
     1173    }
     1174
     1175    // Get source topic
     1176    $source_topic = bbp_get_topic( $source_topic_id );
    11771177
    11781178    // Source topic not found
    1179     } elseif ( ! $source_topic = bbp_get_topic( $source_topic_id ) ) {
     1179    if ( empty( $source_topic ) ) {
    11801180        bbp_add_error( 'bbp_merge_topic_source_not_found', __( '<strong>Error</strong>: The topic you want to merge was not found.', 'bbpress' ) );
    11811181        return;
    … …  
    11971197    }
    11981198
     1199    // Get the destination topic
     1200    $destination_topic = bbp_get_topic( $destination_topic_id );
     1201
    11991202    // Destination topic not found
    1200     if ( ! $destination_topic = bbp_get_topic( $destination_topic_id ) ) {
     1203    if ( empty( $destination_topic ) ) {
    12011204        bbp_add_error( 'bbp_merge_topic_destination_not_found', __( '<strong>Error</strong>: The topic you want to merge to was not found.', 'bbpress' ) );
    12021205    }
    … …  
    15071510            case 'reply' :
    15081511            default :
    1509 
    15101512                // User needs to be able to publish topics
    15111513                if ( current_user_can( 'publish_topics' ) ) {
    … …  
    18121814
    18131815            // No tag name was provided
    1814             if ( empty( $_POST['tag-name'] ) || ! $name = $_POST['tag-name'] ) {
     1816            if ( empty( $_POST['tag-name'] ) || ! is_scalar( $_POST['tag-name'] ) ) {
    18151817                bbp_add_error( 'bbp_manage_topic_tag_update_name', __( '<strong>Error</strong>: You need to enter a tag name.', 'bbpress' ) );
    18161818                return;
    … …  
    18181820
    18191821            // Attempt to update the tag
     1822            $name        = $_POST['tag-name'];
    18201823            $slug        = ! empty( $_POST['tag-slug']        ) ? $_POST['tag-slug'] : '';
    18211824            $description = ! empty( $_POST['tag-description'] ) ? $_POST['tag-description'] : '';
    … …  
    18611864
    18621865            // No tag name was provided
    1863             if ( empty( $_POST['tag-existing-name'] ) || ! $name = $_POST['tag-existing-name'] ) {
     1866            if ( empty( $_POST['tag-existing-name'] ) || ! is_scalar( $_POST['tag-existing-name'] ) ) {
    18641867                bbp_add_error( 'bbp_manage_topic_tag_merge_name', __( '<strong>Error</strong>: You need to enter a tag name.', 'bbpress' ) );
    18651868                return;
    18661869            }
    18671870
     1871            // Unsanitized name of existing tag
     1872            $name = $_POST['tag-existing-name'];
     1873
    18681874            // If term does not exist, create it
    1869             if ( ! $tag = term_exists( $name, bbp_get_topic_tag_tax_id() ) ) {
     1875            $tag = term_exists( $name, bbp_get_topic_tag_tax_id() );
     1876            if ( empty( $tag ) ) {
    18701877                $tag = wp_insert_term( $name, bbp_get_topic_tag_tax_id() );
    18711878            }
    … …  
    39313938        <?php if ( bbp_has_topics( $topics_query ) ) : ?>
    39323939
    3933             <?php while ( bbp_topics() ) : bbp_the_topic(); ?>
     3940            <?php while ( bbp_topics() ) :
     3941
     3942                bbp_the_topic(); ?>
    39343943
    39353944                <item>

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

    r7378 r7379  
    33163316                $r['selected'] = sanitize_key( $_POST[ $r['select_id'] ] );
    33173317
    3318             // No Post value passed
    3319             } else {
    3320 
    3321                 // Edit topic
    3322                 if ( bbp_is_single_topic() || bbp_is_topic_edit() ) {
    3323 
    3324                     // Get current topic id
    3325                     $r['topic_id'] = bbp_get_topic_id( $r['topic_id'] );
    3326 
    3327                     // Topic is super sticky
    3328                     if ( bbp_is_topic_super_sticky( $r['topic_id'] ) ) {
    3329                         $r['selected'] = 'super';
    3330 
    3331                     // Topic is sticky or normal
    3332                     } else {
    3333                         $r['selected'] = bbp_is_topic_sticky( $r['topic_id'], false )
    3334                             ? 'stick'
    3335                             : 'unstick';
    3336                     }
     3318            // Edit topic
     3319            } elseif ( bbp_is_single_topic() || bbp_is_topic_edit() ) {
     3320
     3321                // Get current topic id
     3322                $r['topic_id'] = bbp_get_topic_id( $r['topic_id'] );
     3323
     3324                // Topic is super sticky
     3325                if ( bbp_is_topic_super_sticky( $r['topic_id'] ) ) {
     3326                    $r['selected'] = 'super';
     3327
     3328                // Topic is sticky or normal
     3329                } else {
     3330                    $r['selected'] = bbp_is_topic_sticky( $r['topic_id'], false )
     3331                        ? 'stick'
     3332                        : 'unstick';
    33373333                }
    33383334            }
    … …  
    34093405                $r['selected'] = sanitize_key( $_POST[ $r['select_id'] ] );
    34103406
    3411             // No Post value was passed
     3407            // Edit topic
     3408            } elseif ( bbp_is_topic_edit() ) {
     3409                $r['topic_id'] = bbp_get_topic_id( $r['topic_id'] );
     3410                $r['selected'] = bbp_get_topic_status( $r['topic_id'] );
     3411
     3412            // New topic
    34123413            } else {
    3413 
    3414                 // Edit topic
    3415                 if ( bbp_is_topic_edit() ) {
    3416                     $r['topic_id'] = bbp_get_topic_id( $r['topic_id'] );
    3417                     $r['selected'] = bbp_get_topic_status( $r['topic_id'] );
    3418 
    3419                 // New topic
    3420                 } else {
    3421                     $r['selected'] = bbp_get_public_status_id();
    3422                 }
     3414                $r['selected'] = bbp_get_public_status_id();
    34233415            }
    34243416        }
Note: See TracChangeset for help on using the changeset viewer.