Ticket #13411: transparancy.patch

wp-admin/includes/image.php

@@ -41 +41 @@
  * @param int $dst_h The destination height.
  * @param int $src_abs Optional. If the source crop points are absolute.
  * @param string $dst_file Optional. The destination file to write to.
+ * @param bool $dst_transparency Optional. The destination file type. Transparency = PNG.
  * @return string|WP_Error|false New filepath on success, WP_Error or false on failure.
  */
-function wp_crop_image( $src_file, $src_x, $src_y, $src_w, $src_h, $dst_w, $dst_h, $src_abs = false, $dst_file = false ) {
+function wp_crop_image( $src_file, $src_x, $src_y, $src_w, $src_h, $dst_w, $dst_h, $src_abs = false, $dst_file = false, $dst_transparency = false ) {
         if ( is_numeric( $src_file ) ) // Handle int as attachment ID
                 $src_file = get_attached_file( $src_file );
 
@@ -63 +64 @@
                 imageantialias( $dst, true );
 
         imagecopyresampled( $dst, $src, 0, 0, $src_x, $src_y, $dst_w, $dst_h, $src_w, $src_h );
-
         imagedestroy( $src ); // Free up memory
 
         if ( ! $dst_file )
                 $dst_file = str_replace( basename( $src_file ), 'cropped-' . basename( $src_file ), $src_file );
 
-        $dst_file = preg_replace( '/\\.[^\\.]+$/', '.jpg', $dst_file );
+        $type = '.jpeg';
+        if ( $dst_transparency )
+                $type = '.png';
 
-        if ( imagejpeg( $dst, $dst_file, apply_filters( 'jpeg_quality', 90, 'wp_crop_image' ) ) )
+        $dst_file = preg_replace( '/\\.[^\\.]+$/', $type, $dst_file );
+
+        if ( $dst_transparency && imagepng( $dst, $dst_file) )
                 return $dst_file;
+        elseif ( imagejpeg( $dst, $dst_file) )
+                return $dst_file;
         else
                 return false;
 }