Changeset 58130

Timestamp:

05/10/2024 06:57:53 PM (3 months ago)

Author:

swissspidy

Message:

REST API: Ensure attachments are uploaded to the post's year/month folder.

If organizing uploads into month- and year-based folders, uploading an attachment to an existing post should store the file in wp-content/uploads/<year>/<month> based on the post's publish date. This is in line with the behavior in classic editor / the media modal.

Props swissspidy, adamsilverstein, timothyblynjacobs, skithund, sergeybiryukov, patricia70.
Fixes #61189.

Location:

trunk

Files:

• src/wp-admin/includes/file.php (modified) (2 diffs)
• src/wp-includes/functions.php (modified) (3 diffs)
• src/wp-includes/rest-api/endpoints/class-wp-rest-attachments-controller.php (modified) (5 diffs)
• tests/phpunit/tests/rest-api/rest-attachments-controller.php (modified) (1 diff)

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

@@ -1090 +1090 @@
  *                               to override default variables. Default false.
  *                               See _wp_handle_upload() for accepted values.
- * @param string      $time      Optional. Time formatted in 'yyyy/mm'. Default null.
+ * @param string $time      Optional. Time formatted in 'yyyy/mm'. Default null.
  * @return array See _wp_handle_upload() for return value.
  */
@@ -1121 +1121 @@
  *                               to override default variables. Default false.
  *                               See _wp_handle_upload() for accepted values.
- * @param string      $time      Optional. Time formatted in 'yyyy/mm'. Default null.
+ * @param string $time      Optional. Time formatted in 'yyyy/mm'. Default null.
  * @return array See _wp_handle_upload() for return value.
  */

trunk/src/wp-includes/functions.php

@@ -2345 +2345 @@
  * @uses _wp_upload_dir()
  *
- * @param string $time Optional. Time formatted in 'yyyy/mm'. Default null.
- * @param bool   $create_dir Optional. Whether to check and create the uploads directory.
- *                           Default true for backward compatibility.
- * @param bool   $refresh_cache Optional. Whether to refresh the cache. Default false.
+ * @param string Optional. Time formatted in 'yyyy/mm'. Default null.
+ * @param bool    Optional. Whether to check and create the uploads directory.
+ *                           Default true for backward compatibility.
+ * @param bool   $refresh_cache Optional. Whether to refresh the cache. Default false.
  * @return array {
  *     Array of information about the upload directory.
@@ -2420 +2420 @@
  * @access private
  *
- * @param string $time Optional. Time formatted in 'yyyy/mm'. Default null.
+ * @param string $time Optional. Time formatted in 'yyyy/mm'. Default null.
  * @return array See wp_upload_dir()
  */
@@ -2872 +2872 @@
  * @param null|string $deprecated Never used. Set to null.
  * @param string      $bits       File content
- * @param string      $time       Optional. Time formatted in 'yyyy/mm'. Default null.
+ * @param string $time       Optional. Time formatted in 'yyyy/mm'. Default null.
  * @return array {
  *     Information about the newly-uploaded file.

trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-attachments-controller.php

@@ -255 +255 @@
         $headers = $request->get_headers();
 
+
+
+
+
+
+
+
+
+
+
+
         if ( ! empty( $files ) ) {
-            $file = $this->upload_from_file( $files, $headers );
+            $file = $this->upload_from_file( $files, $headers );
         } else {
-            $file = $this->upload_from_data( $request->get_body(), $headers );
+            $file = $this->upload_from_data( $request->get_body(), $headers );
         }
 
@@ -1036 +1047 @@
      *
      * @since 4.7.0
-     *
-     * @param string $data    Supplied file data.
-     * @param array  $headers HTTP headers from the request.
+     * @since 6.6.0 Added the `$time` parameter.
+     *
+     * @param string      $data    Supplied file data.
+     * @param array       $headers HTTP headers from the request.
+     * @param string|null $time    Optional. Time formatted in 'yyyy/mm'. Default null.
      * @return array|WP_Error Data from wp_handle_sideload().
      */
-    protected function upload_from_data( $data, $headers ) {
+    protected function upload_from_data( $data, $headers ) {
         if ( empty( $data ) ) {
             return new WP_Error(
@@ -1129 +1142 @@
         );
 
-        $sideloaded = wp_handle_sideload( $file_data, $overrides );
+        $sideloaded = wp_handle_sideload( $file_data, $overrides );
 
         if ( isset( $sideloaded['error'] ) ) {
@@ -1247 +1260 @@
      *
      * @since 4.7.0
-     *
-     * @param array $files   Data from the `$_FILES` superglobal.
-     * @param array $headers HTTP headers from the request.
+     * @since 6.6.0 Added the `$time` parameter.
+     *
+     * @param array       $files   Data from the `$_FILES` superglobal.
+     * @param array       $headers HTTP headers from the request.
+     * @param string|null $time    Optional. Time formatted in 'yyyy/mm'. Default null.
      * @return array|WP_Error Data from wp_handle_upload().
      */
-    protected function upload_from_file( $files, $headers ) {
+    protected function upload_from_file( $files, $headers ) {
         if ( empty( $files ) ) {
             return new WP_Error(
@@ -1294 +1309 @@
         require_once ABSPATH . 'wp-admin/includes/file.php';
 
-        $file = wp_handle_upload( $files['file'], $overrides );
+        $file = wp_handle_upload( $files['file'], $overrides );
 
         if ( isset( $file['error'] ) ) {

trunk/tests/phpunit/tests/rest-api/rest-attachments-controller.php

@@ -2076 +2076 @@
     }
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
     public function filter_rest_insert_attachment( $attachment ) {
         ++self::$rest_insert_attachment_count;