Skip action if response already sent by init hook

src/Http/Http.php

@@ -693,7 +693,7 @@ public function match(Request $request, bool $fresh = true): ?Route
      * @param  Route  $route
      * @param  Request  $request
      */
-    public function execute(Route $route, Request $request): static
+    public function execute(Route $route, Request $request, Response $response): static
     {
         $arguments = [];
         $groups = $route->getGroups();
@@ -720,8 +720,10 @@ public function execute(Route $route, Request $request): static
                 }
             }
 
-            $arguments = $this->getArguments($route, $pathValues, $request->getParams());
-            \call_user_func_array($route->getAction(), $arguments);
+            if (!$response->isSent()) {
+                $arguments = $this->getArguments($route, $pathValues, $request->getParams());
+                \call_user_func_array($route->getAction(), $arguments);
+            }
 
             foreach ($groups as $group) {
                 foreach (self::$shutdown as $hook) { // Group shutdown hooks
@@ -953,7 +955,7 @@ private function runInternal(Request $request, Response $response): static
         }
 
         if (null !== $route) {
-            return $this->execute($route, $request);
+            return $this->execute($route, $request, $response);
         } elseif (self::REQUEST_METHOD_OPTIONS == $method) {
             try {
                 foreach ($groups as $group) {

tests/HttpTest.php

@@ -108,7 +108,7 @@ public function testCanExecuteRoute(): void
             });
 
         \ob_start();
-        $this->http->execute($route, new Request());
+        $this->http->execute($route, new Request(), new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -132,7 +132,7 @@ public function testCanExecuteRoute(): void
         \ob_start();
         $request = new UtopiaFPMRequestTest();
         $request::_setParams(['x' => 'param-x', 'y' => 'param-y', 'z' => 'param-z']);
-        $this->http->execute($route, $request);
+        $this->http->execute($route, $request, new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -152,7 +152,7 @@ public function testCanExecuteRoute(): void
         \ob_start();
         $request = new UtopiaFPMRequestTest();
         $request::_setParams(['x' => 'param-x', 'y' => 'param-y']);
-        $this->http->execute($route, $request);
+        $this->http->execute($route, $request, new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -224,7 +224,7 @@ public function testCanExecuteRoute(): void
         \ob_start();
         $request = new UtopiaFPMRequestTest();
         $request::_setParams(['x' => 'param-x', 'y' => 'param-y']);
-        $this->http->execute($route, $request);
+        $this->http->execute($route, $request, new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -234,7 +234,7 @@ public function testCanExecuteRoute(): void
         \ob_start();
         $request = new UtopiaFPMRequestTest();
         $request::_setParams(['x' => 'param-x', 'y' => 'param-y']);
-        $this->http->execute($homepage, $request);
+        $this->http->execute($homepage, $request, new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -264,7 +264,7 @@ public function testCanAddAndExecuteHooks()
             });
 
         \ob_start();
-        $this->http->execute($route, new Request());
+        $this->http->execute($route, new Request(), new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -280,7 +280,7 @@ public function testCanAddAndExecuteHooks()
             });
 
         \ob_start();
-        $this->http->execute($route, new Request());
+        $this->http->execute($route, new Request(), new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -348,15 +348,15 @@ public function testCanHookThrowExceptions()
             });
 
         \ob_start();
-        $this->http->execute($route, new Request());
+        $this->http->execute($route, new Request(), new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
         $this->assertEquals('error-Param "y" is not optional.', $result);
 
         \ob_start();
         $_GET['y'] = 'y-def';
-        $this->http->execute($route, new Request());
+        $this->http->execute($route, new Request(), new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -606,7 +606,7 @@ public function testCallableStringParametersNotExecuted(): void
             });
 
         \ob_start();
-        $this->http->execute($route, new Request());
+        $this->http->execute($route, new Request(), new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -624,7 +624,7 @@ public function testCallableStringParametersNotExecuted(): void
         \ob_start();
         $request = new UtopiaFPMRequestTest();
         $request::_setParams(['func' => 'system']);
-        $this->http->execute($route2, $request);
+        $this->http->execute($route2, $request, new Response());
         $result = \ob_get_contents();
         \ob_end_clean();
 
@@ -642,7 +642,7 @@ public function testCallableStringParametersNotExecuted(): void
             });
 
         \ob_start();
-        $this->http->execute($route3, new Request());
+        $this->http->execute($route3, new Request(), new Response());
         $result = \ob_get_contents();
         \ob_end_clean();