Changeset 50258 for trunk/src/wp-admin/includes/media.php

Timestamp:

02/08/2021 11:20:56 PM (4 years ago)

Author:

antpb

Message:

Media: Allow post_date to be respected in media_handle_sideload().

Previously, date information was unable to be changed when using media_handle_sideload().

Now you can override the date for a media item using $post_data['post_date'] before using the function.

Props jamesgol, mukesh27, SergeyBiryukov, hellofromTonya, Mista-Flo.
Fixes #50972.

File:

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

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

@@ -431 +431 @@
  * @since 5.3.0 The `$post_id` parameter was made optional.
  *
- * @param array  $file_array Array that represents a `$_FILES` upload array.
- * @param int    $post_id    Optional. The post ID the media is associated with.
- * @param string $desc       Optional. Description of the side-loaded file. Default null.
- * @param array  $post_data  Optional. Post data to override. Default empty array.
+ * @param string[]  $file_array Array that represents a `$_FILES` upload array.
+ * @param int       $post_id    Optional. The post ID the media is associated with.
+ * @param string    $desc       Optional. Description of the side-loaded file. Default null.
+ * @param array     $post_data  Optional. Post data to override. Default empty array.
  * @return int|WP_Error The ID of the attachment or a WP_Error on failure.
  */
@@ -440 +440 @@
     $overrides = array( 'test_form' => false );
 
-    $time = current_time( 'mysql' );
-    $post = get_post( $post_id );
-
-    if ( $post ) {
-        if ( substr( $post->post_date, 0, 4 ) > 0 ) {
+    if ( isset( $post_data['post_date'] ) && substr( $post_data['post_date'], 0, 4 ) > 0 ) {
+        $time = $post_data['post_date'];
+    } else {
+        $post = get_post( $post_id );
+        if ( $post && substr( $post->post_date, 0, 4 ) > 0 ) {
             $time = $post->post_date;
+        } else {
+            $time = current_time( 'mysql' );
         }
     }