distribution: ensure format_version exists or emit gco:nilReason=miss…

…ing (#240)
geopython · Jan 16, 2024 · 335e1b8 · 335e1b8
1 parent c1ff8ec
commit 335e1b8
Show file tree

Hide file tree

Showing 6 changed files with 42 additions and 14 deletions.
diff --git a/docs/content/reference/formats/iso19139-hnap.md b/docs/content/reference/formats/iso19139-hnap.md
@@ -47,7 +47,7 @@ distribution:
         format:
             en: Other
             fr: Autre
-        format_version: 0
+        format_version: '0'
         function: download
 
     waf_eng-CAN:
@@ -62,7 +62,7 @@ distribution:
         format:
             en: Other
             fr: Autre
-        format_version: 0
+        format_version: '0'
         function: download
 
     wms_eng-CAN:
@@ -72,7 +72,7 @@ distribution:
             fr: Service Web
         type: OGC:WMS
         format: WMS
-        format_version: 1.1.1
+        format_version: '1.1.1'
         name:
             en: GDPS.ETA_TT
             fr: GDPS.ETA_TT
@@ -85,7 +85,7 @@ distribution:
             fr: Service Web
         type: OGC:WMS
         format: WMS
-        format_version: 1.1.1
+        format_version: '1.1.1'
         name:
             en: GDPS.ETA_TT
             fr: GDPS.ETA_TT

diff --git a/pygeometa/schemas/iso19139/main.j2 b/pygeometa/schemas/iso19139/main.j2
@@ -384,11 +384,17 @@
   <gmd:distributionInfo>
     <gmd:MD_Distribution>
       {% for k, v in record['distribution'].items() %}
-      {% if v['format'] and v['format_version'] %}
+      {% if v['format'] %}
       <gmd:distributionFormat>
         <gmd:MD_Format>
           <gmd:name>{{ v['format'] }}</gmd:name>
-          <gmd:version>{{ v['format_version'] }}</gmd:version>
+          {% if v['format_version'] %}
+          <gmd:version>
+            <gco:CharacterString>{{ v['format_version'] }}</gco:CharacterString>
+          </gmd:version>
+          {% else %}
+          <gmd:version gco:nilReason="missing"/>
+          {% endif %}
         </gmd:MD_Format>
       </gmd:distributionFormat>
       {% endif %}

diff --git a/pygeometa/schemas/iso19139_2/main.j2 b/pygeometa/schemas/iso19139_2/main.j2
@@ -384,11 +384,17 @@
   <gmd:distributionInfo>
     <gmd:MD_Distribution>
       {% for k, v in record['distribution'].items() %}
-      {% if v['format'] and v['format_version'] %}
+      {% if v['format'] %}
       <gmd:distributionFormat>
         <gmd:MD_Format>
           <gmd:name>{{ v['format'] }}</gmd:name>
-          <gmd:version>{{ v['format_version'] }}</gmd:version>
+          {% if v['format_version'] %}
+          <gmd:version>
+            <gco:CharacterString>{{ v['format_version'] }}</gco:CharacterString>
+          </gmd:version>
+          {% else %}
+          <gco:version gco:nilReason="missing"/>
+          {% endif %}
         </gmd:MD_Format>
       </gmd:distributionFormat>
       {% endif %}

diff --git a/pygeometa/schemas/iso19139_hnap/main.j2 b/pygeometa/schemas/iso19139_hnap/main.j2
@@ -541,13 +541,17 @@
     <gmd:MD_Distribution>
       {% set formats = prune_distribution_formats(record['distribution']) %}
       {% for v in formats %}
-      {% if v['format_en'] and v['format_fr'] and v['format_version'] %}
+      {% if v['format_en'] and v['format_fr'] %}
       <gmd:distributionFormat>
         <gmd:MD_Format>
           {{ cs.get_freetext('name', 'fra', get_charstring(v.get('format'), 'en', 'fr')) }}
+          {% if v['format_version'] %}
           <gmd:version>
             <gco:CharacterString>{{ v['format_version'] }}</gco:CharacterString>
           </gmd:version>
+          {% else %}
+          <gmd:version gco:nilReason="missing"/>
+          {% endif %}
         </gmd:MD_Format>
       </gmd:distributionFormat>
       {% endif %}

diff --git a/pygeometa/schemas/wmo_cmp/main.j2 b/pygeometa/schemas/wmo_cmp/main.j2
@@ -364,11 +364,17 @@
   <gmd:distributionInfo>
     <gmd:MD_Distribution>
       {% for k, v in record['distribution'].items() %}
-      {% if v['format'] and v['format_version'] %}
+      {% if v['format'] %}
       <gmd:distributionFormat>
         <gmd:MD_Format>
           <gmd:name>{{ v['format'] }}</gmd:name>
-          <gmd:version>{{ v['format_version'] }}</gmd:version>
+          {% if v['format_version'] %}
+          <gmd:version>
+            <gco:CharacterString>{{ v['format_version'] }}</gco:CharacterString>
+          </gmd:version>
+          {% else %}
+          <gmd:version gco:nilReason="missing"/>
+          {% endif %}
         </gmd:MD_Format>
       </gmd:distributionFormat>
       {% endif %}

diff --git a/tests/run_tests.py b/tests/run_tests.py
@@ -178,13 +178,19 @@ def test_prune_distribution_formats(self):
 
         formats = {
             'wms': {
-                'format_en': 'image', 'format_fr': 'image', 'format_version': 2
+                'format_en': 'image',
+                'format_fr': 'image',
+                'format_version': '2'
             },
             'wfs': {
-                'format_en': 'GRIB2', 'format_fr': 'GRIB2', 'format_version': 2
+                'format_en': 'GRIB2',
+                'format_fr': 'GRIB2',
+                'format_version': '2'
             },
             'wcs': {
-                'format_en': 'GRIB2', 'format_fr': 'GRIB2', 'format_version': 2
+                'format_en': 'GRIB2',
+                'format_fr': 'GRIB2',
+                'format_version': '2'
             }
         }