Changeset 6591 for trunk/src/includes/replies/template.php

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

File:

• trunk/src/includes/replies/template.php (modified) (4 diffs)

trunk/src/includes/replies/template.php

@@ -1132 +1132 @@
                         'type'       => 'both',
                         'size'       => 80,
-                        'sep'        => ' ',
+                        'sep'        => '',
                         'show_role'  => false
                 ), 'get_reply_author_link' );
 
+                // Default return value
+                $author_link = '';
+
                 // Used as reply_id
-                if ( is_numeric( $args ) ) {
-                        $reply_id = bbp_get_reply_id( $args );
-                } else {
-                        $reply_id = bbp_get_reply_id( $r['post_id'] );
-                }
+                $reply_id = is_numeric( $args )
+                        ? bbp_get_reply_id( $args )
+                        : bbp_get_reply_id( $r['post_id'] );
 
                 // Reply ID is good
@@ -1152 +1153 @@
                         // Tweak link title if empty
                         if ( empty( $r['link_title'] ) ) {
-                                $link_title = sprintf( empty( $anonymous ) ? __( 'View %s\'s profile', 'bbpress' ) : __( 'Visit %s\'s website', 'bbpress' ), bbp_get_reply_author_display_name( $reply_id ) );
+                                $author     = bbp_get_reply_author_display_name( $reply_id );
+                                $title      = empty( $anonymous )
+                                        ? __( 'View %s\'s profile',  'bbpress' )
+                                        : __( 'Visit %s\'s website', 'bbpress' );
+                                $link_title = sprintf( $title, $author );
 
                         // Use what was passed if not
@@ -1160 +1165 @@
 
                         // Setup title and author_links array
-                        $link_title   = ! empty( $link_title ) ? ' title="' . esc_attr( $link_title ) . '"' : '';
                         $author_links = array();
-
-                        // Get avatar
-                        if ( 'avatar' === $r['type'] || 'both' === $r['type'] ) {
+                        $link_title   = ! empty( $link_title )
+                                ? ' title="' . esc_attr( $link_title ) . '"'
+                                : '';
+
+                        // Get avatar (unescaped, because HTML)
+                        if ( ( 'avatar' === $r['type'] ) || ( 'both' === $r['type'] ) ) {
                                 $author_links['avatar'] = bbp_get_reply_author_avatar( $reply_id, $r['size'] );
                         }
 
-                        // Get display name
-                        if ( 'name' === $r['type']   || 'both' === $r['type'] ) {
-                                $author_links['name'] = bbp_get_reply_author_display_name( $reply_id );
+                        // Get display name (escaped, because never HTML)
+                        if ( ( 'name' === $r['type'] ) || ( 'both' === $r['type'] ) ) {
+                                $author_links['name'] = esc_html( bbp_get_reply_author_display_name( $reply_id ) );
                         }
 
@@ -1179 +1186 @@
                         if ( empty( $anonymous ) && bbp_user_has_profile( bbp_get_reply_author_id( $reply_id ) ) ) {
 
+                                // Empty array
+                                $links = array();
+
                                 // Assemble the links
                                 foreach ( $author_links as $link => $link_text ) {
-                                        $link_class = ' class="bbp-author-' . $link . '"';
-                                        $author_link[] = sprintf( '<a href="%1$s"%2$s%3$s>%4$s</a>', esc_url( $author_url ), $link_title, $link_class, $link_text );
+                                        $link_class = ' class="bbp-author-' . esc_attr( $link ) . '"';
+                                        $links[]    = sprintf( '<a href="%1$s"%2$s%3$s>%4$s</a>', esc_url( $author_url ), $link_title, $link_class, $link_text );
                                 }
 
+                                // Role is not linked
                                 if ( true === $r['show_role'] ) {
-                                        $author_link[] = bbp_get_reply_author_role( array( 'reply_id' => $reply_id ) );
+                                        $links[] = bbp_get_reply_author_role( array( 'reply_id' => $reply_id ) );
                                 }
 
-                                $author_link = implode( $r['sep'], $author_link );
-
-                        // No links if anonymous
-                        } else {
-                                $author_link = implode( $r['sep'], $author_links );
+                                // Juggle
+                                $author_links = $links;
+                                unset( $links );
                         }
 
-                // No replies so link is empty
-                } else {
-                        $author_link = '';
+                        // Filter sections
+                        $sections    = apply_filters( 'bbp_get_reply_author_links', $author_links, $r, $args );
+
+                        // Assemble sections into author link
+                        $author_link = implode( $r['sep'], $sections );
                 }