Changeset 35579

Timestamp:

11/09/2015 12:44:01 AM (9 years ago)

Author:

dd32

Message:

Media: Allow media_sideload_image() to work when the upload directory is a PHP Stream. Using copy() instead of rename() allows the function to work across different stream sources.
Props mattheu.
Fixes #29257

File:

• trunk/src/wp-admin/includes/file.php (modified) (1 diff)

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

@@ -346 +346 @@
         $move_new_file = @ move_uploaded_file( $file['tmp_name'], $new_file );
     } else {
-        $move_new_file = @ rename( $file['tmp_name'], $new_file );
+        // use copy and unlink because rename breaks streams.
+        $move_new_file = @ copy( $file['tmp_name'], $new_file );
+        unlink( $file['tmp_name'] );
     }