Changeset 6480

Timestamp:

01/31/2018 12:24:25 AM (7 years ago)

Author:

danielbachhuber

Message:

phpunit-test-reporter: Display reporter avatars in the test results

See ​https://github.com/WordPress/phpunit-test-reporter/pull/55

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/parts

Files:

• result-set.php (modified) (1 diff)
• single-result.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/parts/result-set.php

@@ -44 +44 @@
                     $user = get_user_by( 'id', $report->post_author );
                     if ( $user ) {
-                        $host = $user->display_name;
+                        $host = '';
+                        if ( ! empty( $user->user_url ) ) {
+                            $host .= '<a target="_blank" rel="nofollow" href="' . esc_url( $user->user_url ) . '">';
+                        }
+                        $host .= get_avatar( $user->ID, 18, '', '', array(
+                            'extra_attr' => 'style="vertical-align: middle;margin-right:5px;"',
+                        ) );
+                        if ( ! empty( $user->user_url ) ) {
+                            $host .= '</a>';
+                        }
+                        if ( ! empty( $user->user_url ) ) {
+                            $host .= '<a target="_blank" rel="nofollow" href="' . esc_url( $user->user_url ) . '">';
+                        }
+                        $host .= $user->display_name;
+                        if ( ! empty( $user->user_url ) ) {
+                            $host .= '</a>';
+                        }
                     }
                     ?>
                 <tr>
                     <td><a href="<?php echo esc_url( get_permalink( $report->ID ) ); ?>" title="<?php echo esc_attr( $status_title ); ?>" class="<?php echo esc_attr( 'ptr-status-badge ptr-status-badge-' . strtolower( $status ) ); ?>"><?php echo esc_html( $status ); ?></a></td>
-                    <td><?php echo esc_html( $host ); ?></td>
+                    <td><?php echo wp_kses_post( $host ); ?></td>
                     <td><?php echo esc_html( Display::get_display_php_version( $report->ID ) ); ?></td>
                     <td><?php echo esc_html( Display::get_display_mysql_version( $report->ID ) ); ?></td>

sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/parts/single-result.php

@@ -12 +12 @@
 $user = get_user_by( 'id', $report->post_author );
 if ( $user ) {
-    $host = $user->display_name;
+    $host = '';
+    if ( ! empty( $user->user_url ) ) {
+        $host .= '<a target="_blank" rel="nofollow" href="' . esc_url( $user->user_url ) . '">';
+    }
+    $host .= get_avatar( $user->ID, 18, '', '', array(
+        'extra_attr' => 'style="vertical-align: middle;margin-right:5px;"',
+    ) );
+    if ( ! empty( $user->user_url ) ) {
+        $host .= '</a>';
+    }
+    if ( ! empty( $user->user_url ) ) {
+        $host .= '<a target="_blank" rel="nofollow" href="' . esc_url( $user->user_url ) . '">';
+    }
+    $host .= $user->display_name;
+    if ( ! empty( $user->user_url ) ) {
+        $host .= '</a>';
+    }
 } ?>
 
@@ -31 +47 @@
     <tr>
         <td><strong>Host</strong></td>
-        <td><?php echo esc_html( $host ); ?></td>
+        <td><?php echo wp_kses_post( $host ); ?></td>
     </tr>
     <tr>