Let user choose between GD and ImageMagick

[fierevere]

Since WP 3.5 default image editor extension can be choosen between Imagick and old good GD, choice is done automatically after simple test and Imagick is preferred.
User cannot choose which extension they like.
Some servers have buggy and unusable Imagick installed, while GD works, it will be nice to let user disable Imagick via wp-config.php define.

Simple patch provided.

[fierevere]

simple patch for define('WP_DISABLE_IMAGICK')

[nacin]

Some servers have buggy and unusable Imagick installed

Could you provide some information here? Rather than patching the symptom we'd rather target the actual problem — that is, we may not be detecting a "buggy and unusable" version of Imagick.

It shouldn't be necessary for the user to even think about how their images are getting resized and manipulated. We try to ensure we are working with a good version of Imagick — we require 2.2.0+, for example.

This is already feasible via the wp_image_editors filter, as in a plugin can serve this role. I don't think it is worth adding a constant, unless we really can't solve for buggy setups.

[fierevere]

Actually a problem came from this topic - ​http://ru.forums.wordpress.org/topic/Изображения-загружаются-только-в-полном-размер/
(Russian)

brief: Images are not resized on upload, media editor tools (flip,resize etc) dont work
while phpinfo() shows a functional imagick extension, there is a problem with TEMP,TMP,TEMPDIR variables not set by hosting support (they are slow, so i tried to find some temporary workarounds), GD editor works, unless a JPEG image is too big (--enable-memlimit setting for libjpeg, when it tries to make a temporary file)
I expect a problem with Imagick ext is that TMP is not defined.
Wordpress had issues too, but it was solved by define( 'WP_TEMP_DIR', ABSPATH . '/tmp/') ;

imagick
imagick module	enabled
imagick module version 	3.0.1
imagick classes 	Imagick, ImagickDraw, ImagickPixel, ImagickPixelIterator
ImageMagick version 	ImageMagick 6.7.3-1 2011-10-12 Q16 http://www.imagemagick.org
ImageMagick copyright 	Copyright (C) 1999-2011 ImageMagick Studio LLC
ImageMagick release date 	2011-10-12
ImageMagick number of supported formats: 	197
ImageMagick supported formats 	3FR, A, AAI, AI, ART, ARW, AVI, AVS, B, BGR, BGRA, BMP, BMP2, BMP3, BRF, C, CAL, CALS, CANVAS, CAPTION, CIN, CIP, CLIP, CMYK, CMYKA, CR2, CRW, CUR, CUT, DCM, DCR, DCX, DDS, DFONT, DNG, DOT, DPX, EPDF, EPI, EPS, EPS2, EPS3, EPSF, EPSI, EPT, EPT2, EPT3, ERF, FAX, FITS, FRACTAL, FTS, G, G3, GIF, GIF87, GRADIENT, GRAY, GROUP4, HALD, HDR, HISTOGRAM, HRZ, HTM, HTML, ICB, ICO, ICON, INFO, INLINE, IPL, ISOBRL, JNG, JPEG, JPG, K, K25, KDC, LABEL, M, M2V, M4V, MAC, MAP, MAT, MATTE, MIFF, MNG, MONO, MOV, MP4, MPC, MPEG, MPG, MRW, MSL, MSVG, MTV, MVG, NEF, NULL, O, ORF, OTB, OTF, PAL, PALM, PAM, PATTERN, PBM, PCD, PCDS, PCL, PCT, PCX, PDB, PDF, PDFA, PEF, PES, PFA, PFB, PFM, PGM, PICON, PICT, PIX, PJPEG, PLASMA, PNG, PNG24, PNG32, PNG8, PNM, PPM, PREVIEW, PS, PS2, PS3, PSB, PSD, PTIF, PWP, R, RADIAL-GRADIENT, RAF, RAS, RGB, RGBA, RGBO, RLA, RLE, SCR, SCT, SFW, SGI, SHTML, SR2, SRF, STEGANO, SUN, SVG, SVGZ, TEXT, TGA, THUMBNAIL, TIFF, TIFF64, TILE, TIM, TTC, TTF, TXT, UBRL, UIL, UYVY, VDA, VICAR, VID, VIFF, VST, WBMP, WMV, WPG, X, X3F, XBM, XC, XCF, XPM, XPS, XV, XWD, Y, YCbCr, YCbCrA, YUV

Directive	Local Value	Master Value
imagick.locale_fix	0	0
imagick.progress_monitor	0	0

[nacin]

My initial questions would be:

• Which host is this?
• What exactly does phpinfo() reveal about tmp directories? upload_tmp_dir? TMPDIR? Other environment variables?
• Why does Imagick work but GD does not?
• Is there a way to specify the temporary location used by Imagick?
• Without WP_TEMP_DIR, what does get_temp_dir()? If things aren't writable, at the very least wp-content should be, and that's what we use before falling back to /tmp/.

[markoheijnen]

To me adding a define is a wontfix. There is a filter so we should use that one.

As nacin I'm also curious in this setup and see if we can fix something in core to make it more stable.

[fierevere]

Which host is this? -> jino.ru

upload_tmp_dir -> defined, working

environment variables -> they have only LANG and PATH, nothing else, not even HOSTNAME, HOME or USER

Why does Imagick work but GD does not? -> Imagick does NOT, GD works (unless not attempting to create tempfile)

I am not sure if /tmp is writeable on that server, they dont allow unrestricted ssh access and CPanel and FTP is limited to user's home. Theme and plugin installer wasnt working without define( 'WP_TEMP_DIR', ABSPATH . '/tmp/') ;

PS: they use mod_pizza (closed, proprietary) access controller for apache , it seem to restrict access to upper level folders than user's home

[fierevere]

Сергей, спасибо за решение с фильтрами, к сожалению в свете последних несправедливых событий, я не считаю то что какая-либо помощь сообществу принесет что-либо кроме черной неблагодарности, а посему деталей о Джино не будет.

Т.к. патч не примут - тикет закрываю.

[markoheijnen]

Clearing out the milestones for closed tickets on Awaiting Review

[miqrogroove]

Having the same problem in #39387 BlueHost Imagick is crashing every time I upload a GIF and I need some obvious way to disable the library!

[voldemortensen]

@miqrogroove Can you email wordpress at bluehost dot com with details about that? I'd like to take a look and get issues resolved across the board if this is wide spread.

[miqrogroove]

Will do.