feat: spark filter:check shows "Required Filters"

codeigniter4 · Nov 29, 2023 · 631fdea · 631fdea
1 parent 507c79c
commit 631fdea
Show file tree

Hide file tree

Showing 4 changed files with 66 additions and 2 deletions.
diff --git a/system/Commands/Utilities/FilterCheck.php b/system/Commands/Utilities/FilterCheck.php
@@ -106,6 +106,8 @@ public function run(array $params)
             return EXIT_ERROR;
         }
 
+        $filters = $this->addRequiredFilters($filterCollector, $filters);
+
         $tbody[] = [
             strtoupper($method),
             $route,
@@ -124,4 +126,29 @@ public function run(array $params)
 
         return EXIT_SUCCESS;
     }
+
+    private function addRequiredFilters(FilterCollector $filterCollector, array $filters): array
+    {
+        $output = [];
+
+        $required = $filterCollector->getRequiredFilters();
+
+        $colored = [];
+
+        foreach ($required['before'] as $filter) {
+            $filter    = CLI::color($filter, 'yellow');
+            $colored[] = $filter;
+        }
+        $output['before'] = array_merge($colored, $filters['before']);
+
+        $colored = [];
+
+        foreach ($required['after'] as $filter) {
+            $filter    = CLI::color($filter, 'yellow');
+            $colored[] = $filter;
+        }
+        $output['after'] = array_merge($filters['after'], $colored);
+
+        return $output;
+    }
 }
diff --git a/system/Commands/Utilities/Routes/FilterCollector.php b/system/Commands/Utilities/Routes/FilterCollector.php
@@ -13,6 +13,7 @@
 
 use CodeIgniter\Config\Services;
 use CodeIgniter\Filters\Filters;
+use CodeIgniter\HTTP\Method;
 use CodeIgniter\HTTP\Request;
 use CodeIgniter\Router\Router;
 use Config\Filters as FiltersConfig;
@@ -37,6 +38,8 @@ public function __construct(bool $resetRoutes = false)
     }
 
     /**
+     * Returns filters for the URI
+     *
      * @param string $method HTTP verb like `GET`,`POST` or `CLI`.
      * @param string $uri    URI path to find filters for
      *
@@ -76,6 +79,24 @@ public function get(string $method, string $uri): array
         return $finder->find($uri);
     }
 
+    /**
+     * Returns Required Filters
+     *
+     * @return array{before: list<string>, after: list<string>} array of filter alias or classname
+     */
+    public function getRequiredFilters(): array
+    {
+        $request = Services::incomingrequest(null, false);
+        $request->setMethod(Method::GET);
+
+        $router  = $this->createRouter($request);
+        $filters = $this->createFilters($request);
+
+        $finder = new FilterFinder($router, $filters);
+
+        return $finder->getRequiredFilters();
+    }
+
     private function createRouter(Request $request): Router
     {
         $routes = Services::routes();

diff --git a/system/Commands/Utilities/Routes/FilterFinder.php b/system/Commands/Utilities/Routes/FilterFinder.php
@@ -77,4 +77,20 @@ public function find(string $uri): array
             ];
         }
     }
+
+    /**
+     * Returns Required Filters
+     *
+     * @return array{before: list<string>, after:list<string>}
+     */
+    public function getRequiredFilters(): array
+    {
+        [$requiredBefore] = $this->filters->getRequiredFilters('before');
+        [$requiredAfter]  = $this->filters->getRequiredFilters('after');
+
+        return [
+            'before' => $requiredBefore,
+            'after'  => $requiredAfter,
+        ];
+    }
 }
diff --git a/tests/system/Commands/FilterCheckTest.php b/tests/system/Commands/FilterCheckTest.php
@@ -45,8 +45,8 @@ public function testFilterCheckDefinedRoute(): void
         command('filter:check GET /');
 
         $this->assertStringContainsString(
-            '|GET|/|||',
-            str_replace(' ', '', $this->getBuffer())
+            '| GET    | /     | forcehttps pagecache | pagecache performance toolbar |',
+            preg_replace('/\033\[.+?m/u', '', $this->getBuffer())
         );
     }