Allow configuration of placeholder overriding attributes

Fixes PHP-DI#86

src/Bridge.php

@@ -22,7 +22,10 @@
  */
 class Bridge
 {
-    public static function create(?ContainerInterface $container = null): App
+    public static function create(
+        ?ContainerInterface $container = null,
+        bool $prioritiseAttributesOverParams = false
+    ): App
     {
         $container = $container ?: new Container;
 
@@ -33,7 +36,7 @@ public static function create(?ContainerInterface $container = null): App
 
         $container->set(App::class, $app);
 
-        $controllerInvoker = static::createControllerInvoker($container);
+        $controllerInvoker = static::createControllerInvoker($container, $prioritiseAttributesOverParams);
         $app->getRouteCollector()->setDefaultInvocationStrategy($controllerInvoker);
 
         return $app;
@@ -59,8 +62,8 @@ protected static function createInvoker(ContainerInterface $container): Invoker
     /**
      * Create a controller invoker with the default resolvers.
      */
-    protected static function createControllerInvoker(ContainerInterface $container): ControllerInvoker
+    protected static function createControllerInvoker(ContainerInterface $container, bool $prioritiseAttributesOverParams): ControllerInvoker
     {
-        return new ControllerInvoker(self::createInvoker($container));
+        return new ControllerInvoker(self::createInvoker($container), $prioritiseAttributesOverParams);
     }
 }

src/ControllerInvoker.php

@@ -12,9 +12,13 @@ class ControllerInvoker implements InvocationStrategyInterface
     /** @var InvokerInterface */
     private $invoker;
 
-    public function __construct(InvokerInterface $invoker)
+    /** @var bool Whether attributes should override parameters */
+    protected $prioritiseAttributesOverParams;
+
+    public function __construct(InvokerInterface $invoker, bool $prioritiseAttributesOverParams)
     {
         $this->invoker = $invoker;
+        $this->prioritiseAttributesOverParams = $prioritiseAttributesOverParams;
     }
 
     /**
@@ -33,7 +37,7 @@ public function __invoke(
     ): ResponseInterface {
         // Inject the request and response by parameter name
         $parameters = [
-            'request'  => self::injectRouteArguments($request, $routeArguments),
+            'request'  => self::injectRouteArguments($request, $routeArguments, $this->prioritiseAttributesOverParams),
             'response' => $response,
         ];
         // Inject the route arguments by name
@@ -60,10 +64,17 @@ protected function processResponse($response)
      *
      * @param array                  $routeArguments
      */
-    protected static function injectRouteArguments(ServerRequestInterface $request, array $routeArguments): ServerRequestInterface
+    protected static function injectRouteArguments(
+        ServerRequestInterface $request,
+        array $routeArguments,
+        bool $prioritiseAttributesOverParams
+    ): ServerRequestInterface
     {
         $requestWithArgs = $request;
         foreach ($routeArguments as $key => $value) {
+            if ($prioritiseAttributesOverParams && $request->getAttribute($key) !== null) {
+                continue;
+            }
             $requestWithArgs = $requestWithArgs->withAttribute($key, $value);
         }
         return $requestWithArgs;

tests/RoutingTest.php

@@ -127,6 +127,26 @@ public function injects_route_placeholder_over_request_attribute()
         $this->assertEquals('Hello matt', (string) $response->getBody());
     }
 
+    /**
+     * @test
+     */
+    public function prefers_request_attribute_over_route_placeholder_if_configured()
+    {
+        $app = Bridge::create(null, true);
+        $app->add(function (ServerRequestInterface $request, RequestHandlerInterface $next) {
+            return $next->handle($request->withAttribute('name', 'Bob'));
+        });
+        $app->get('/{name}', function ($name, $request, $response) {
+            $response->getBody()->write('Hello ' . $name . ', from ' . $request->getAttribute('name'));
+            return $response;
+        });
+
+        $response = $app->handle(RequestFactory::create('/matt'));
+
+        // The route placeholder does not override the request attribute but the placeholder is still provided to the controller
+        $this->assertEquals('Hello matt, from Bob', (string) $response->getBody());
+    }
+
     /**
      * @test
      */