Changeset 49979 for trunk

Timestamp:

01/18/2021 12:53:30 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Media: Use consistent method for instantiating an attachment author object in Media Library.

Previously, attachments without an author could cause a PHP fatal error due to calling the ::exists() method on a false value.

Props antpb, carloscastilloadhoc, hellofromTonya, garrett-eclipse.
Fixes #52030.

Location:

trunk/src

Files:

• wp-admin/includes/media.php (modified) (1 diff)
• wp-includes/media.php (modified) (1 diff)

trunk/src/wp-admin/includes/media.php

@@ -3289 +3289 @@
     $att_url = wp_get_attachment_url( $attachment_id );
 
-    $author = get_userdata( $post->post_author );
+    $author = new WP_User( $post->post_author );
 
     $uploaded_by_name = __( '(no author)' );
     $uploaded_by_link = '';
+
     if ( $author->exists() ) {
         $uploaded_by_name = $author->display_name ? $author->display_name : $author->nickname;

trunk/src/wp-includes/media.php

@@ -3930 +3930 @@
 
     $author = new WP_User( $attachment->post_author );
+
     if ( $author->exists() ) {
         $author_name            = $author->display_name ? $author->display_name : $author->nickname;