Merge branch 'ignore-empty-iiif-metadata' into 'master'

Ignore empty IIIF manifest metadata

See merge request !344

arkindex/dataimport/iiif.py

@@ -241,6 +241,9 @@ class ManifestParser(IIIFParser):
             if not all(prop in item for prop in ('label', 'value')):
                 logger.warning('Metadata does not have the required label and value properties')
                 continue
+            if not item['value']:
+                logger.warning('Metadata with label {} has an empty value - ignoring'.format(item['value']))
+                continue
             data.append(item)
 
         def grouper(item):

arkindex/dataimport/tests/manifest_samples/base.json

@@ -30,6 +30,10 @@
         {
             "label": "Label 1",
             "value": "Value 4"
+        },
+        {
+            "label": "Ignore me",
+            "value": ""
         }
     ],
     "license": "http://creativecommons.org/licenses/by-nc/3.0/deed.fr",