fix: getFilterCaller() does not support Filter classes as array

datamweb · Oct 18, 2023 · 66b572a · 66b572a
1 parent cc185a8
commit 66b572a
Show file tree

Hide file tree

Showing 2 changed files with 60 additions and 7 deletions.
diff --git a/system/Test/FilterTestTrait.php b/system/Test/FilterTestTrait.php
@@ -132,26 +132,68 @@ protected function getFilterCaller($filter, string $position): Closure
                     throw new RuntimeException("No filter found with alias '{$filter}'");
                 }
 
-                $filter = $this->filtersConfig->aliases[$filter];
+                $filterClasses = $this->filtersConfig->aliases[$filter];
             }
 
-            // Get an instance
-            $filter = new $filter();
+            $filterClasses = (array) $filterClasses;
         }
 
-        if (! $filter instanceof FilterInterface) {
-            throw FilterException::forIncorrectInterface(get_class($filter));
+        foreach ($filterClasses as $class) {
+            // Get an instance
+            $filter = new $class();
+
+            if (! $filter instanceof FilterInterface) {
+                throw FilterException::forIncorrectInterface(get_class($filter));
+            }
         }
 
         $request = clone $this->request;
 
         if ($position === 'before') {
-            return static fn (?array $params = null) => $filter->before($request, $params);
+            return static function (?array $params = null) use ($filterClasses, $request) {
+                foreach ($filterClasses as $class) {
+                    $filter = new $class();
+
+                    $result = $filter->before($request, $params);
+
+                    // @TODO The following logic is in Filters class.
+                    //       Should use Filters class.
+                    if ($result instanceof RequestInterface) {
+                        $request = $result;
+
+                        continue;
+                    }
+                    if ($result instanceof ResponseInterface) {
+                        return $result;
+                    }
+                    if (empty($result)) {
+                        continue;
+                    }
+                }
+
+                return $result;
+            };
         }
 
         $response = clone $this->response;
 
-        return static fn (?array $params = null) => $filter->after($request, $response, $params);
+        return static function (?array $params = null) use ($filterClasses, $request, $response) {
+            foreach ($filterClasses as $class) {
+                $filter = new $class();
+
+                $result = $filter->after($request, $response, $params);
+
+                // @TODO The following logic is in Filters class.
+                //       Should use Filters class.
+                if ($result instanceof ResponseInterface) {
+                    $response = $result;
+
+                    continue;
+                }
+            }
+
+            return $result;
+        };
     }
 
     /**

diff --git a/tests/system/Test/FilterTestTraitTest.php b/tests/system/Test/FilterTestTraitTest.php
@@ -12,6 +12,7 @@
 namespace CodeIgniter\Test;
 
 use CodeIgniter\HTTP\RequestInterface;
+use Config\Services;
 use Tests\Support\Filters\Customfilter;
 
 /**
@@ -62,6 +63,16 @@ public function testGetCallerInvalidPosition(): void
         $this->getFilterCaller('test-customfilter', 'banana');
     }
 
+    public function testCallerSupportArray(): void
+    {
+        $this->filtersConfig->aliases['test-customfilter'] = [Customfilter::class];
+
+        $caller = $this->getFilterCaller('test-customfilter', 'before');
+        $result = $caller();
+
+        $this->assertSame('http://hellowworld.com', $result->getBody());
+    }
+
     public function testCallerUsesClonedInstance(): void
     {
         $caller = $this->getFilterCaller('test-customfilter', 'before');