Judicious doc: ensure that parameters is always a list

Even for functions without parameters or with one single parameter.
LuCEresearchlab · Sep 9, 2024 · f6e97f5 · f6e97f5
1 parent e520125
commit f6e97f5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/docs/judicious.xslt b/docs/judicious.xslt
@@ -41,7 +41,9 @@
         <xsl:apply-templates select="desc_content/paragraph[position() > 0]"/>
         <xsl:apply-templates select="desc_content/figure"/>
       </description>
-      <xsl:apply-templates select="desc_signature/desc_parameterlist/desc_parameter"/>
+      <parameters>
+        <xsl:apply-templates select="desc_signature/desc_parameterlist/desc_parameter"/>
+      </parameters>
       <xsl:if test="desc_signature/desc_returns">
         <returnValue>
           <type>

diff --git a/docs/judicious_transformer.py b/docs/judicious_transformer.py
@@ -19,8 +19,8 @@ def plural(element_type):
 
 
 def postprocessor(path, key, value):
-    if key == "parameter":
-        return plural(key), value
+    if key == "parameters":
+        return key, value["parameter"] if value is not None else []
     if key == "sideEffects":
         return key, True if value == "true" else False
     return key, value
@@ -43,7 +43,7 @@ def pytamaro_module(folder_name):
     for filename in languagedir.glob("*.xml"):
         xml = ET.parse(filename)
         new_xml = xslt_transform(xml)
-        new_xml_as_dict = xmltodict.parse(ET.tostring(new_xml, encoding='unicode'), force_list=['element', 'p'], postprocessor=postprocessor)
+        new_xml_as_dict = xmltodict.parse(ET.tostring(new_xml, encoding='unicode'), force_list=['element', 'p', 'parameter'], postprocessor=postprocessor)
         docs["elements"].extend(new_xml_as_dict['elements']["element"])
     output_path = OUTPUT_DIR / (module_name + ".json")
     with open(output_path, "w") as f: