Merge pull request #526 from lolautruche/patch-1

Fix crash when no post processor is defined

Author: makasim

Imagine/Filter/FilterManager.php

@@ -141,6 +141,7 @@ public function apply(BinaryInterface $binary, array $config)
      */
     public function applyPostProcessors(BinaryInterface $binary, $config)
     {
+        $config += array('post_processors' => array());
         foreach ($config['post_processors'] as $postProcessorName => $postProcessorOptions) {
             if (!isset($this->postProcessors[$postProcessorName])) {
                 throw new \InvalidArgumentException(sprintf(

Tests/Imagine/Filter/FilterManagerTest.php

@@ -1098,6 +1098,18 @@ public function testThrowsIfNoPostProcessorAddedForFilterOnApplyFilter()
         $filterManager->applyFilter($binary, 'thumbnail');
     }
 
+    public function testApplyPostProcessorsWhenNotDefined()
+    {
+        $binary = $this->getMock('Liip\ImagineBundle\Binary\BinaryInterface');
+        $filterManager = new FilterManager(
+            $this->createFilterConfigurationMock(),
+            $this->createImagineMock(),
+            $this->getMockMimeTypeGuesser()
+        );
+
+        $this->assertSame($binary, $filterManager->applyPostProcessors($binary, array()));
+    }
+
     /**
      * @return \PHPUnit_Framework_MockObject_MockObject|LoaderInterface
      */