IBX-10116: Fixed ibexa_render() not using decorated fragment renders (#579)

Author: vidarl

src/contracts/MVC/Templating/BaseRenderStrategy.php

@@ -34,7 +34,9 @@ public function __construct(
         RequestStack $requestStack
     ) {
         foreach ($fragmentRenderers as $fragmentRenderer) {
-            $this->fragmentRenderers[$fragmentRenderer->getName()] = $fragmentRenderer;
+            if (!isset($this->fragmentRenderers[$fragmentRenderer->getName()])) {
+                $this->fragmentRenderers[$fragmentRenderer->getName()] = $fragmentRenderer;
+            }
         }
 
         $this->defaultRenderer = $defaultRenderer;

tests/lib/MVC/Symfony/Templating/RenderContentStrategyTest.php

@@ -93,6 +93,27 @@ public function testMultipleFragmentRenderers(): void
         );
     }
 
+    public function testDuplicatedFragmentRenderers(): void
+    {
+        $renderContentStrategy = $this->createRenderStrategy(
+            RenderContentStrategy::class,
+            [
+                $this->createFragmentRenderer('method_a', 'decorator service used'),
+                $this->createFragmentRenderer('method_a', 'original service used'),
+            ],
+        );
+
+        $contentMock = $this->createMock(Content::class);
+        self::assertTrue($renderContentStrategy->supports($contentMock));
+
+        self::assertSame(
+            'decorator service used',
+            $renderContentStrategy->render($contentMock, new RenderOptions([
+                'method' => 'method_a',
+            ]))
+        );
+    }
+
     public function testExpectedMethodRenderArgumentsFormat(): void
     {
         $request = new Request();