Changeset 49542 for trunk/src/wp-includes/class-wp-image-editor-imagick.php

Timestamp:

11/09/2020 01:16:54 PM (6 years ago)

Author:

johnbillion

Message:

Media: Restore the ability of WP_Image_Editor_Imagick->save() to create a missing directory when needed.

Props eemitch, mikeschroder, hellofromTonya, p00ya, johnbillion

Fixes #51665

File:

• trunk/src/wp-includes/class-wp-image-editor-imagick.php (modified) (1 diff)

trunk/src/wp-includes/class-wp-image-editor-imagick.php

@@ -742 +742 @@
                         }
                 } else {
+                        $dir_name   = dirname( $filename );
+                        $dir_exists = wp_mkdir_p( $dir_name );
+
+                        if ( ! $dir_exists ) {
+                                return new WP_Error(
+                                        'image_save_error',
+                                        sprintf(
+                                                /* translators: %s: Directory path. */
+                                                __( 'Unable to create directory %s. Is its parent directory writable by the server?' ),
+                                                esc_html( $dir_name )
+                                        )
+                                );
+                        }
+
                         try {
                                 return $image->writeImage( $filename );