Merge pull request #122 from spiffyjr/patch-1

Update AssetFilterManager.php

Author: RWOverdijk

src/AssetManager/Service/AssetFilterManager.php

@@ -83,6 +83,9 @@ public function setFilters($path, AssetInterface $asset)
         }
 
         foreach ($filters as $filter) {
+            if (is_null($filter)) {
+                continue;
+            }
             if (!empty($filter['filter'])) {
                 $this->ensureByFilter($asset, $filter['filter']);
             } elseif (!empty($filter['service'])) {

tests/AssetManagerTest/Service/AssetFilterManagerTest.php

@@ -17,6 +17,21 @@ public function setUp()
         require_once __DIR__ . '/../../_files/CustomFilter.php';
     }
 
+    public function testNulledValuesAreSkipped()
+    {
+        $assetFilterManager = new AssetFilterManager(array(
+        'test/path.test' => array(
+            'null_filters' => null
+        )
+        ));
+
+        $asset = new StringAsset('Herp Derp');
+
+        $assetFilterManager->setFilters('test/path.test', $asset);
+
+        $this->assertEquals('called', $asset->dump());
+    }
+
     public function testensureByService()
     {
         $assetFilterManager = new AssetFilterManager(array(