Fix silverstripe#11087: Improve handling of nonexistent actions in de…

…legateToMultipleHandlers

This commit addresses issue silverstripe#11087 by enhancing the error handling in the delegateToMultipleHandlers method of the Security class. Previously, attempting a nonexistent action would generate a generic warning. With this modification, the method now checks for the presence of an HTTPResponse in the results array and returns it directly, providing a more informative error message.

tests/php/Security/SecurityTest.php

@@ -7,6 +7,7 @@
 use SilverStripe\Control\Director;
 use SilverStripe\Control\HTTPRequest;
 use SilverStripe\Control\HTTPResponse;
+use SilverStripe\Control\RequestHandler;
 use SilverStripe\Control\Session;
 use SilverStripe\Core\Config\Config;
 use SilverStripe\Core\Convert;
@@ -820,4 +821,30 @@ protected function getValidationResult()
         }
         return null;
     }
+
+
+    /**
+     *  Test for DelegateToMultipleHandlers() in Security.php
+     *  @return void
+     */
+    protected function testDelegateToMultipleHandlers()
+    {
+        $security = new Security(); // Assurez-vous d'adapter le namespace et la classe selon votre projet
+        $handler1 = $this->getMockBuilder(RequestHandler::class)->getMock();
+        $handler2 = $this->getMockBuilder(RequestHandler::class)->getMock();
+
+        // Simulation of HTTPResponse
+        $handler2->expects($this->once())
+            ->method('handleRequest')
+            ->willReturn(new HTTPResponse('This is an HTTPResponse', 404));
+
+        $result = $security->delegateToMultipleHandlers([$handler1, $handler2], 'Test Title', ['template'], function ($results) {
+            return 'Aggregated Result';
+        });
+
+        $this->assertInstanceOf(HTTPResponse::class, $result);
+        $this->assertEquals('This is an HTTPResponse', $result->getBody());
+        $this->assertEquals(404, $result->getStatusCode());
+    }
+
 }