[BUGFIX] Warn when a directive has not title

resolves #983

Author: linawolf

packages/guides-restructured-text/src/RestructuredText/Directives/ConfvalDirective.php

@@ -22,9 +22,11 @@
 use phpDocumentor\Guides\RestructuredText\Parser\InlineParser;
 use phpDocumentor\Guides\RestructuredText\Parser\Productions\Rule;
 use phpDocumentor\Guides\RestructuredText\TextRoles\GenericLinkProvider;
+use Psr\Log\LoggerInterface;
 
 use function boolval;
 use function in_array;
+use function trim;
 
 /**
  * The confval directive configuration values.
@@ -41,6 +43,7 @@ public function __construct(
         GenericLinkProvider $genericLinkProvider,
         private readonly AnchorNormalizer $anchorReducer,
         private readonly InlineParser $inlineParser,
+        private readonly LoggerInterface|null $logger = null,
     ) {
         parent::__construct($startingRule);
 
@@ -71,6 +74,12 @@ protected function processSub(
         $required = false;
         $default = null;
         $additionalOptions = [];
+        if (trim($directive->getData()) === '') {
+            if ($this->logger !== null) {
+                $this->logger->warning('A directive must have a title: ..  confval:: [some_title]', $blockContext->getLoggerInformation());
+            }
+        }
+
         if ($directive->hasOption('type')) {
             $type = $this->inlineParser->parse($directive->getOption('type')->toString(), $blockContext);
         }

tests/Integration/tests/confval/confval-no-title/expected/index.html

@@ -0,0 +1,24 @@
+<!-- content start -->
+    <div class="section" id="confval-directive">
+            <h1>Confval directive</h1>
+            <dl class="confval">
+    <dt id="confval-">
+        <code class="sig-name descname"><span class="pre"></span></code></dt>
+    <dd>
+        <div class="line-block">
+             <div class="line"><strong>Type:</strong>  <code>&quot;Hello World&quot;</code></div>
+             <div class="line"><strong>Required:</strong> true</div>
+            <div class="line"><strong>Custom Info:</strong> <strong>custom</strong></div>
+
+            </div>
+        <div class="confval-description">
+
+    <p>This is the confval <code>demo</code> content!</p>
+
+    <p>Another paragraph.</p>
+
+        </div>
+    </dd>
+</dl>
+    </div>
+<!-- content end -->

tests/Integration/tests/confval/confval-no-title/expected/logs/warning.log

@@ -0,0 +1 @@
+app.WARNING: A directive must have a title

tests/Integration/tests/confval/confval-no-title/input/index.rst

@@ -0,0 +1,12 @@
+Confval directive
+=================
+
+.. confval::
+    :type: :php:`string`
+    :default: ``"Hello World"``
+    :required: true
+    :Custom Info: **custom**
+
+    This is the confval ``demo`` content!
+
+    Another paragraph.

tests/Integration/tests/graphs/plantuml-server-error/expected/index.html

@@ -1,11 +1,11 @@
 <!-- content start -->
-<div class="section" id="uml-directive">
-    <h1>Uml Directive</h1>
-    <figure
-            class="uml-diagram"
-            style="width: 1000"    ><figcaption>Figure 1-1: Application flow</figcaption></figure>
-    <figure
-            class="uml-diagram"
-            style="width: 1000"    ><figcaption>Figure 1-1: Application flow</figcaption></figure>
-</div>
+    <div class="section" id="uml-directive">
+            <h1>Uml Directive</h1>
+            <figure
+        class="uml-diagram"
+        style="width: 1000"    ><figcaption>Figure 1-1: Application flow</figcaption></figure>
+            <figure
+        class="uml-diagram"
+        style="width: 1000"    ><figcaption>Figure 1-1: Application flow</figcaption></figure>
+    </div>
 <!-- content end -->