[wp-trac] [WordPress Trac] #22538: 'wp_image_editor_class' vs. 'wp_image_editors'

WordPress Trac noreply at wordpress.org
Thu Nov 22 17:50:04 UTC 2012


#22538: 'wp_image_editor_class' vs. 'wp_image_editors'
--------------------------+--------------------
 Reporter:  scribu        |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  3.5
Component:  Media         |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:  dev-feedback  |
--------------------------+--------------------

Comment (by markoheijnen):

 Replying to [comment:4 macbrink]:
 > If $args is passed, you could do with just one.
 >
 > Imho the class selection could be more elegant. It seems the reasoning
 behind this was to select between Imagick en GD, however, if a  plugin
 developer wants to add or change functionality, he has to extend both
 WP_Image_Editor_GD and WP_Image_Editor_Imagick. I would rather have one
 image editor class which internally checks the image library to use.

 I disagree with this. You can't mix GD and Imagick. Or at least you
 shouldn't so the current way of doing is best. Also there are more then
 two image libraries like jpegtran and gmagick.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/22538#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list