search

Oryzone / OryzoneMediaStorageBundle

Bundle to abstract the process of storing and retrieving media files
8 stars 11 forks source link

ImageProvider throwing ProviderProcessException with incorrect arguments #23

Closed piotrminkina closed 11 years ago

piotrminkina commented 11 years ago

Missing one argument for thrown ProviderProcessException by ImageProvider.

https://github.com/Oryzone/OryzoneMediaStorageBundle/blob/2.0/Provider/ImageProvider.php#L166 should be

throw new ProviderProcessException(sprintf('[...]', $media), $this, $media, $variant);

instead of

throw new ProviderProcessException(sprintf('[...]', $media), $media, $variant);
lmammino commented 11 years ago

Closed with 09fc5a8e531fadfb2163fe030890e82b9df4f583 . Thanks a lot for reporting it. Are you using the bundle? This new version is still incomplete and not totally tested, but I'd like to receive a feedback from you if you've tried it.