Skip to:
Content

bbPress.org

Changeset 6591


Ignore:
Timestamp:
06/19/2017 09:46:29 AM (9 years ago)
Author:
johnjamesjacoby
Message:

Authors: Rejig author link functions.

  • Allow for easier filtering of link sections
  • Update default arguments to remove no-break space and <br>, and rely on styling instead
  • Surrounding code clean-up
  • Add escaping to display names in links
Location:
trunk/src
Files:
6 edited

Legend:

Unmodified
Added
Removed

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

    r6583 r6591  
    11321132                        'type'       => 'both',
    11331133                        'size'       => 80,
    1134                         'sep'        => '&nbsp;',
     1134                        'sep'        => '',
    11351135                        'show_role'  => false
    11361136                ), 'get_reply_author_link' );
    11371137
     1138                // Default return value
     1139                $author_link = '';
     1140
    11381141                // Used as reply_id
    1139                 if ( is_numeric( $args ) ) {
    1140                         $reply_id = bbp_get_reply_id( $args );
    1141                 } else {
    1142                         $reply_id = bbp_get_reply_id( $r['post_id'] );
    1143                 }
     1142                $reply_id = is_numeric( $args )
     1143                        ? bbp_get_reply_id( $args )
     1144                        : bbp_get_reply_id( $r['post_id'] );
    11441145
    11451146                // Reply ID is good
    … …  
    11521153                        // Tweak link title if empty
    11531154                        if ( empty( $r['link_title'] ) ) {
    1154                                 $link_title = sprintf( empty( $anonymous ) ? __( 'View %s\'s profile', 'bbpress' ) : __( 'Visit %s\'s website', 'bbpress' ), bbp_get_reply_author_display_name( $reply_id ) );
     1155                                $author     = bbp_get_reply_author_display_name( $reply_id );
     1156                                $title      = empty( $anonymous )
     1157                                        ? __( 'View %s\'s profile',  'bbpress' )
     1158                                        : __( 'Visit %s\'s website', 'bbpress' );
     1159                                $link_title = sprintf( $title, $author );
    11551160
    11561161                        // Use what was passed if not
    … …  
    11601165
    11611166                        // Setup title and author_links array
    1162                         $link_title   = ! empty( $link_title ) ? ' title="' . esc_attr( $link_title ) . '"' : '';
    11631167                        $author_links = array();
    1164 
    1165                         // Get avatar
    1166                         if ( 'avatar' === $r['type'] || 'both' === $r['type'] ) {
     1168                        $link_title   = ! empty( $link_title )
     1169                                ? ' title="' . esc_attr( $link_title ) . '"'
     1170                                : '';
     1171
     1172                        // Get avatar (unescaped, because HTML)
     1173                        if ( ( 'avatar' === $r['type'] ) || ( 'both' === $r['type'] ) ) {
    11671174                                $author_links['avatar'] = bbp_get_reply_author_avatar( $reply_id, $r['size'] );
    11681175                        }
    11691176
    1170                         // Get display name
    1171                         if ( 'name' === $r['type']   || 'both' === $r['type'] ) {
    1172                                 $author_links['name'] = bbp_get_reply_author_display_name( $reply_id );
     1177                        // Get display name (escaped, because never HTML)
     1178                        if ( ( 'name' === $r['type'] ) || ( 'both' === $r['type'] ) ) {
     1179                                $author_links['name'] = esc_html( bbp_get_reply_author_display_name( $reply_id ) );
    11731180                        }
    11741181
    … …  
    11791186                        if ( empty( $anonymous ) && bbp_user_has_profile( bbp_get_reply_author_id( $reply_id ) ) ) {
    11801187
     1188                                // Empty array
     1189                                $links = array();
     1190
    11811191                                // Assemble the links
    11821192                                foreach ( $author_links as $link => $link_text ) {
    1183                                         $link_class = ' class="bbp-author-' . $link . '"';
    1184                                         $author_link[] = sprintf( '<a href="%1$s"%2$s%3$s>%4$s</a>', esc_url( $author_url ), $link_title, $link_class, $link_text );
     1193                                        $link_class = ' class="bbp-author-' . esc_attr( $link ) . '"';
     1194                                        $links[]    = sprintf( '<a href="%1$s"%2$s%3$s>%4$s</a>', esc_url( $author_url ), $link_title, $link_class, $link_text );
    11851195                                }
    11861196
     1197                                // Role is not linked
    11871198                                if ( true === $r['show_role'] ) {
    1188                                         $author_link[] = bbp_get_reply_author_role( array( 'reply_id' => $reply_id ) );
     1199                                        $links[] = bbp_get_reply_author_role( array( 'reply_id' => $reply_id ) );
    11891200                                }
    11901201
    1191                                 $author_link = implode( $r['sep'], $author_link );
    1192 
    1193                         // No links if anonymous
    1194                         } else {
    1195                                 $author_link = implode( $r['sep'], $author_links );
     1202                                // Juggle
     1203                                $author_links = $links;
     1204                                unset( $links );
    11961205                        }
    11971206
    1198                 // No replies so link is empty
    1199                 } else {
    1200                         $author_link = '';
     1207                        // Filter sections
     1208                        $sections    = apply_filters( 'bbp_get_reply_author_links', $author_links, $r, $args );
     1209
     1210                        // Assemble sections into author link
     1211                        $author_link = implode( $r['sep'], $sections );
    12011212                }
    12021213

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

    r6583 r6591  
    13971397                        'type'       => 'both',
    13981398                        'size'       => 80,
    1399                         'sep'        => '&nbsp;',
     1399                        'sep'        => '',
    14001400                        'show_role'  => false
    14011401                ), 'get_topic_author_link' );
    14021402
     1403                // Default return value
     1404                $author_link = '';
     1405
    14031406                // Used as topic_id
    1404                 if ( is_numeric( $args ) ) {
    1405                         $topic_id = bbp_get_topic_id( $args );
    1406                 } else {
    1407                         $topic_id = bbp_get_topic_id( $r['post_id'] );
    1408                 }
     1407                $topic_id = is_numeric( $args )
     1408                        ? bbp_get_topic_id( $args )
     1409                        : bbp_get_topic_id( $r['post_id'] );
    14091410
    14101411                // Topic ID is good
    … …  
    14171418                        // Tweak link title if empty
    14181419                        if ( empty( $r['link_title'] ) ) {
    1419                                 $title      = empty( $anonymous ) ? __( 'View %s\'s profile', 'bbpress' ) : __( 'Visit %s\'s website', 'bbpress' );
    14201420                                $author     = bbp_get_topic_author_display_name( $topic_id );
     1421                                $title      = empty( $anonymous )
     1422                                        ? __( 'View %s\'s profile',  'bbpress' )
     1423                                        : __( 'Visit %s\'s website', 'bbpress' );
    14211424                                $link_title = sprintf( $title, $author );
    14221425
    … …  
    14271430
    14281431                        // Setup title and author_links array
    1429                         $link_title   = ! empty( $link_title ) ? ' title="' . esc_attr( $link_title ) . '"' : '';
    14301432                        $author_links = array();
    1431 
    1432                         // Get avatar
    1433                         if ( 'avatar' === $r['type'] || 'both' === $r['type'] ) {
     1433                        $link_title   = ! empty( $link_title )
     1434                                ? ' title="' . esc_attr( $link_title ) . '"'
     1435                                : '';
     1436
     1437                        // Get avatar (unescaped, because HTML)
     1438                        if ( ( 'avatar' === $r['type'] ) || ( 'both' === $r['type'] ) ) {
    14341439                                $author_links['avatar'] = bbp_get_topic_author_avatar( $topic_id, $r['size'] );
    14351440                        }
    14361441
    1437                         // Get display name
    1438                         if ( 'name' === $r['type'] || 'both' === $r['type'] ) {
    1439                                 $author_links['name'] = bbp_get_topic_author_display_name( $topic_id );
     1442                        // Get display name (escaped, because never HTML)
     1443                        if ( ( 'name' === $r['type'] ) || ( 'both' === $r['type'] ) ) {
     1444                                $author_links['name'] = esc_html( bbp_get_topic_author_display_name( $topic_id ) );
    14401445                        }
    14411446
    … …  
    14451450                        // Add links if not anonymous
    14461451                        if ( empty( $anonymous ) && bbp_user_has_profile( bbp_get_topic_author_id( $topic_id ) ) ) {
     1452
     1453                                // Empty array
     1454                                $links = array();
    14471455
    14481456                                // Assemble the links
    14491457                                foreach ( $author_links as $link => $link_text ) {
    14501458                                        $link_class = ' class="bbp-author-' . esc_attr( $link ) . '"';
    1451                                         $author_link[] = sprintf( '<a href="%1$s"%2$s%3$s>%4$s</a>', esc_url( $author_url ), $link_title, $link_class, $link_text );
     1459                                        $links[]    = sprintf( '<a href="%1$s"%2$s%3$s>%4$s</a>', esc_url( $author_url ), $link_title, $link_class, $link_text );
    14521460                                }
    14531461
     1462                                // Role is not linked
    14541463                                if ( true === $r['show_role'] ) {
    1455                                         $author_link[] = bbp_get_topic_author_role( array( 'topic_id' => $topic_id ) );
     1464                                        $links[] = bbp_get_topic_author_role( array( 'topic_id' => $topic_id ) );
    14561465                                }
    14571466
    1458                                 $author_link = implode( $r['sep'], $author_link );
    1459 
    1460                         // No links if anonymous
    1461                         } else {
    1462                                 $author_link = implode( $r['sep'], $author_links );
     1467                                // Juggle
     1468                                $author_links = $links;
     1469                                unset( $links );
    14631470                        }
    14641471
    1465                 } else {
    1466                         $author_link = '';
    1467                 }
    1468 
    1469                 // Filter & return
    1470                 return apply_filters( 'bbp_get_topic_author_link', $author_link, $args );
     1472                        // Filter sections
     1473                        $sections    = apply_filters( 'bbp_get_topic_author_links', $author_links, $r, $args );
     1474
     1475                        // Assemble sections into author link
     1476                        $author_link = implode( $r['sep'], $sections );
     1477                }
     1478
     1479                // Filter & return
     1480                return apply_filters( 'bbp_get_topic_author_link', $author_link, $r, $args );
    14711481        }
    14721482

  • trunk/src/templates/default/bbpress/content-single-topic-lead.php

    r6258 r6591  
    5353                                <?php do_action( 'bbp_theme_before_topic_author_details' ); ?>
    5454
    55                                 <?php bbp_topic_author_link( array( 'sep' => '<br />', 'show_role' => true ) ); ?>
     55                                <?php bbp_topic_author_link( array( 'show_role' => true ) ); ?>
    5656
    5757                                <?php if ( current_user_can( 'moderate', bbp_get_reply_id() ) ) : ?>

  • trunk/src/templates/default/bbpress/loop-search-reply.php

    r6258 r6591  
    3030                <?php do_action( 'bbp_theme_before_reply_author_details' ); ?>
    3131
    32                 <?php bbp_reply_author_link( array( 'sep' => '<br />', 'show_role' => true ) ); ?>
     32                <?php bbp_reply_author_link( array( 'show_role' => true ) ); ?>
    3333
    3434                <?php if ( bbp_is_user_keymaster() ) : ?>

  • trunk/src/templates/default/bbpress/loop-search-topic.php

    r6258 r6591  
    5353                <?php do_action( 'bbp_theme_before_topic_author_details' ); ?>
    5454
    55                 <?php bbp_topic_author_link( array( 'sep' => '<br />', 'show_role' => true ) ); ?>
     55                <?php bbp_topic_author_link( array( 'show_role' => true ) ); ?>
    5656
    5757                <?php if ( bbp_is_user_keymaster() ) : ?>

  • trunk/src/templates/default/bbpress/loop-single-reply.php

    r6258 r6591  
    4242                <?php do_action( 'bbp_theme_before_reply_author_details' ); ?>
    4343
    44                 <?php bbp_reply_author_link( array( 'sep' => '<br />', 'show_role' => true ) ); ?>
     44                <?php bbp_reply_author_link( array( 'show_role' => true ) ); ?>
    4545
    4646                <?php if ( current_user_can( 'moderate', bbp_get_reply_id() ) ) : ?>
Note: See TracChangeset for help on using the changeset viewer.