liip/LiipImagineBundle — Issue #4: liip_imagine.formats option - inacurate description / weird behaviour
Author: mheleniak
Created: 2011-10-11 19:42:09+00:00
State: open

In the docs we have:

formats - optional list of image formats to which images may be converted to.

Right now if this configuration option is not defined, no images will be generated. Instead we will get the following error:

Source image not found in "/images/file-name.jpg"

This is because of https://github.com/liip/LiipImagineBundle/blob/master/Imagine/DataLoader/FileSystemLoader.php#L36-38

Defining `liip_imagine.formats` fixes the issue. Is this option required? What's the point of specifying this? If it's a filter to pass only images of specified format to imagine, shouldn't we check this in `apply_filter`, instead of generating errors? And support allow-all instead of deny-all setting by default?