Changeset 23374

Timestamp:

02/02/2013 02:01:59 AM (12 years ago)

Author:

SergeyBiryukov

Message:

Return WP_Error from wp_crop_image() if saving has failed. props macbrink. fixes #23325.

File:

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

trunk/wp-admin/includes/image.php

@@ -21 +21 @@
  * @param int $src_abs Optional. If the source crop points are absolute.
  * @param string $dst_file Optional. The destination file to write to.
- * @return string|WP_Error|false New filepath on success, WP_Error or false on failure.
+ * @return string|WP_Error New filepath on success, WP_Error on failure.
  */
 function wp_crop_image( $src, $src_x, $src_y, $src_w, $src_h, $dst_w, $dst_h, $src_abs = false, $dst_file = false ) {
@@ -55 +55 @@
 
     $result = $editor->save( $dst_file );
+    if ( is_wp_error( $result ) )
+        return $result;
+
     return $dst_file;
 }