Changeset 10436 for sites/trunk/wordpress.org/public_html/wp-content/plugins/trac-notifications/trac-notifications.php

Timestamp:

11/10/2020 04:10:17 AM (4 years ago)

Author:

dd32

Message:

Trac: Only output the 'First ticket' notice if the user exists. Some old tickets have users who no longer exist.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/trac-notifications/trac-notifications.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/trac-notifications/trac-notifications.php

@@ -360 +360 @@
         }
 
+        $reporter = get_user_by( 'login', $ticket['reporter'] );
+        if ( ! $reporter ) {
+            return;
+        }
+
         if ( 1 == count( $activity['tickets'] ) ) {
             $output = sprintf( '<strong>Make sure %s receives a warm welcome.</strong><br/>', $ticket['reporter'] );
@@ -383 +388 @@
 
         echo '<p class="ticket-note note-new-reporter">';
-        echo get_avatar( get_user_by( 'login', $ticket['reporter'] )->user_email, 36, 'retro' );
+        echo get_avatar( $reporter->user_email, 36, 'retro' );
         echo '<span class="note">' . $output . '</span>';
         echo '<span class="dashicons dashicons-welcome-learn-more"></span>';