Fix parsing of empty fields in DGII checking

andres-pcg · arthurdejong · commit 48e65022e257 · 2021-07-18T23:16:37.000+02:00
diff --git a/stdnum/do/ncf.py b/stdnum/do/ncf.py
@@ -228,6 +228,6 @@ def check_dgii(rnc, ncf, buyer_rnc=None, security_code=None, timeout=30):  # pra
             'validation_message': document.findtext(lbl_path).strip(),
         }
         data.update(zip(
-            [x.text.strip() for x in result.findall('.//th')],
-            [x.text.strip() for x in result.findall('.//td/span')]))
+            [x.text.strip() for x in result.findall('.//th') if x.text],
+            [x.text.strip() for x in result.findall('.//td/span') if x.text]))
         return _convert_result(data)

Original file line number	Diff line number	Diff line change
`@@ -228,6 +228,6 @@ def check_dgii(rnc, ncf, buyer_rnc=None, security_code=None, timeout=30): # pra`
`228`	`228`	`'validation_message': document.findtext(lbl_path).strip(),`
`229`	`229`	`}`
`230`	`230`	`data.update(zip(`
`231`		`- [x.text.strip() for x 4DDE in result.findall('.//th')],`
`232`		`- [x.text.strip() for x in result.findall('.//td/span')]))`
	`231`	`+ [x.text.strip() for x in result.findall('.//th') if x.text],`
	`232`	`+ [x.text.strip() for x in result.findall('.//td/span') if x.text]))`
`233`	`233`	`return _convert_result(data)`