I study your code atm.
I saw in _imagemagick_check_path() the following sequence:
// If only the name of the executable is given, we only check whether it is in
// the path and can be invoked.
if ($file != 'convert' && $file != 'gm') {
...
}
In my opinion that means that the string doesn't have to be 'convert' as well as 'gm'.
To produce this in code in my mind there should be a "||" instead "&&", isn't it?
Best wishes
maen
Closed: outdated
1.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.