Fix: Only try to map values

This is because if the data includes "Null" this gets parsed to None. Example doi: 10.14280/08241.22
datacite · Apr 20, 2020 · 8115d0e · 8115d0e
1 parent 5cde842
commit 8115d0e
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/viringo/metadata.py b/viringo/metadata.py
@@ -42,10 +42,11 @@ def nsdc(name):
             'rights'
         ]:
         for value in _map.get(name, []):
-            new_element = etree.SubElement(e_dc, nsdc(name))
-            # The regular expression here is to filter only valid XML chars
-            # Char ::= #x9 | #xA | #xD | [#x20-#xD7FF] | [#xE000-#xFFFD] | [#x10000-#x10FFFF]
-            new_element.text = re.sub(u'[^\u0020-\uD7FF\u0009\u000A\u000D\uE000-\uFFFD\U00010000-\U0010FFFF]+', '', value)
+            if value:
+                new_element = etree.SubElement(e_dc, nsdc(name))
+                # The regular expression here is to filter only valid XML chars
+                # Char ::= #x9 | #xA | #xD | [#x20-#xD7FF] | [#xE000-#xFFFD] | [#x10000-#x10FFFF]
+                new_element.text = re.sub(u'[^\u0020-\uD7FF\u0009\u000A\u000D\uE000-\uFFFD\U00010000-\U0010FFFF]+', '', value)
 
 def datacite_writer(element: etree.Element, metadata):
     """Writer for writing data in a metadata object out into raw datacite format"""