WordPress.org

Make WordPress Core

Ticket #22356: 22356.6.diff

File 22356.6.diff, 9.7 KB (added by DH-Shredder, 17 months ago)

22356.3.2.diff​, with get_instance() now accepting a discrete $path argument as previously.

  • wp-admin/includes/media.php

    diff --git wp-admin/includes/media.php wp-admin/includes/media.php
    index 478dfdd..71c9c48 100644
    function get_media_item( $attachment_id, $args = null ) { 
    11221122        $media_dims = apply_filters( 'media_meta', $media_dims, $post ); 
    11231123 
    11241124        $image_edit_button = ''; 
    1125         if ( gd_edit_image_support( $post->post_mime_type ) ) { 
     1125        if ( WP_Image_Editor::supports( array( 'mime_type' => $post->post_mime_type ) ) ) { 
    11261126                $nonce = wp_create_nonce( "image_editor-$post->ID" ); 
    11271127                $image_edit_button = "<input type='button' id='imgedit-open-btn-$post->ID' onclick='imageEdit.open( $post->ID, \"$nonce\" )' class='button' value='" . esc_attr__( 'Edit Image' ) . "' /> <span class='spinner'></span>"; 
    11281128        } 
    function edit_form_image_editor() { 
    22592259        $att_url = wp_get_attachment_url( $post->ID ); 
    22602260 
    22612261        $image_edit_button = ''; 
    2262         if ( gd_edit_image_support( $post->post_mime_type ) ) { 
     2262        if ( WP_Image_Editor::supports( array( 'mime_type' => $post->post_mime_type ) ) ) { 
    22632263                $nonce = wp_create_nonce( "image_editor-$post->ID" ); 
    22642264                $image_edit_button = "<input type='button' id='imgedit-open-btn-$post->ID' onclick='imageEdit.open( $post->ID, \"$nonce\" )' class='button' value='" . esc_attr__( 'Edit Image' ) . "' /> <span class='spinner'></span>"; 
    22652265        } 
  • wp-includes/class-wp-image-editor-gd.php

    diff --git wp-includes/class-wp-image-editor-gd.php wp-includes/class-wp-image-editor-gd.php
    index ca76006..c52735a 100644
    class WP_Image_Editor_GD extends WP_Image_Editor { 
    100100         * @return boolean 
    101101         */ 
    102102        public static function supports_mime_type( $mime_type ) { 
    103                 $allowed_mime_types = array( 'image/gif', 'image/png', 'image/jpeg' ); 
     103                $image_types = imagetypes(); 
     104                switch( $mime_type ) { 
     105                        case 'image/jpeg': 
     106                                return ($image_types & IMG_JPG) != 0; 
     107                        case 'image/png': 
     108                                return ($image_types & IMG_PNG) != 0; 
     109                        case 'image/gif': 
     110                                return ($image_types & IMG_GIF) != 0; 
     111                } 
    104112 
    105                 return in_array( $mime_type, $allowed_mime_types ); 
     113                return false; 
    106114        } 
    107115 
    108116        /** 
    class WP_Image_Editor_GD extends WP_Image_Editor { 
    261269         * @since 3.5.0 
    262270         * @access public 
    263271         * 
    264          * @param boolean $horz Horizonal Flip 
     272         * @param boolean $horz Horizontal Flip 
    265273         * @param boolean $vert Vertical Flip 
    266274         * @returns boolean|WP_Error 
    267275         */ 
  • wp-includes/class-wp-image-editor-imagick.php

    diff --git wp-includes/class-wp-image-editor-imagick.php wp-includes/class-wp-image-editor-imagick.php
    index 601b99b..0edcde0 100644
    class WP_Image_Editor_Imagick extends WP_Image_Editor { 
    147147         * @return boolean 
    148148         */ 
    149149        public static function supports_mime_type( $mime_type ) { 
    150                 if ( ! $mime_type ) 
    151                         return false; 
    152  
    153150                $imagick_extension = strtoupper( self::get_extension( $mime_type ) ); 
    154151 
     152                if ( ! $imagick_extension ) 
     153                        return false; 
     154 
    155155                try { 
    156156                        return ( (bool) Imagick::queryFormats( $imagick_extension ) ); 
    157157                } 
    class WP_Image_Editor_Imagick extends WP_Image_Editor { 
    312312         * @since 3.5.0 
    313313         * @access public 
    314314         * 
    315          * @param boolean $horz Horizonal Flip 
     315         * @param boolean $horz Horizontal Flip 
    316316         * @param boolean $vert Vertical Flip 
    317317         * @returns boolean|WP_Error 
    318318         */ 
  • wp-includes/class-wp-image-editor.php

    diff --git wp-includes/class-wp-image-editor.php wp-includes/class-wp-image-editor.php
    index 920e4a4..9474a44 100644
    abstract class WP_Image_Editor { 
    2929         * @access public 
    3030         * 
    3131         * @param string $path Path to File to Load 
     32         * @param string|array $args Array of requirements.  Accepts { 'mime_type'=>string, 'methods'=>{string, string, ...} } 
    3233         * @param array $required_methods Methods to require in implementation 
    3334         * @return WP_Image_Editor|WP_Error 
    3435         */ 
    35         public final static function get_instance( $path = null, $required_methods = null ) { 
    36                 $implementation = apply_filters( 'wp_image_editor_class', self::choose_implementation( $required_methods ), $path ); 
     36        public final static function get_instance( $path, $args = array() ) { 
     37                $args['path'] = $path; 
     38 
     39                $implementation = apply_filters( 'wp_image_editor_class', self::choose_implementation( $args ) ); 
    3740 
    3841                if ( $implementation ) { 
    39                         $editor = new $implementation( $path ); 
     42                        $editor = new $implementation( $args['path'] ); 
    4043                        $loaded = $editor->load(); 
    4144 
    4245                        if ( is_wp_error( $loaded ) ) 
    abstract class WP_Image_Editor { 
    4548                        return $editor; 
    4649                } 
    4750 
    48                 return new WP_Error( 'no_editor', __('No editor could be selected') ); 
     51                return new WP_Error( 'image_no_editor', __('No editor could be selected.') ); 
    4952        } 
    5053 
    5154        /** 
    abstract class WP_Image_Editor { 
    5457         * @since 3.5.0 
    5558         * @access private 
    5659         * 
    57          * @param array $required_methods String array of all methods required for implementation returned. 
     60         * @param array $args Array of requirements. Accepts { 'path'=>string, 'mime_type'=>string, 'methods'=>{string, string, ...} } 
    5861         * @return string|bool Class name for the first editor that claims to support the request. False if no editor claims to support the request. 
    5962         */ 
    60         private final static function choose_implementation( $required_methods = null ) { 
     63        private final static function choose_implementation( $args = array() ) { 
    6164                $request_order = apply_filters( 'wp_image_editors', 
    6265                        array( 'WP_Image_Editor_Imagick', 'WP_Image_Editor_GD' ) ); 
    6366 
    64                 if ( ! $required_methods ) 
    65                         $required_methods = array(); 
     67                if ( ! isset( $args['mime_type'] ) && isset( $args['path'] ) ) { 
     68                        $file_info  = wp_check_filetype( $args['path'] ); 
     69 
     70                        // If $file_info['type'] is false, then we let the editor attempt to 
     71                        // figure out the file type, rather than forcing a failure based on extension. 
     72                        if ( isset( $file_info ) && $file_info['type'] ) 
     73                                $args['mime_type'] = $file_info['type']; 
     74                } 
    6675 
    6776                // Loop over each editor on each request looking for one which will serve this request's needs 
    6877                foreach ( $request_order as $editor ) { 
    6978                        // Check to see if this editor is a possibility, calls the editor statically 
    70                         if ( ! call_user_func( array( $editor, 'test' ) ) ) 
     79                        if ( ! call_user_func( array( $editor, 'test' ), $args ) ) 
     80                                continue; 
     81 
     82                        if ( isset( $args['mime_type'] ) && 
     83                                ! call_user_func( 
     84                                        array( $editor, 'supports_mime_type' ), 
     85                                        $args['mime_type'] ) ) { 
    7186                                continue; 
     87                        } 
    7288 
    7389                        // Make sure that all methods are supported by editor. 
    74                         if ( array_diff( $required_methods, get_class_methods( $editor ) ) ) 
     90                        if ( isset( $args['methods'] ) && 
     91                                 array_diff( $args['methods'], get_class_methods( $editor ) ) ) { 
    7592                                continue; 
     93                        } 
    7694 
    7795                        return $editor; 
    7896                } 
     97 
    7998                return false; 
    8099        } 
    81100 
    82101        /** 
     102         * Tests whether there is an editor that supports a given mime type or methods. 
     103         * 
     104         * @since 3.5.0 
     105         * @access public 
     106         * 
     107         * @param string|array $args Array of requirements.  Accepts { 'path'=>string, 'mime_type'=>string, 'methods'=>{string, string, ...} } 
     108         * @return boolean true if an eligible editor is found; false otherwise 
     109         */ 
     110        public final static function supports( $args = array() ) { 
     111                return ( (bool) self::choose_implementation( $args ) ); 
     112        } 
     113 
     114        /** 
    83115         * Loads image from $this->file into editor. 
    84116         * 
    85117         * @since 3.5.0 
    abstract class WP_Image_Editor { 
    168200         * @access public 
    169201         * @abstract 
    170202         * 
    171          * @param boolean $horz Horizonal Flip 
     203         * @param boolean $horz Horizontal Flip 
    172204         * @param boolean $vert Vertical Flip 
    173205         * @return boolean|WP_Error 
    174206         */ 
  • wp-includes/deprecated.php

    diff --git wp-includes/deprecated.php wp-includes/deprecated.php
    index 9e99614..5b841df 100644
    function user_pass_ok($user_login, $user_pass) { 
    33283328 * @since 2.3.0 
    33293329 * @deprecated 3.5.0 
    33303330 */ 
    3331 function _save_post_hook() {} 
    3332  No newline at end of file 
     3331function _save_post_hook() {} 
     3332 
     3333/** 
     3334 * Check if the installed version of GD supports particular image type 
     3335 * 
     3336 * @since 2.9.0 
     3337 * @deprecated 3.5.0 
     3338 * @see WP_Image_Editor::supports() 
     3339 * 
     3340 * @param string $mime_type 
     3341 * @return bool 
     3342 */ 
     3343function gd_edit_image_support($mime_type) { 
     3344        _deprecated_function( __FUNCTION__, '3.5', 'WP_Image_Editor::supports()' ); 
     3345        if ( function_exists('imagetypes') ) { 
     3346                switch( $mime_type ) { 
     3347                        case 'image/jpeg': 
     3348                                return (imagetypes() & IMG_JPG) != 0; 
     3349                        case 'image/png': 
     3350                                return (imagetypes() & IMG_PNG) != 0; 
     3351                        case 'image/gif': 
     3352                                return (imagetypes() & IMG_GIF) != 0; 
     3353                } 
     3354        } else { 
     3355                switch( $mime_type ) { 
     3356                        case 'image/jpeg': 
     3357                                return function_exists('imagecreatefromjpeg'); 
     3358                        case 'image/png': 
     3359                                return function_exists('imagecreatefrompng'); 
     3360                        case 'image/gif': 
     3361                                return function_exists('imagecreatefromgif'); 
     3362                } 
     3363        } 
     3364        return false; 
     3365} 
     3366 No newline at end of file 
  • wp-includes/media.php

    diff --git wp-includes/media.php wp-includes/media.php
    index bafb0a4..5f9385d 100644
    function get_taxonomies_for_attachments( $output = 'names' ) { 
    902902} 
    903903 
    904904/** 
    905  * Check if the installed version of GD supports particular image type 
    906  * 
    907  * @since 2.9.0 
    908  * 
    909  * @param string $mime_type 
    910  * @return bool 
    911  */ 
    912 function gd_edit_image_support($mime_type) { 
    913         if ( function_exists('imagetypes') ) { 
    914                 switch( $mime_type ) { 
    915                         case 'image/jpeg': 
    916                                 return (imagetypes() & IMG_JPG) != 0; 
    917                         case 'image/png': 
    918                                 return (imagetypes() & IMG_PNG) != 0; 
    919                         case 'image/gif': 
    920                                 return (imagetypes() & IMG_GIF) != 0; 
    921                 } 
    922         } else { 
    923                 switch( $mime_type ) { 
    924                         case 'image/jpeg': 
    925                                 return function_exists('imagecreatefromjpeg'); 
    926                         case 'image/png': 
    927                                 return function_exists('imagecreatefrompng'); 
    928                         case 'image/gif': 
    929                                 return function_exists('imagecreatefromgif'); 
    930                 } 
    931         } 
    932         return false; 
    933 } 
    934  
    935 /** 
    936905 * Create new GD image resource with transparency support 
    937906 * @TODO: Deprecate if possible. 
    938907 *