feat: controllers can have optional params

Author: ipranjal

src/RouterEngine.php

@@ -207,9 +207,14 @@ private function getArguments(string $controller, string $method): bool|array
         }
         // Check weather arguments are passed else throw a 404 error
         $classMethod = new \ReflectionMethod($controllerObj, $method);
-
-        // Optional parameter introduced in version 3.0.2
-        if (count($arguments) < count($classMethod->getParameters())) {
+        $params = $classMethod->getParameters();
+        // Remove params if it allows null
+        foreach ($params as $key => $param) {
+            if ($param->isOptional()) {
+                unset($params[$key]);
+            }
+        }
+        if (count($arguments) < count($params)) {
             $this->debug('Not enough arguments given to the method');
 
             return false;

tests/Demo/Param.php

@@ -0,0 +1,15 @@
+<?php
+namespace Tests\Demo;
+class Param
+{
+
+    public function allIndex($id=null)
+    {
+        return "Index Test";
+    }
+
+    public function getTest($id=null)
+    {
+        return "Test Test";
+    }
+}

tests/Demo/Test.php

@@ -1,9 +1,10 @@
 <?php
 namespace Tests\Demo;
-class Test{
-
-public function getHi(): string{
-    return "Hi Test";
-}
+class Test
+{
 
+    public function getHi(): string
+    {
+        return "Hi Test";
+    }
 }

tests/Unit/RouterEngineTest.php

@@ -110,4 +110,19 @@
 
   expect($status)->toBe(\Scrawler\Router\Router::NOT_FOUND);
 
+});
+
+
+it('tests method call with optional parameter',function(): void{
+
+  $engine = new \Scrawler\Router\RouterEngine(getCollection(false));
+  [$status,$handler,$args,$debug] = $engine->route('GET',uri: '/param');
+  expect($handler)->toBe('Tests\Demo\Param::allIndex');
+  [$status,$handler,$args,$debug] = $engine->route('GET',uri: '/param/12');
+  expect($handler)->toBe('Tests\Demo\Param::allIndex');
+  [$status,$handler,$args,$debug] = $engine->route('GET',uri: '/param/test');
+  expect($handler)->toBe('Tests\Demo\Param::getTest');
+  [$status,$handler,$args,$debug] = $engine->route('GET',uri: '/param/test/12');
+  expect($handler)->toBe('Tests\Demo\Param::getTest');
+
 });