Browse code

Utils/parseNodes: cleanup - avoid returning empty dicts inside the result list

Florent Viard authored on 2020/06/22 10:55:30
Showing 1 changed files
... ...
@@ -61,6 +61,7 @@ def parseNodes(nodes):
61 61
     ## WARNING: Ignores text nodes from mixed xml/text.
62 62
     ## For instance <tag1>some text<tag2>other text</tag2></tag1>
63 63
     ## will be ignore "some text" node
64
+    ## WARNING 2: Any node at first level without children will also be ignored
64 65
     retval = []
65 66
     for node in nodes:
66 67
         retval_item = {}
... ...
@@ -74,7 +75,8 @@ def parseNodes(nodes):
74 74
                     retval_item[name] = decode_from_s3(found_text)
75 75
                 else:
76 76
                     retval_item[name] = None
77
-        retval.append(retval_item)
77
+        if retval_item:
78
+            retval.append(retval_item)
78 79
     return retval
79 80
 __all__.append("parseNodes")
80 81