-

Author: javiereguiluz

src/DocsKernel.php

@@ -20,6 +20,7 @@
 use SymfonyDocsBuilder\Listener\AdmonitionListener;
 use SymfonyDocsBuilder\Listener\AssetsCopyListener;
 use SymfonyDocsBuilder\Listener\CopyImagesListener;
+use SymfonyDocsBuilder\Listener\DuplicatedHeaderIdListener;
 
 class DocsKernel extends Kernel
 {
@@ -49,6 +50,11 @@ private function initializeListeners(EventManager $eventManager, ErrorManager $e
            new AdmonitionListener()
        );
 
+        $eventManager->addEventListener(
+            PreParseDocumentEvent::PRE_PARSE_DOCUMENT,
+            new DuplicatedHeaderIdListener()
+        );
+
         $eventManager->addEventListener(
            PreNodeRenderEvent::PRE_NODE_RENDER,
            new CopyImagesListener($this->buildConfig, $errorManager)

src/Listener/DuplicatedHeaderIdListener.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * This file is part of the Docs Builder package.
+ * (c) Ryan Weaver <ryan@symfonycasts.com>
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace SymfonyDocsBuilder\Listener;
+
+use Doctrine\RST\Event\PreParseDocumentEvent;
+use SymfonyDocsBuilder\Renderers\TitleNodeRenderer;
+
+final class DuplicatedHeaderIdListener
+{
+    public function preParseDocument(PreParseDocumentEvent $event): void
+    {
+        // needed because we only need to handle duplicated headers within
+        // the same file, not across all the files being generated
+        TitleNodeRenderer::resetHeaderIdCache();
+    }
+}

src/Renderers/TitleNodeRenderer.php

@@ -32,6 +32,11 @@ public function __construct(TitleNode $titleNode, TemplateRenderer $templateRend
         $this->templateRenderer = $templateRenderer;
     }
 
+    public static function resetHeaderIdCache(): void
+    {
+        self::$idUsagesCountByFilename = [];
+    }
+
     public function render(): string
     {
         $filename = $this->titleNode->getEnvironment()->getCurrentFileName();

tests/IntegrationTest.php

@@ -21,15 +21,6 @@
 
 class IntegrationTest extends AbstractIntegrationTest
 {
-    public static function setUpBeforeClass(): void
-    {
-        $reflection = new \ReflectionClass(TitleNodeRenderer::class);
-        $property = $reflection->getProperty('idUsagesCountByFilename');
-        $property->setAccessible(true);
-
-        $property->setValue([]);
-    }
-
     /**
      * @dataProvider integrationProvider
      */

tests/JsonIntegrationTest.php

@@ -14,15 +14,6 @@
 
 class JsonIntegrationTest extends AbstractIntegrationTest
 {
-    public function setUp(): void
-    {
-        $reflection = new \ReflectionClass(TitleNodeRenderer::class);
-        $property = $reflection->getProperty('idUsagesCountByFilename');
-        $property->setAccessible(true);
-
-        $property->setValue([]);
-    }
-
     /**
      * @dataProvider getJsonTests
      */