Fix invalid xstructural generation for named parameters.

io7m-com · Mar 8, 2024 · 8dcb216 · 8dcb216
1 parent fe274dc
commit 8dcb216
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 19 deletions.
diff --git a/README-CHANGES.xml b/README-CHANGES.xml
@@ -29,11 +29,16 @@
         <c:change date="2023-11-04T00:00:00+00:00" summary="Add parameterValueRequireNow for optional parameters."/>
       </c:changes>
     </c:release>
-    <c:release date="2024-01-06T11:43:10+00:00" is-open="false" ticket-system="com.github.io7m.quarrel" version="1.5.0">
+    <c:release date="2024-01-06T00:00:00+00:00" is-open="false" ticket-system="com.github.io7m.quarrel" version="1.5.0">
       <c:changes>
         <c:change date="2024-01-06T00:00:00+00:00" summary="Change format of xstructural parameters for better layout."/>
       </c:changes>
     </c:release>
+    <c:release date="2024-03-08T13:58:55+00:00" is-open="true" ticket-system="com.github.io7m.quarrel" version="1.6.0">
+      <c:changes>
+        <c:change date="2024-03-08T13:58:55+00:00" summary="Fix invalid xstructural generation for named parameters."/>
+      </c:changes>
+    </c:release>
   </c:releases>
   <c:ticket-systems>
     <c:ticket-system default="true" id="com.github.io7m.quarrel" url="https://www.github.com/io7m/quarrel/issues/"/>

diff --git a/...7m.quarrel.ext.xstructural/src/main/java/com/io7m/quarrel/ext/xstructural/QCommandXS.java b/...7m.quarrel.ext.xstructural/src/main/java/com/io7m/quarrel/ext/xstructural/QCommandXS.java
@@ -190,6 +190,17 @@ private static void showCommand(
 
         sectionName(context, command, document, root);
         sectionDescription(context, command, document, root);
+
+        final var named = command.onListNamedParameters();
+        if (!named.isEmpty()) {
+          final var e =
+            (Element) root.appendChild(
+              document.createElementNS(NS_XI, "xi:include")
+            );
+
+          e.setAttribute("href", context.parameterValue(PARAMETERS_INCLUDE_NAME));
+        }
+
         sectionExamples(document, root);
       }
       case "parameters" -> {
@@ -265,24 +276,6 @@ private static void sectionDescription(
     para.appendChild(document.createTextNode("The "));
     para.appendChild(term);
     para.appendChild(document.createTextNode(" command... "));
-
-    final var named = command.onListNamedParameters();
-    if (!named.isEmpty()) {
-      final var formal =
-        (Element) description.appendChild(
-          document.createElementNS(NS, "FormalItem")
-        );
-
-      formal.setAttribute("title", "Parameters");
-
-      final var e =
-        (Element) description.appendChild(
-          document.createElementNS(NS_XI, "xi:include")
-        );
-
-      e.setAttribute("href", context.parameterValue(PARAMETERS_INCLUDE_NAME));
-      formal.appendChild(e);
-    }
   }
 
   private static Element parameterTable(

diff --git a/pom.xml b/pom.xml
@@ -34,6 +34,7 @@
     <com.io7m.jxtrand.version>1.1.0</com.io7m.jxtrand.version>
     <com.io7m.xstructural.version>1.8.0</com.io7m.xstructural.version>
     <io7m.api.previousVersion>1.4.0</io7m.api.previousVersion>
+    <io7m.java.targetJavaVersion>21</io7m.java.targetJavaVersion>
     <jqwik.version>1.7.3</jqwik.version>
     <junit.version>5.9.3</junit.version>
   </properties>