fix: Handle empty <Paket> definition (mypy was right) (#41)

hf-kklein · web-flow · commit a049dbaeab74 · 2024-10-02T11:30:39.000+02:00
fix: Handle empty `&lt;Paket&gt;` definition

in those cases, the `element.text` is `None` and not empty string
fixes: "AttributeError: 'NoneType' object has no attribute 'strip'"
diff --git a/src/fundamend/reader/ahbreader.py b/src/fundamend/reader/ahbreader.py
@@ -60,7 +60,7 @@ def _to_ub_bedingung(element: ET.Element) -> UbBedingung:
 def _to_paket(element: ET.Element) -> Paket:
     return Paket(
         nummer=element.attrib["Nummer"].lstrip("[").rstrip("]"),
-        text=element.text.strip(),  # type:ignore[union-attr]
+        text=(element.text or "").strip(),
     )
 
 

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ def _to_ub_bedingung(element: ET.Element) -> UbBedingung:`
`60`	`60`	`def _to_paket(element: ET.Element) -> Paket:`
`61`	`61`	`return Paket(`
`62`	`62`	`nummer=element.attrib["Nummer"].lstrip("[").rstrip("]"),`
`63`		`- text=element.text.strip(), # type:ignore[union-attr]`
	`63`	`+ text=(element.text or "").strip(),`
`64`	`64`	`)`
`65`	`65`
`66`	`66`