Added cache clearing & setting cachePrefix for Aws S3

[rvanlaarhoven]

Not sure if this is the best solution, but I used a setter to get the cachePrefix to fix #269 & #251.
This would require adding the prefix to the resolver like;

acme.imagine.cache.resolver.amazon_s3:
        class: Liip\ImagineBundle\Imagine\Cache\Resolver\AwsS3Resolver
        arguments:
            - "@acme.amazon_s3"
            - "%amazon_s3.bucket%"
        tags:
            - { name: 'liip_imagine.cache.resolver', resolver: 'amazon_s3' }
        calls:
            - [setCachePrefix, ["%liip_imagine.cache_prefix%"]]

Anyway, it clears all files inside the predefined directory in your bucket when running the app/console cache:clear command.

[makasim]

looks good to me

[makasim]

please do nothing here, as exception is a BC break.

FYI: in 1.0 this method was removed.

[rvanlaarhoven]

You mean replacing the Exception with a simple return, like in the new commit?

[makasim]

yes, could add {} for if statement as it require PSR

[rvanlaarhoven]

Done

[makasim]

thanks

[ribeiropaulor]

Hello,

am I missing something or this improvement was overwritten in ( 8cd9e06 )?

It's not, for sure, in the last dev-master version of the repository.