Added context parameter to filter strategy callback.

Author: Paul

src/Strategy/Filter.php

@@ -27,8 +27,18 @@ public function __invoke($data, $context = null)
             return null;
         }
 
-        return array_filter($data, $this->callback ?: function ($value) {
-            return $value !== null;
-        });
+        $filter = function (array $data, $context) {
+            $callback = $this->callback ?: function ($value) {
+                return $value !== null;
+            };
+
+            foreach ($data as $datum) {
+                if ($callback($datum, $context)) {
+                    yield $datum;
+                }
+            }
+        };
+
+        return iterator_to_array($filter($data, $context));
     }
 }

test/MockFactory.php

@@ -1,6 +1,7 @@
 <?php
 namespace ScriptFUSIONTest;
 
+use Mockery\MockInterface;
 use ScriptFUSION\Mapper\Mapper;
 use ScriptFUSION\StaticClass;
 
@@ -11,7 +12,7 @@ final class MockFactory
     /**
      * @param mixed $data
      *
-     * @return Mapper
+     * @return Mapper|MockInterface
      */
     public static function mockMapper($data)
     {

test/Unit/Mapper/Strategy/FilterTest.php

@@ -35,6 +35,16 @@ public function testCustomCallback()
         self::assertEquals([['foo' => 'bar']], $filter([]));
     }
 
+    public function testContextPassed()
+    {
+        $filter = new Filter(null, function ($_, $context) {
+            self::assertSame('foo', $context);
+        });
+        $filter->setMapper(MockFactory::mockMapper(['bar']));
+
+        $filter([], 'foo');
+    }
+
     public function testNonArray()
     {
         $filter = new Filter(null);