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

Timestamp:

11/17/2017 05:04:56 PM (8 years ago)

Author:

johnjamesjacoby

Message:

Anonymous Users: improve styling for anonymous topic/reply author boxes.

This change wraps anonymous author data in span elements where a elements would normally be used for registered users, and tweaks the default template-pack CSS to accomodate these new elements.

File:

• trunk/src/includes/replies/template.php (modified) (1 diff)

trunk/src/includes/replies/template.php

@@ -1174 +1174 @@
             }
 
-            // Add links if not anonymous and existing user
-            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-' . 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'] ) {
-                    $links[] = bbp_get_reply_author_role( array( 'reply_id' => $reply_id ) );
-                }
-
-                // Juggle
-                $author_links = $links;
-                unset( $links );
+            // Empty array
+            $links  = array();
+            $sprint = empty( $anonymous ) && bbp_user_has_profile( bbp_get_reply_author_id( $reply_id ) )
+                ? '<a href="%1$s"%2$s%3$s>%4$s</a>'
+                : '<span %2$s%3$s>%4$s</span>';
+
+            // Wrap each link
+            foreach ( $author_links as $link => $link_text ) {
+                $link_class = ' class="bbp-author-' . esc_attr( $link ) . '"';
+                $links[]    = sprintf( $sprint, esc_url( $author_url ), $link_title, $link_class, $link_text );
             }
+
+            // Role is not linked
+            if ( true === $r['show_role'] ) {
+                $links[] = bbp_get_reply_author_role( array( 'reply_id' => $reply_id ) );
+            }
+
+            // Juggle
+            $author_links = $links;
+            unset( $links );
 
             // Filter sections