Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
04/27/2024 05:51:41 PM (2 years ago)
Author:
imath
Message:

Activity component: improve PHP code standards using WPCS

See #7228 (trunk)

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r13757 r13816  
    1818 */
    1919function bp_activity_slug() {
    20         echo bp_get_activity_slug();
     20        echo esc_url( bp_get_activity_slug() );
    2121}
    2222        /**
    … …  
    4747 */
    4848function bp_activity_root_slug() {
    49         echo bp_get_activity_root_slug();
     49        echo esc_url( bp_get_activity_root_slug() );
    5050}
    5151        /**
    … …  
    458458 */
    459459function bp_activity_pagination_count() {
    460         echo bp_get_activity_pagination_count();
     460        echo esc_html( bp_get_activity_pagination_count() );
    461461}
    462462
    … …  
    495495 */
    496496function bp_activity_pagination_links() {
     497        // Escaping is done in WordPress's `paginate_links()` function.
     498        // phpcs:ignore WordPress.Security.EscapeOutput
    497499        echo bp_get_activity_pagination_links();
    498500}
    … …  
    561563 */
    562564function bp_activity_count() {
    563         echo bp_get_activity_count();
     565        echo intval( bp_get_activity_count() );
    564566}
    565567
    … …  
    593595 */
    594596function bp_activity_per_page() {
    595         echo bp_get_activity_per_page();
     597        echo intval( bp_get_activity_per_page() );
    596598}
    597599
    … …  
    625627 */
    626628function bp_activity_id() {
    627         echo bp_get_activity_id();
     629        echo intval( bp_get_activity_id() );
    628630}
    629631
    … …  
    657659 */
    658660function bp_activity_item_id() {
    659         echo bp_get_activity_item_id();
     661        echo intval( bp_get_activity_item_id() );
    660662}
    661663
    … …  
    689691 */
    690692function bp_activity_secondary_item_id() {
    691         echo bp_get_activity_secondary_item_id();
     693        echo intval( bp_get_activity_secondary_item_id() );
    692694}
    693695
    … …  
    721723 */
    722724function bp_activity_date_recorded() {
    723         echo bp_get_activity_date_recorded();
     725        echo esc_html( bp_get_activity_date_recorded() );
    724726}
    725727
    … …  
    753755 */
    754756function bp_activity_member_display_name() {
    755         echo bp_get_activity_member_display_name();
     757        echo esc_html( bp_get_activity_member_display_name() );
    756758}
    757759
    … …  
    789791 */
    790792function bp_activity_object_name() {
    791         echo bp_get_activity_object_name();
     793        echo esc_html( bp_get_activity_object_name() );
    792794}
    793795
    … …  
    821823 */
    822824function bp_activity_type() {
    823         echo bp_get_activity_type();
     825        echo esc_html( bp_get_activity_type() );
    824826}
    825827
    … …  
    878880         *
    879881         */
    880         function bp_activity_action_name() { echo bp_activity_type(); }
     882        function bp_activity_action_name() { bp_activity_type(); }
    881883
    882884        /**
    … …  
    902904 */
    903905function bp_activity_user_id() {
    904         echo bp_get_activity_user_id();
     906        echo intval( bp_get_activity_user_id() );
    905907}
    906908
    … …  
    934936 */
    935937function bp_activity_user_link() {
    936         echo bp_get_activity_user_link();
     938        echo esc_url( bp_get_activity_user_link() );
    937939}
    938940
    … …  
    975977 */
    976978function bp_activity_avatar( $args = '' ) {
     979        // phpcs:ignore WordPress.Security.EscapeOutput
    977980        echo bp_get_activity_avatar( $args );
    978981}
    … …  
    10161019
    10171020                // Prepend some descriptive text to alt.
    1018                 $alt_default = __( 'Profile picture', 'buddypress' );
     1021                $alt_default = esc_html__( 'Profile picture', 'buddypress' );
    10191022                if ( ! empty( $dn_default ) ) {
    10201023                        /* translators: %s: member name */
    1021                         $alt_default = sprintf( __( 'Profile picture of %s', 'buddypress' ), $dn_default );
     1024                        $alt_default = sprintf( esc_html__( 'Profile picture of %s', 'buddypress' ), $dn_default );
    10221025                }
    10231026
    … …  
    10351038                );
    10361039
    1037                 extract( $r, EXTR_SKIP );
    1038 
    1039                 if ( !isset( $height ) && !isset( $width ) ) {
     1040                if ( ! isset( $r['height'] ) && ! isset( $r['width'] ) ) {
    10401041
    10411042                        // Backpat.
    10421043                        if ( isset( $bp->avatar->full->height ) || isset( $bp->avatar->thumb->height ) ) {
    1043                                 $height = ( 'full' == $type ) ? $bp->avatar->full->height : $bp->avatar->thumb->height;
     1044                                $r['height'] = ( 'full' == $r['type'] ) ? $bp->avatar->full->height : $bp->avatar->thumb->height;
    10441045                        } else {
    1045                                 $height = 20;
     1046                                $r['height'] = 20;
    10461047                        }
    10471048
    10481049                        // Backpat.
    10491050                        if ( isset( $bp->avatar->full->width ) || isset( $bp->avatar->thumb->width ) ) {
    1050                                 $width = ( 'full' == $type ) ? $bp->avatar->full->width : $bp->avatar->thumb->width;
     1051                                $r['width'] = ( 'full' == $r['type'] ) ? $bp->avatar->full->width : $bp->avatar->thumb->width;
    10511052                        } else {
    1052                                 $width = 20;
     1053                                $r['width'] = 20;
    10531054                        }
    10541055                }
    … …  
    10661067                 */
    10671068                $object  = apply_filters( 'bp_get_activity_avatar_object_' . $current_activity_item->component, 'user' );
    1068                 $item_id = !empty( $user_id ) ? $user_id : $current_activity_item->user_id;
     1069                $item_id = $current_activity_item->user_id;
     1070
     1071                if ( ! empty( $r['user_id'] ) ) {
     1072                        $item_id = (int) $r['user_id'];
     1073                }
    10691074
    10701075                /**
    … …  
    10781083
    10791084                // If this is a user object pass the users' email address for Gravatar so we don't have to prefetch it.
    1080                 if ( 'user' == $object && empty( $user_id ) && empty( $email ) && isset( $current_activity_item->user_email ) ) {
    1081                         $email = $current_activity_item->user_email;
     1085                if ( 'user' == $object && empty( $r['user_id'] ) && empty( $r['email'] ) && isset( $current_activity_item->user_email ) ) {
     1086                        $r['email'] = $current_activity_item->user_email;
    10821087                }
    10831088
    … …  
    10891094                 * @param array $value HTML image element containing the activity avatar.
    10901095                 */
    1091                 return apply_filters( 'bp_get_activity_avatar', bp_core_fetch_avatar( array(
    1092                         'item_id' => $item_id,
    1093                         'object'  => $object,
    1094                         'type'    => $type,
    1095                         'alt'     => $alt,
    1096                         'class'   => $class,
    1097                         'width'   => $width,
    1098                         'height'  => $height,
    1099                         'email'   => $email
    1100                 ) ) );
     1096                return apply_filters( 'bp_get_activity_avatar',
     1097                        bp_core_fetch_avatar(
     1098                                array(
     1099                                        'item_id' => $item_id,
     1100                                        'object'  => $object,
     1101                                        'type'    => $r['type'],
     1102                                        'alt'     => $r['alt'],
     1103                                        'class'   => $r['class'],
     1104                                        'width'   => $r['width'],
     1105                                        'height'  => $r['height'],
     1106                                        'email'   => $r['email'],
     1107                                )
     1108                        )
     1109                );
    11011110        }
    11021111
    … …  
    11111120 */
    11121121function bp_activity_secondary_avatar( $args = '' ) {
     1122        // phpcs:ignore WordPress.Security.EscapeOutput
    11131123        echo bp_get_activity_secondary_avatar( $args );
    11141124}
    … …  
    11511161                );
    11521162
    1153                 extract( $r, EXTR_SKIP );
    1154 
    11551163                // Set item_id and object (default to user).
    11561164                switch ( $activities_template->activity->component ) {
    … …  
    11721180                                }
    11731181
    1174                                 if ( empty( $alt ) ) {
    1175                                         $alt = __( 'Group logo', 'buddypress' );
     1182                                if ( empty( $r['alt'] ) ) {
     1183                                        $r['alt'] = esc_html__( 'Group logo', 'buddypress' );
    11761184
    11771185                                        if ( ! empty( $name ) ) {
    11781186                                                /* translators: %s: the Group name */
    1179                                                 $alt = sprintf( __( 'Group logo of %s', 'buddypress' ), $name );
     1187                                                $r['alt'] = sprintf( esc_html__( 'Group logo of %s', 'buddypress' ), $name );
    11801188                                        }
    11811189                                }
    … …  
    11871195                                $link    = home_url();
    11881196
    1189                                 if ( empty( $alt ) ) {
     1197                                if ( empty( $r['alt'] ) ) {
    11901198                                        /* translators: %s: the blog name */
    1191                                         $alt = sprintf( __( 'Profile picture of the author of the site %s', 'buddypress' ), get_blog_option( $item_id, 'blogname' ) );
     1199                                        $r['alt'] = sprintf( esc_html__( 'Profile picture of the author of the site %s', 'buddypress' ), esc_html( get_blog_option( $item_id, 'blogname' ) ) );
    11921200                                }
    11931201
    … …  
    11981206                                $link    = bp_core_get_userlink( $item_id, false, true );
    11991207
    1200                                 if ( empty( $alt ) ) {
     1208                                if ( empty( $r['alt'] ) ) {
    12011209                                        /* translators: %s: member name */
    1202                                         $alt = sprintf( __( 'Profile picture of %s', 'buddypress' ), bp_core_get_user_displayname( $activities_template->activity->secondary_item_id ) );
     1210                                        $r['alt'] = sprintf( esc_html__( 'Profile picture of %s', 'buddypress' ), bp_core_get_user_displayname( $activities_template->activity->secondary_item_id ) );
    12031211                                }
    12041212
    12051213                                break;
    12061214                        default :
    1207                                 $object  = 'user';
    1208                                 $item_id = $activities_template->activity->user_id;
    1209                                 $email  = $activities_template->activity->user_email;
    1210                                 $link    = bp_core_get_userlink( $item_id, false, true );
    1211 
    1212                                 if ( empty( $alt ) ) {
     1215                                $object     = 'user';
     1216                                $item_id    = $activities_template->activity->user_id;
     1217                                $r['email'] = $activities_template->activity->user_email;
     1218                                $link       = bp_core_get_userlink( $item_id, false, true );
     1219
     1220                                if ( empty( $r['alt'] ) ) {
    12131221                                        /* translators: %s: member name */
    1214                                         $alt = sprintf( __( 'Profile picture of %s', 'buddypress' ), $activities_template->activity->display_name );
     1222                                        $r['alt'] = sprintf( esc_html__( 'Profile picture of %s', 'buddypress' ), esc_html( $activities_template->activity->display_name ) );
    12151223                                }
    12161224
    … …  
    12291237                 * @param string $object Component being displayed.
    12301238                 */
    1231                 $object  = apply_filters( 'bp_get_activity_secondary_avatar_object_' . $activities_template->activity->component, $object );
     1239                $object = apply_filters( 'bp_get_activity_secondary_avatar_object_' . $activities_template->activity->component, $object );
    12321240
    12331241                /**
    … …  
    12461254
    12471255                // Get the avatar.
    1248                 $avatar = bp_core_fetch_avatar( array(
    1249                         'item_id' => $item_id,
    1250                         'object'  => $object,
    1251                         'type'    => $type,
    1252                         'alt'     => $alt,
    1253                         'class'   => $class,
    1254                         'width'   => $width,
    1255                         'height'  => $height,
    1256                         'email'   => $email
    1257                 ) );
    1258 
    1259                 if ( !empty( $linked ) ) {
     1256                $avatar = bp_core_fetch_avatar(
     1257                        array(
     1258                                'item_id' => $item_id,
     1259                                'object'  => $object,
     1260                                'type'    => $r['type'],
     1261                                'alt'     => $r['alt'],
     1262                                'class'   => $r['class'],
     1263                                'width'   => $r['width'],
     1264                                'height'  => $r['height'],
     1265                                'email'   => $r['email'],
     1266                        )
     1267                );
     1268
     1269                if ( ! empty( $r['linked'] ) ) {
    12601270
    12611271                        /**
    … …  
    12791289
    12801290                        return sprintf( '<a href="%s" class="%s">%s</a>',
    1281                                 $link,
    1282                                 $link_class,
     1291                                esc_url( $link ),
     1292                                esc_attr( $r['link_class'] ),
    12831293                                $avatar
    12841294                        );
    … …  
    12971307 */
    12981308function bp_activity_action( $args = array() ) {
    1299         echo bp_get_activity_action( $args );
     1309        echo wp_kses(
     1310                bp_get_activity_action( $args ),
     1311                array(
     1312                        'p'    => true,
     1313                        'a'    => array(
     1314                                'href'            => true,
     1315                                'class'           => true,
     1316                                'data-bp-tooltip' => true,
     1317                        ),
     1318                        'span' => array(
     1319                                'class' => true,
     1320                        ),
     1321                        'img'  => array(
     1322                                'src'     => true,
     1323                                'loading' => true,
     1324                                'class'   => true,
     1325                                'alt'     => true,
     1326                                'height'  => true,
     1327                                'width'   => true,
     1328                        ),
     1329                )
     1330        );
    13001331}
    13011332
    … …  
    13631394 */
    13641395function bp_activity_content_body() {
     1396        // Escaping is made in `bp-activity/bp-activity-filters.php`.
     1397        // phpcs:ignore WordPress.Security.EscapeOutput
    13651398        echo bp_get_activity_content_body();
    13661399}
    … …  
    16081641 */
    16091642function bp_activity_generated_content_part( $property = '' ) {
     1643        // phpcs:ignore WordPress.Security.EscapeOutput
    16101644        echo bp_activity_get_generated_content_part( $property );
    16111645}
    … …  
    16641698 */
    16651699function bp_activity_content() {
     1700        // Escaping is made in `bp-activity/bp-activity-filters.php`.
     1701        // phpcs:ignore WordPress.Security.EscapeOutput
    16661702        echo bp_get_activity_content();
    16671703}
    … …  
    18421878 */
    18431879function bp_activity_parent_content( $args = '' ) {
    1844         echo bp_get_activity_parent_content($args);
     1880        // Escaping is made in `bp-activity/bp-activity-filters.php`.
     1881        // phpcs:ignore WordPress.Security.EscapeOutput
     1882        echo bp_get_activity_parent_content( $args );
    18451883}
    18461884
    … …  
    19061944 */
    19071945function bp_activity_parent_user_id() {
    1908         echo bp_get_activity_parent_user_id();
     1946        echo intval( bp_get_activity_parent_user_id() );
    19091947}
    19101948
    … …  
    19591997 */
    19601998function bp_activity_is_favorite() {
     1999        // phpcs:ignore WordPress.Security.EscapeOutput
    19612000        echo bp_get_activity_is_favorite();
    19622001}
    … …  
    19992038                        sprintf(
    20002039                                /* translators: 1: the name of the function. 2: the name of the file. */
    2001                                 __( '%1$s no longer accepts arguments. See the inline documentation at %2$s for more details.', 'buddypress' ),
     2040                                esc_html__( '%1$s no longer accepts arguments. See the inline documentation at %2$s for more details.', 'buddypress' ),
    20022041                                __FUNCTION__,
    20032042                                __FILE__
    … …  
    20062045        }
    20072046
     2047        // phpcs:ignore WordPress.Security.EscapeOutput
    20082048        echo bp_activity_get_comments();
    20092049}
    … …  
    20322072                                sprintf(
    20332073                                        /* translators: 1: the name of the function. 2: the name of the file. */
    2034                                         __( '%1$s no longer accepts arguments. See the inline documentation at %2$s for more details.', 'buddypress' ),
     2074                                        esc_html__( '%1$s no longer accepts arguments. See the inline documentation at %2$s for more details.', 'buddypress' ),
    20352075                                        __FUNCTION__,
    20362076                                        __FILE__
    … …  
    20702110                        }
    20712111
    2072                         /**
    2073                          * Filters the opening tag for the template that lists activity comments.
    2074                          *
    2075                          * @since 1.6.0
    2076                          *
    2077                          * @param string $value Opening tag for the HTML markup to use.
    2078                          */
    2079                         echo apply_filters( 'bp_activity_recurse_comments_start_ul', '<ul>' );
     2112                        // phpcs:ignore WordPress.Security.EscapeOutput
     2113                        echo apply_filters(
     2114                                /**
     2115                                 * Filters the opening tag for the template that lists activity comments.
     2116                                 *
     2117                                 * @since 1.6.0
     2118                                 *
     2119                                 * @param string $value Opening tag for the HTML markup to use.
     2120                                 */
     2121                                'bp_activity_recurse_comments_start_ul',
     2122                                '<ul>'
     2123                        );
     2124
    20802125                        foreach ( (array) $comment->children as $comment_child ) {
    20812126
    … …  
    21082153                        }
    21092154
    2110                         /**
    2111                          * Filters the closing tag for the template that list activity comments.
    2112                          *
    2113                          * @since  1.6.0
    2114                          *
    2115                          * @param string $value Closing tag for the HTML markup to use.
    2116                          */
    2117                         echo apply_filters( 'bp_activity_recurse_comments_end_ul', '</ul>' );
     2155                        // phpcs:ignore WordPress.Security.EscapeOutput
     2156                        echo apply_filters(
     2157                                /**
     2158                                 * Filters the closing tag for the template that list activity comments.
     2159                                 *
     2160                                 * @since  1.6.0
     2161                                 *
     2162                                 * @param string $value Closing tag for the HTML markup to use.
     2163                                 */
     2164                                'bp_activity_recurse_comments_end_ul',
     2165                                '</ul>'
     2166                        );
    21182167                }
    21192168
    … …  
    21532202 */
    21542203function bp_activity_comment_id() {
    2155         echo bp_get_activity_comment_id();
     2204        echo intval( bp_get_activity_comment_id() );
    21562205}
    21572206
    … …  
    21882237 */
    21892238function bp_activity_comment_user_id() {
    2190         echo bp_get_activity_comment_user_id();
     2239        echo intval( bp_get_activity_comment_user_id() );
    21912240}
    21922241
    … …  
    22232272 */
    22242273function bp_activity_comment_user_link() {
    2225         echo bp_get_activity_comment_user_link();
     2274        echo esc_url( bp_get_activity_comment_user_link() );
    22262275}
    22272276
    … …  
    22542303 */
    22552304function bp_activity_comment_name() {
    2256         echo bp_get_activity_comment_name();
     2305        echo esc_html( bp_get_activity_comment_name() );
    22572306}
    22582307
    … …  
    22962345 */
    22972346function bp_activity_comment_date_recorded() {
    2298         echo bp_get_activity_comment_date_recorded();
     2347        echo esc_html( bp_get_activity_comment_date_recorded() );
    22992348}
    23002349
    … …  
    23272376 */
    23282377function bp_activity_comment_date_recorded_raw() {
    2329         echo bp_get_activity_comment_date_recorded_raw();
     2378        echo esc_html( bp_get_activity_comment_date_recorded_raw() );
    23302379}
    23312380
    … …  
    23602409 */
    23612410function bp_activity_comment_delete_link() {
    2362         echo bp_get_activity_comment_delete_link();
     2411        echo esc_url( bp_get_activity_comment_delete_link() );
    23632412}
    23642413
    … …  
    23992448 */
    24002449function bp_activity_comment_content() {
     2450        // Escaping is made in `bp-activity/bp-activity-filters.php`.
     2451        // phpcs:ignore WordPress.Security.EscapeOutput
    24012452        echo bp_get_activity_comment_content();
    24022453}
    … …  
    24412492 */
    24422493function bp_activity_comment_count() {
    2443         echo bp_activity_get_comment_count();
     2494        echo intval( bp_activity_get_comment_count() );
    24442495}
    24452496
    … …  
    24642515                                sprintf(
    24652516                                        /* translators: 1: the name of the function. 2: the name of the file. */
    2466                                         __( '%1$s no longer accepts arguments. See the inline documentation at %2$s for more details.', 'buddypress' ),
     2517                                        esc_html__( '%1$s no longer accepts arguments. See the inline documentation at %2$s for more details.', 'buddypress' ),
    24672518                                        __FUNCTION__,
    24682519                                        __FILE__
    … …  
    25342585 */
    25352586function bp_activity_comment_depth( $comment = 0 ) {
    2536         echo bp_activity_get_comment_depth( $comment );
     2587        echo intval( bp_activity_get_comment_depth( $comment ) );
    25372588}
    25382589
    … …  
    26012652 */
    26022653function bp_activity_comment_link() {
    2603         echo bp_get_activity_comment_link();
     2654        echo esc_url( bp_get_activity_comment_link() );
    26042655}
    26052656
    … …  
    26482699 */
    26492700function bp_activity_comment_form_nojs_display() {
     2701        // phpcs:ignore WordPress.Security.EscapeOutput
    26502702        echo bp_get_activity_comment_form_nojs_display();
    26512703}
    … …  
    27522804 */
    27532805function bp_activity_permalink_id() {
    2754         echo bp_get_activity_permalink_id();
     2806        echo esc_html( bp_get_activity_permalink_id() );
    27552807}
    27562808
    … …  
    28532905 */
    28542906function bp_activity_favorite_link() {
    2855         echo bp_get_activity_favorite_link();
     2907        echo esc_url( bp_get_activity_favorite_link() );
    28562908}
    28572909
    … …  
    28952947 */
    28962948function bp_activity_unfavorite_link() {
    2897         echo bp_get_activity_unfavorite_link();
     2949        echo esc_url( bp_get_activity_unfavorite_link() );
    28982950}
    28992951
    … …  
    29372989 */
    29382990function bp_activity_css_class() {
    2939         echo bp_get_activity_css_class();
     2991        echo esc_attr( bp_get_activity_css_class() );
    29402992}
    29412993
    … …  
    29953047 */
    29963048function bp_activity_delete_link() {
     3049        // phpcs:ignore WordPress.Security.EscapeOutput
    29973050        echo bp_get_activity_delete_link();
    29983051}
    … …  
    30203073                }
    30213074
    3022                 $link = '<a href="' . esc_url( $url ) . '" class="button item-button bp-secondary-action ' . $class . ' confirm" rel="nofollow">' . __( 'Delete', 'buddypress' ) . '</a>';
     3075                $link = '<a href="' . esc_url( $url ) . '" class="button item-button bp-secondary-action ' . esc_attr( $class ) . ' confirm" rel="nofollow">' . esc_html__( 'Delete', 'buddypress' ) . '</a>';
    30233076
    30243077                /**
    … …  
    30963149 */
    30973150function bp_activity_latest_update( $user_id = 0 ) {
     3151        // Escaping is made in `bp-activity/bp-activity-filters.php`.
     3152        // phpcs:ignore WordPress.Security.EscapeOutput
    30983153        echo bp_get_activity_latest_update( $user_id );
    30993154}
    … …  
    31633218 */
    31643219function bp_activity_filter_links( $args = false ) {
     3220        // phpcs:ignore WordPress.Security.EscapeOutput
    31653221        echo bp_get_activity_filter_links( $args );
    31663222}
    … …  
    32013257
    32023258                        // Skip the activity comment filter.
    3203                         if ( 'activity' == $component ) {
     3259                        if ( 'activity' === $component ) {
    32043260                                continue;
    32053261                        }
    32063262
    3207                         if ( isset( $_GET['afilter'] ) && $component == $_GET['afilter'] ) {
     3263                        if ( isset( $_GET['afilter'] ) && $component === $_GET['afilter'] ) {
    32083264                                $selected = ' class="selected"';
    32093265                        } else {
    … …  
    32503306
    32513307                if ( isset( $_GET['afilter'] ) ) {
    3252                         $component_links[] = '<' . $tag . ' id="afilter-clear"><a href="' . esc_url( $link ) . '">' . __( 'Clear Filter', 'buddypress' ) . '</a></' . $tag . '>';
     3308                        $component_links[] = '<' . $tag . ' id="afilter-clear"><a href="' . esc_url( $link ) . '">' . esc_html__( 'Clear Filter', 'buddypress' ) . '</a></' . $tag . '>';
    32533309                }
    32543310
    … …  
    33783434 */
    33793435function bp_total_favorite_count_for_user( $user_id = 0 ) {
    3380         echo bp_get_total_favorite_count_for_user( $user_id );
     3436        echo intval( bp_get_total_favorite_count_for_user( $user_id ) );
    33813437}
    33823438
    … …  
    34283484 */
    34293485function bp_total_mention_count_for_user( $user_id = 0 ) {
    3430         echo bp_get_total_mention_count_for_user( $user_id );
     3486        echo intval( bp_get_total_mention_count_for_user( $user_id ) );
    34313487}
    34323488
    … …  
    35403596 */
    35413597function bp_mentioned_user_display_name( $user_id_or_username = false ) {
     3598        // phpcs:ignore WordPress.Security.EscapeOutput
    35423599        echo bp_get_mentioned_user_display_name( $user_id_or_username );
    35433600}
    … …  
    35593616                // If user somehow has no name, return this really lame string.
    35603617                if ( empty( $name ) ) {
    3561                         $name = __( 'a user', 'buddypress' );
     3618                        $name = esc_html__( 'a user', 'buddypress' );
    35623619                }
    35633620
    … …  
    35833640 */
    35843641function bp_send_public_message_button( $args = '' ) {
     3642        // Escaping is done in `BP_Core_HTML_Element()`.
     3643        // phpcs:ignore WordPress.Security.EscapeOutput
    35853644        echo bp_get_send_public_message_button( $args );
    35863645}
    … …  
    36623721 */
    36633722function bp_activity_post_form_action() {
    3664         echo bp_get_activity_post_form_action();
     3723        echo esc_url( bp_get_activity_post_form_action() );
    36653724}
    36663725
    … …  
    37343793
    37353794                        // Get avatar for this user.
    3736                         $image_html   = bp_core_fetch_avatar( array(
    3737                                 'item_id' => $user_id,
    3738                                 'height'  => $r['height'],
    3739                                 'html'    => $r['html'],
    3740                                 'type'    => $r['type'],
    3741                                 'width'   => $r['width']
    3742                         ) );
     3795                        $image_html = bp_core_fetch_avatar(
     3796                                array(
     3797                                        'item_id' => $user_id,
     3798                                        'height'  => $r['height'],
     3799                                        'html'    => $r['html'],
     3800                                        'type'    => $r['type'],
     3801                                        'width'   => $r['width']
     3802                                )
     3803                        );
    37433804
    37443805                        // If user has link & avatar, add them to the output array.
    … …  
    37543815        }
    37553816
    3756         /**
    3757          * Filters the list of linked avatars for users who have commented on the current activity item.
    3758          *
    3759          * @since 1.7.0
    3760          *
    3761          * @param string $retval HTML markup for the list of avatars.
    3762          * @param array  $r      Array of arguments used for each avatar.
    3763          * @param array  $output Array of each avatar found, before imploded into single string.
    3764          */
    3765         echo apply_filters( 'bp_activity_comments_user_avatars', $retval, $r, $output );
     3817        // phpcs:ignore WordPress.Security.EscapeOutput
     3818        echo apply_filters(
     3819                /**
     3820                 * Filters the list of linked avatars for users who have commented on the current activity item.
     3821                 *
     3822                 * @since 1.7.0
     3823                 *
     3824                 * @param string $retval HTML markup for the list of avatars.
     3825                 * @param array  $r      Array of arguments used for each avatar.
     3826                 * @param array  $output Array of each avatar found, before imploded into single string.
     3827                 */
     3828                'bp_activity_comments_user_avatars',
     3829                $retval,
     3830                $r,
     3831                $output
     3832        );
    37663833}
    37673834
    … …  
    38423909 */
    38433910function bp_displayed_user_mentionname() {
    3844         echo bp_get_displayed_user_mentionname();
     3911        echo esc_html( bp_get_displayed_user_mentionname() );
    38453912}
    38463913        /**
    … …  
    39003967                switch ( $output ) {
    39013968                        case 'select' :
     3969                                // phpcs:ignore WordPress.Security.EscapeOutput
    39023970                                printf( '<option value="%1$s" %2$s>%3$s</option>', esc_attr( $type ), $selected, esc_html( $description ) );
    39033971                                break;
    39043972                        case 'checkbox' :
     3973                                // phpcs:ignore WordPress.Security.EscapeOutput
    39053974                                printf( '<label style="" for="%1$s[]">%2$s<input type="checkbox" id="%1$s[]" name="%1$s[]" value="%3$s" %4$s/></label>', esc_attr( $args['checkbox_name'] ), esc_html( $description ), esc_attr( $args['checkbox_name'] ), esc_attr( $args['checkbox_name'] ), esc_attr( $type ), $checked );
    39063975                                break;
    … …  
    39384007 */
    39394008function bp_sitewide_activity_feed_link() {
    3940         echo bp_get_sitewide_activity_feed_link();
     4009        echo esc_url( bp_get_sitewide_activity_feed_link() );
    39414010}
    39424011
    … …  
    40544123 */
    40554124function bp_activity_feed_item_guid() {
    4056         echo bp_get_activity_feed_item_guid();
     4125        echo esc_html( bp_get_activity_feed_item_guid() );
    40574126}
    40584127
    … …  
    40864155 */
    40874156function bp_activity_feed_item_title() {
     4157        // phpcs:ignore WordPress.Security.EscapeOutput
    40884158        echo bp_get_activity_feed_item_title();
    40894159}
    … …  
    41354205 */
    41364206function bp_activity_feed_item_link() {
    4137         echo bp_get_activity_feed_item_link();
     4207        echo esc_url( bp_get_activity_feed_item_link() );
    41384208}
    41394209
    … …  
    41714241 */
    41724242function bp_activity_feed_item_date() {
    4173         echo bp_get_activity_feed_item_date();
     4243        echo esc_html( bp_get_activity_feed_item_date() );
    41744244}
    41754245
    … …  
    42074277 */
    42084278function bp_activity_feed_item_description() {
     4279        // Escaping is made in `bp-activity/bp-activity-filters.php`.
     4280        // phpcs:ignore WordPress.Security.EscapeOutput
    42094281        echo bp_get_activity_feed_item_description();
    42104282}
    … …  
    42514323?>
    42524324
    4253         <link rel="alternate" type="application/rss+xml" title="<?php bloginfo( 'name' ) ?> | <?php _e( 'Site Wide Activity RSS Feed', 'buddypress' ) ?>" href="<?php bp_sitewide_activity_feed_link() ?>" />
     4325        <link rel="alternate" type="application/rss+xml" title="<?php bloginfo( 'name' ) ?> | <?php esc_html_e( 'Site Wide Activity RSS Feed', 'buddypress' ) ?>" href="<?php bp_sitewide_activity_feed_link() ?>" />
    42544326
    42554327<?php
    … …  
    42664338 */
    42674339function bp_activity_show_filters( $context = '' ) {
     4340        // phpcs:ignore WordPress.Security.EscapeOutput
    42684341        echo bp_get_activity_show_filters( $context );
    42694342}
Note: See TracChangeset for help on using the changeset viewer.