-
Notifications
You must be signed in to change notification settings - Fork 379
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Webp - Your PHP version is compiled without WebP support. #1322
Comments
thank you for the PR. I am holding off from merging as I first want to look into #1319 .. maybe you also can have a look as it relates to the just released WebP support. |
Sorry for this. My mistake. |
The solution offered by @maraism work great for older version of bundle (in my case 2.1) 👏 |
Hi,
I think there is an error in Liip\ImagineBundle\DependencyInjection\Configuration.php for webp.
We need to replace :
->ifTrue(function ($v) { return !$v || function_exists('imagewebp'); })
by
->ifTrue(function ($v) { return $v && !function_exists('imagewebp'); })
Thank's !
The text was updated successfully, but these errors were encountered: