Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
04/23/2024 09:39:11 PM (12 months ago)
Author:
imath
Message:

Members component: improve PHP code standards using WPCS

See #7228 (trunk)

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-members/bp-members-template.php

    r13647 r13799  
    1919 */
    2020function bp_profile_slug() {
    21     echo bp_get_profile_slug();
     21    echo esc_url( bp_get_profile_slug() );
    2222}
    2323    /**
    … …  
    4646 */
    4747function bp_members_slug() {
    48     echo bp_get_members_slug();
     48    echo esc_url( bp_get_members_slug() );
    4949}
    5050    /**
    … …  
    7373 */
    7474function bp_members_root_slug() {
    75     echo bp_get_members_root_slug();
     75    echo esc_url( bp_get_members_root_slug() );
    7676}
    7777    /**
    … …  
    216216 */
    217217function bp_signup_slug() {
    218     echo bp_get_signup_slug();
     218    echo esc_url( bp_get_signup_slug() );
    219219}
    220220    /**
    … …  
    249249 */
    250250function bp_activate_slug() {
    251     echo bp_get_activate_slug();
     251    echo esc_url( bp_get_activate_slug() );
    252252}
    253253    /**
    … …  
    282282 */
    283283function bp_members_invitations_slug() {
    284     echo bp_get_members_invitations_slug();
     284    echo esc_url( bp_get_members_invitations_slug() );
    285285}
    286286    /**
    … …  
    490490 */
    491491function bp_members_pagination_count() {
    492     echo bp_get_members_pagination_count();
     492    echo esc_html( bp_get_members_pagination_count() );
    493493}
    494494    /**
    … …  
    559559 */
    560560function bp_members_pagination_links() {
     561    // Escaping is done in WordPress's `paginate_links()` function.
     562    // phpcs:ignore WordPress.Security.EscapeOutput
    561563    echo bp_get_members_pagination_links();
    562564}
    … …  
    589591 */
    590592function bp_member_user_id() {
    591     echo bp_get_member_user_id();
     593    echo intval( bp_get_member_user_id() );
    592594}
    593595    /**
    … …  
    625627 */
    626628function bp_member_class( $classes = array() ) {
     629    // phpcs:ignore WordPress.Security.EscapeOutput
    627630    echo bp_get_member_class( $classes );
    628631}
    … …  
    673676        if ( $member_types = bp_get_member_type( $members_template->member->id, false ) ) {
    674677            foreach ( $member_types as $member_type ) {
    675                 $classes[] = sprintf( 'member-type-%s', esc_attr( $member_type ) );
     678                $classes[] = sprintf( 'member-type-%s', $member_type );
    676679            }
    677680        }
    … …  
    682685         * @since 1.7.0
    683686         *
    684          * @param string $classes Classes to be added to the HTML element.
    685          */
    686         $classes = apply_filters( 'bp_get_member_class', $classes );
     687         * @param array $classes Classes to be added to the HTML element.
     688         */
     689        $classes = array_map( 'sanitize_html_class', apply_filters( 'bp_get_member_class', $classes ) );
    687690        $classes = array_merge( $classes, array() );
    688691        $retval  = 'class="' . join( ' ', $classes ) . '"';
    … …  
    697700 */
    698701function bp_member_user_nicename() {
    699     echo bp_get_member_user_nicename();
     702    echo esc_html( bp_get_member_user_nicename() );
    700703}
    701704    /**
    … …  
    727730 */
    728731function bp_member_user_login() {
    729     echo bp_get_member_user_login();
     732    echo esc_html( bp_get_member_user_login() );
    730733}
    731734    /**
    … …  
    757760 */
    758761function bp_member_user_email() {
    759     echo bp_get_member_user_email();
     762    echo esc_html( bp_get_member_user_email() );
    760763}
    761764    /**
    … …  
    811814 */
    812815function bp_member_avatar( $args = '' ) {
     816    // phpcs:disable WordPress.Security.EscapeOutput
    813817
    814818    /**
    … …  
    822826     */
    823827    echo apply_filters( 'bp_member_avatar', bp_get_member_avatar( $args ), $args );
     828    // phpcs:enable
    824829}
    825830    /**
    … …  
    950955 */
    951956function bp_member_name() {
     957    // phpcs:disable WordPress.Security.EscapeOutput
    952958
    953959    /**
    … …  
    959965     */
    960966    echo apply_filters( 'bp_member_name', bp_get_member_name() );
     967    // phpcs:enable
    961968}
    962969    /**
    … …  
    10171024 */
    10181025function bp_member_last_active( $args = array() ) {
    1019     echo bp_get_member_last_active( $args );
     1026    echo esc_html( bp_get_member_last_active( $args ) );
    10201027}
    10211028    /**
    … …  
    10911098 */
    10921099function bp_member_latest_update( $args = '' ) {
     1100    // phpcs:ignore WordPress.Security.EscapeOutput
    10931101    echo bp_get_member_latest_update( $args );
    10941102}
    … …  
    12071215 */
    12081216function bp_member_profile_data( $args = '' ) {
     1217    // phpcs:ignore WordPress.Security.EscapeOutput
    12091218    echo bp_get_member_profile_data( $args );
    12101219}
    … …  
    13151324 */
    13161325function bp_member_registered( $args = array() ) {
    1317     echo bp_get_member_registered( $args );
     1326    echo esc_html( bp_get_member_registered( $args ) );
    13181327}
    13191328    /**
    … …  
    13691378 */
    13701379function bp_member_random_profile_data() {
    1371     if ( bp_is_active( 'xprofile' ) ) { ?>
    1372         <?php $random_data = xprofile_get_random_profile_data( bp_get_member_user_id(), true ); ?>
     1380    if ( bp_is_active( 'xprofile' ) ) {
     1381        $random_data = xprofile_get_random_profile_data( bp_get_member_user_id(), true );
     1382        // phpcs:disable WordPress.Security.EscapeOutput
     1383        ?>
    13731384            <strong><?php echo wp_filter_kses( $random_data[0]->name ) ?></strong>
    13741385            <?php echo wp_filter_kses( $random_data[0]->value ) ?>
    1375     <?php }
     1386        <?php
     1387        // phpcs:enable
     1388    }
    13761389}
    13771390
    … …  
    14141427    $search_form_html = '<form action="" method="get" id="search-members-form">
    14151428        <label for="members_search"><input type="text" name="' . esc_attr( $query_arg ) . '" id="members_search" placeholder="'. esc_attr( $search_value ) .'" /></label>
    1416         <input type="submit" id="members_search_submit" name="members_search_submit" value="' . __( 'Search', 'buddypress' ) . '" />
     1429        <input type="submit" id="members_search_submit" name="members_search_submit" value="' . esc_html__( 'Search', 'buddypress' ) . '" />
    14171430    </form>';
    14181431
    1419     /**
    1420      * Filters the Members component search form.
    1421      *
    1422      * @since 1.9.0
    1423      *
    1424      * @param string $search_form_html HTML markup for the member search form.
    1425      */
    1426     echo apply_filters( 'bp_directory_members_search_form', $search_form_html );
     1432    // phpcs:ignore WordPress.Security.EscapeOutput
     1433    echo apply_filters(
     1434        /**
     1435         * Filters the Members component search form.
     1436         *
     1437         * @since 1.9.0
     1438         *
     1439         * @param string $search_form_html HTML markup for the member search form.
     1440         */
     1441        'bp_directory_members_search_form',
     1442        $search_form_html
     1443    );
    14271444}
    14281445
    … …  
    14331450 */
    14341451function bp_total_site_member_count() {
    1435     echo bp_get_total_site_member_count();
     1452    echo esc_html( bp_get_total_site_member_count() );
    14361453}
    14371454    /**
    … …  
    15021519        }
    15031520
    1504         // Echo out the final list item.
    1505         echo apply_filters_ref_array( 'bp_get_loggedin_user_nav_' . $nav_item->css_id, array( '<li id="li-nav-' . $nav_item->css_id . '" ' . $selected . '><a id="my-' . $nav_item->css_id . '" href="' . $nav_item->link . '">' . $nav_item->name . '</a></li>', &$nav_item ) );
     1521        // phpcs:ignore WordPress.Security.EscapeOutput
     1522        echo apply_filters_ref_array( 'bp_get_loggedin_user_nav_' . $nav_item->css_id, array( '<li id="li-nav-' . esc_attr( $nav_item->css_id ) . '" ' . $selected . '><a id="my-' . esc_attr( $nav_item->css_id ) . '" href="' . esc_url( $nav_item->link ) . '">' . esc_html( $nav_item->name ) . '</a></li>', &$nav_item ) );
    15061523    }
    15071524
    15081525    // Always add a log out list item to the end of the navigation.
    1509     $logout_link = '<li><a id="wp-logout" href="' .  wp_logout_url( bp_get_root_url() ) . '">' . __( 'Log Out', 'buddypress' ) . '</a></li>';
    1510 
     1526    $logout_link = '<li><a id="wp-logout" href="' .  esc_url( wp_logout_url( bp_get_root_url() ) ) . '">' . esc_html__( 'Log Out', 'buddypress' ) . '</a></li>';
     1527
     1528    // phpcs:ignore WordPress.Security.EscapeOutput
    15111529    echo apply_filters( 'bp_logout_nav_link', $logout_link );
    15121530}
    … …  
    16301648        }
    16311649
    1632         /**
    1633          * Filters the navigation markup for the displayed user.
    1634          *
    1635          * This is a dynamic filter that is dependent on the navigation tab component being rendered.
    1636          *
    1637          * @since 1.1.0
    1638          *
    1639          * @param string $value         Markup for the tab list item including link.
    1640          * @param array  $user_nav_item Array holding parts used to construct tab list item.
    1641          *                              Passed by reference.
    1642          */
    1643         echo apply_filters_ref_array( 'bp_get_displayed_user_nav_' . $user_nav_item->css_id, array( '<li id="' . $user_nav_item->css_id . '-personal-li" ' . $selected . '><a id="user-' . $user_nav_item->css_id . '" href="' . $link . '">' . $user_nav_item->name . '</a></li>', &$user_nav_item ) );
     1650        // phpcs:ignore WordPress.Security.EscapeOutput
     1651        echo apply_filters_ref_array(
     1652            /**
     1653             * Filters the navigation markup for the displayed user.
     1654             *
     1655             * This is a dynamic filter that is dependent on the navigation tab component being rendered.
     1656             *
     1657             * @since 1.1.0
     1658             *
     1659             * @param string $value         Markup for the tab list item including link.
     1660             * @param array  $user_nav_item Array holding parts used to construct tab list item.
     1661             *                              Passed by reference.
     1662             */
     1663            'bp_get_displayed_user_nav_' . $user_nav_item->css_id,
     1664            array(
     1665                '<li id="' . esc_attr( $user_nav_item->css_id ) . '-personal-li" ' . $selected . '><a id="user-' . esc_attr( $user_nav_item->css_id ) . '" href="' . esc_url( $link ) . '">' . wp_kses( $user_nav_item->name, array( 'span' => array( 'class' => true ) ) ) . '</a></li>',
     1666                &$user_nav_item
     1667            )
     1668        );
    16441669    }
    16451670}
    … …  
    16711696 */
    16721697function bp_loggedin_user_avatar( $args = '' ) {
     1698    // phpcs:ignore WordPress.Security.EscapeOutput
    16731699    echo bp_get_loggedin_user_avatar( $args );
    16741700}
    … …  
    17301756 */
    17311757function bp_displayed_user_avatar( $args = '' ) {
     1758    // phpcs:ignore WordPress.Security.EscapeOutput
    17321759    echo bp_get_displayed_user_avatar( $args );
    17331760}
    … …  
    17851812 */
    17861813function bp_displayed_user_email() {
    1787     echo bp_get_displayed_user_email();
     1814    echo esc_html( bp_get_displayed_user_email() );
    17881815}
    17891816    /**
    … …  
    18241851 */
    18251852function bp_last_activity( $user_id = 0 ) {
    1826     echo bp_get_last_activity( $user_id );
     1853    echo esc_html( bp_get_last_activity( $user_id ) );
    18271854}
    18281855    /**
    … …  
    18611888 */
    18621889function bp_user_firstname() {
    1863     echo bp_get_user_firstname();
     1890    echo esc_html( bp_get_user_firstname() );
    18641891}
    18651892    /**
    … …  
    19932020     * @param string $url Generated link for the displayed user's profile.
    19942021     */
    1995     return apply_filters( 'bp_displayed_user_domain',$url );
     2022    return apply_filters( 'bp_displayed_user_domain', $url );
    19962023}
    19972024
    … …  
    20892116 */
    20902117function bp_displayed_user_fullname() {
    2091     echo bp_get_displayed_user_fullname();
     2118    echo esc_html( bp_get_displayed_user_fullname() );
    20922119}
    20932120    /**
    … …  
    21162143     * @since 1.0.0
    21172144     */
    2118     function bp_user_fullname() { echo bp_get_displayed_user_fullname(); }
     2145    function bp_user_fullname() { echo esc_html( bp_get_displayed_user_fullname() ); }
    21192146
    21202147
    … …  
    21252152 */
    21262153function bp_loggedin_user_fullname() {
    2127     echo bp_get_loggedin_user_fullname();
     2154    echo esc_html( bp_get_loggedin_user_fullname() );
    21282155}
    21292156    /**
    … …  
    21532180 */
    21542181function bp_displayed_user_username() {
    2155     echo bp_get_displayed_user_username();
     2182    echo esc_html( bp_get_displayed_user_username() );
    21562183}
    21572184    /**
    … …  
    21872214 */
    21882215function bp_loggedin_user_username() {
    2189     echo bp_get_loggedin_user_username();
     2216    echo esc_html( bp_get_loggedin_user_username() );
    21902217}
    21912218    /**
    … …  
    22212248 */
    22222249function bp_current_member_type_message() {
    2223     echo bp_get_current_member_type_message();
     2250    echo wp_kses( bp_get_current_member_type_message(), array( 'strong' => true ) );
    22242251}
    22252252    /**
    … …  
    22542281 */
    22552282function bp_member_type_directory_link( $member_type = '' ) {
     2283    // phpcs:ignore WordPress.Security.EscapeOutput
    22562284    echo bp_get_member_type_directory_link( $member_type );
    22572285}
    … …  
    23022330 */
    23032331function bp_member_type_list( $user_id = 0, $r = array() ) {
     2332    // phpcs:ignore WordPress.Security.EscapeOutput
    23042333    echo bp_get_member_type_list( $user_id, $r );
    23052334}
    … …  
    25772606 */
    25782607function bp_signup_username_value() {
    2579     echo bp_get_signup_username_value();
     2608    echo esc_html( bp_get_signup_username_value() );
    25802609}
    25812610    /**
    … …  
    26092638 */
    26102639function bp_signup_email_value() {
    2611     echo bp_get_signup_email_value();
     2640    echo esc_html( bp_get_signup_email_value() );
    26122641}
    26132642    /**
    … …  
    26472676 */
    26482677function bp_signup_with_blog_value() {
    2649     echo bp_get_signup_with_blog_value();
     2678    echo intval( bp_get_signup_with_blog_value() );
    26502679}
    26512680    /**
    … …  
    26772706 */
    26782707function bp_signup_blog_url_value() {
    2679     echo bp_get_signup_blog_url_value();
     2708    echo esc_url( bp_get_signup_blog_url_value() );
    26802709}
    26812710    /**
    … …  
    27092738 */
    27102739function bp_signup_subdomain_base() {
    2711     echo bp_signup_get_subdomain_base();
     2740    echo esc_url( bp_signup_get_subdomain_base() );
    27122741}
    27132742    /**
    … …  
    27452774 */
    27462775function bp_signup_blog_title_value() {
    2747     echo bp_get_signup_blog_title_value();
     2776    echo esc_html( bp_get_signup_blog_title_value() );
    27482777}
    27492778    /**
    … …  
    27772806 */
    27782807function bp_signup_blog_privacy_value() {
    2779     echo bp_get_signup_blog_privacy_value();
     2808    echo esc_html( bp_get_signup_blog_privacy_value() );
    27802809}
    27812810    /**
    … …  
    28092838 */
    28102839function bp_signup_avatar_dir_value() {
    2811     echo bp_get_signup_avatar_dir_value();
     2840    echo esc_html( bp_get_signup_avatar_dir_value() );
    28122841}
    28132842    /**
    … …  
    28222851
    28232852        // Check if signup_avatar_dir is passed.
    2824         if ( !empty( $_POST['signup_avatar_dir'] ) )
     2853        if ( ! empty( $_POST['signup_avatar_dir'] ) ) {
    28252854            $signup_avatar_dir = $_POST['signup_avatar_dir'];
    28262855
    2827         // If not, check if global is set.
    2828         elseif ( !empty( $bp->signup->avatar_dir ) )
     2856            // If not, check if global is set.
     2857        } elseif ( ! empty( $bp->signup->avatar_dir ) ) {
    28292858            $signup_avatar_dir = $bp->signup->avatar_dir;
    28302859
    2831         // If not, set false.
    2832         else
     2860            // If not, set false.
     2861        } else {
    28332862            $signup_avatar_dir = false;
     2863        }
    28342864
    28352865        /**
    … …  
    28722902 */
    28732903function bp_current_signup_step() {
    2874     echo bp_get_current_signup_step();
     2904    echo esc_html( bp_get_current_signup_step() );
    28752905}
    28762906    /**
    … …  
    28952925 */
    28962926function bp_signup_avatar( $args = '' ) {
     2927    // phpcs:ignore WordPress.Security.EscapeOutput
    28972928    echo bp_get_signup_avatar( $args );
    28982929}
    … …  
    29242955        );
    29252956
    2926         extract( $r, EXTR_SKIP );
    2927 
    29282957        $signup_avatar_dir = bp_get_signup_avatar_dir_value();
    29292958
    … …  
    29352964                'avatar_dir' => 'avatars/signups',
    29362965                'type'       => 'full',
    2937                 'width'      => $size,
    2938                 'height'     => $size,
    2939                 'alt'        => $alt,
    2940                 'class'      => $class,
     2966                'width'      => $r['size'],
     2967                'height'     => $r['size'],
     2968                'alt'        => $r['alt'],
     2969                'class'      => $r['class'],
    29412970            ) );
    29422971
    … …  
    29622991            $gravatar_url    = apply_filters( 'bp_gravatar_url', '//www.gravatar.com/avatar/' );
    29632992            $md5_lcase_email = md5( strtolower( bp_get_signup_email_value() ) );
    2964             $gravatar_img    = '<img src="' . $gravatar_url . $md5_lcase_email . '?d=' . $default_grav . '&amp;s=' . $size . '" width="' . $size . '" height="' . $size . '" alt="' . $alt . '" class="' . $class . '" />';
     2993            $gravatar_img    = '<img src="' . $gravatar_url . $md5_lcase_email . '?d=' . $default_grav . '&amp;s=' . $r['size'] . '" width="' . esc_attr( $r['size'] ) . '" height="' . esc_attr( $r['size'] ) . '" alt="' . esc_attr( $r['alt'] ) . '" class="' . esc_attr( $r['class'] ) . '" />';
    29652994        }
    29662995
    … …  
    29843013 */
    29853014function bp_signup_allowed() {
     3015    // phpcs:ignore WordPress.Security.EscapeOutput
    29863016    echo bp_get_signup_allowed();
    29873017}
    … …  
    30813111 */
    30823112function bp_members_activity_feed() {
    3083     if ( !bp_is_active( 'activity' ) || !bp_is_user() )
    3084         return; ?>
    3085 
    3086     <link rel="alternate" type="application/rss+xml" title="<?php bloginfo( 'name' ) ?> | <?php bp_displayed_user_fullname() ?> | <?php _e( 'Activity RSS Feed', 'buddypress' ) ?>" href="<?php bp_member_activity_feed_link() ?>" />
    3087 
    3088 <?php
     3113    if ( ! bp_is_active( 'activity' ) || ! bp_is_user() ) {
     3114        return;
     3115    }
     3116    // phpcs:disable WordPress.Security.EscapeOutput
     3117    ?>
     3118    <link rel="alternate" type="application/rss+xml" title="<?php bloginfo( 'name' ) ?> | <?php bp_displayed_user_fullname() ?> | <?php esc_attr_e( 'Activity RSS Feed', 'buddypress' ) ?>" href="<?php bp_member_activity_feed_link() ?>" />
     3119    <?php
     3120    // phpcs:enable
    30893121}
    30903122add_action( 'bp_head', 'bp_members_activity_feed' );
    … …  
    31703202 */
    31713203function bp_avatar_delete_link() {
    3172     echo bp_get_avatar_delete_link();
     3204    echo esc_url( bp_get_avatar_delete_link() );
    31733205}
    31743206    /**
    … …  
    33123344 */
    33133345function bp_members_invitations_pagination_count() {
    3314     echo bp_get_members_invitations_pagination_count();
     3346    echo esc_html( bp_get_members_invitations_pagination_count() );
    33153347}
    33163348    /**
    … …  
    33523384 */
    33533385function bp_members_invitations_pagination_links() {
     3386    // phpcs:ignore WordPress.Security.EscapeOutput
    33543387    echo bp_get_members_invitations_pagination_links();
    33553388}
    … …  
    33883421    }
    33893422
    3390     /**
    3391      * Use this filter to sanitize the output.
    3392      *
    3393      * @since 8.0.0
    3394      *
    3395      * @param int|string $value    The value for the requested property.
    3396      * @param string     $property The name of the requested property.
    3397      * @param string     $context  The context of display.
    3398      */
    3399     echo apply_filters( 'bp_the_members_invitation_property', bp_get_the_members_invitation_property( $property ), $property, $context );
     3423    // phpcs:ignore WordPress.Security.EscapeOutput
     3424    echo apply_filters(
     3425        /**
     3426         * Use this filter to sanitize the output.
     3427         *
     3428         * @since 8.0.0
     3429         *
     3430         * @param int|string $value    The value for the requested property.
     3431         * @param string     $property The name of the requested property.
     3432         * @param string     $context  The context of display.
     3433         */
     3434        'bp_the_members_invitation_property',
     3435        bp_get_the_members_invitation_property( $property ),
     3436        $property,
     3437        $context
     3438    );
    34003439}
    34013440    /**
    … …  
    34523491 */
    34533492function bp_the_members_invitation_action_links( $args = '' ) {
     3493    // phpcs:ignore WordPress.Security.EscapeOutput
    34543494    echo bp_get_the_members_invitation_action_links( $args );
    34553495}
    … …  
    35083548 */
    35093549function bp_the_members_invitations_resend_link( $user_id = 0 ) {
     3550    // phpcs:ignore WordPress.Security.EscapeOutput
    35103551    echo bp_get_the_members_invitation_delete_link( $user_id );
    35113552}
    … …  
    35273568        }
    35283569
    3529         $retval = sprintf( '<a href="%1$s" class="resend secondary confirm bp-tooltip">%2$s</a>', esc_url( bp_get_the_members_invitations_resend_url( $user_id ) ), __( 'Resend', 'buddypress' ) );
     3570        $retval = sprintf( '<a href="%1$s" class="resend secondary confirm bp-tooltip">%2$s</a>', esc_url( bp_get_the_members_invitations_resend_url( $user_id ) ), esc_html__( 'Resend', 'buddypress' ) );
    35303571
    35313572        /**
    … …  
    35993640 */
    36003641function bp_the_members_invitations_delete_link( $user_id = 0 ) {
     3642    // phpcs:ignore WordPress.Security.EscapeOutput
    36013643    echo bp_get_the_members_invitation_delete_link( $user_id );
    36023644}
    … …  
    36963738 */
    36973739function bp_members_invitations_list_invites_permalink( $user_id = 0 ) {
    3698     echo bp_get_members_invitations_list_invites_permalink( $user_id );
     3740    echo esc_url( bp_get_members_invitations_list_invites_permalink( $user_id ) );
    36993741}
    37003742    /**
    … …  
    37343776 */
    37353777function bp_members_invitations_send_invites_permalink( $user_id = 0 ) {
    3736     echo bp_get_members_invitations_send_invites_permalink( $user_id );
     3778    echo esc_url( bp_get_members_invitations_send_invites_permalink( $user_id ) );
    37373779}
    37383780    /**
Note: See TracChangeset for help on using the changeset viewer.