Changeset 49084 for trunk/src/wp-includes/post.php

Timestamp:

10/01/2020 05:53:38 PM (4 years ago)

Author:

helen

Message:

Media: Don't unnecessarily check for a valid attachment before getting meta.

This makes wp_get_attachment_metadata() run significantly faster. See ticket for benchmarking.

Props Tkama, donmhico.
Fixes #50679.

File:

• trunk/src/wp-includes/post.php (modified) (2 diffs)

trunk/src/wp-includes/post.php

@@ -5993 +5993 @@
     $attachment_id = (int) $attachment_id;
 
-    $post = get_post( $attachment_id );
-    if ( ! $post ) {
+    $data = get_post_meta( $attachment_id, '_wp_attachment_metadata', true );
+
+    if ( empty( $data ) ) {
         return false;
     }
-
-    $data = get_post_meta( $post->ID, '_wp_attachment_metadata', true );
 
     if ( $unfiltered ) {
@@ -6013 +6012 @@
      * @param int        $attachment_id Attachment post ID.
      */
-    return apply_filters( 'wp_get_attachment_metadata', $data, $post->ID );
+    return apply_filters( 'wp_get_attachment_metadata', $data, $attachment_id );
 }