Changeset 4200

Timestamp:

10/07/2016 06:17:36 AM (8 years ago)

Author:

dd32

Message:

Plugin Directory: Trial switching out the Author Byline to utilise the plugin header Author fields.

This improves the consistency with how WordPress itself displays plugins both in the plugin listing, and in the plugin installer.

See #1719.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-plugins/template-parts/plugin-single.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-plugins/template-parts/plugin-single.php

@@ -61 +61 @@
         <?php the_title( '<h1 class="plugin-title">', '</h1>' ); ?>
 
-        <span class="byline"><?php printf( esc_html_x( 'By %s', 'post author', 'wporg-plugins' ), '<span class="author vcard"><a class="url fn n" href="' . esc_url( get_author_posts_url( get_the_author_meta( 'ID' ) ) ) . '">' . esc_html( Template::encode( get_the_author() ) ) . '</a></span>' ); ?></span>
+        <span class="byline"><?php
+            $url = get_post_meta( get_the_ID(), 'header_author_uri', true );
+            $author = get_post_meta( get_the_ID(), 'header_author', true ) ?: get_the_author();
+
+            printf(
+                _x( 'By %s', 'post author', 'wporg-plugins' ),
+                '<span class="author vcard">' .
+                ( $url ? '<a class="url fn n" href="' . esc_url( $url ) . '">' : '' ) .
+                esc_html( Template::encode( $author ) ) .
+                ( $url ? '</a>' : '' ) .
+                '</span>'
+            );
+        ?></span>
     </header><!-- .entry-header -->