refactor: argument nillable is made optional.

Author: dapper91

pydantic_xml/fields.py

@@ -36,7 +36,7 @@ class XmlEntityInfoP(typing.Protocol):
     path: Optional[str]
     ns: Optional[str]
     nsmap: Optional[NsMap]
-    nillable: bool
+    nillable: Optional[bool]
     wrapped: Optional['XmlEntityInfoP']
 
 
@@ -54,7 +54,7 @@ def __init__(
             path: Optional[str] = None,
             ns: Optional[str] = None,
             nsmap: Optional[NsMap] = None,
-            nillable: bool = False,
+            nillable: Optional[bool] = None,
             wrapped: Optional[pd.fields.FieldInfo] = None,
             **kwargs: Any,
     ):
@@ -113,7 +113,7 @@ def element(
         tag: Optional[str] = None,
         ns: Optional[str] = None,
         nsmap: Optional[NsMap] = None,
-        nillable: bool = False,
+        nillable: Optional[bool] = None,
         *,
         default: Any = pdc.PydanticUndefined,
         default_factory: Optional[Callable[[], Any]] = _Unset,
@@ -179,7 +179,7 @@ class ComputedXmlEntityInfo(pd.fields.ComputedFieldInfo):
     path: Optional[str]
     ns: Optional[str]
     nsmap: Optional[NsMap]
-    nillable: bool
+    nillable: Optional[bool]
     wrapped: Optional[XmlEntityInfoP]  # to be compliant with XmlEntityInfoP protocol
 
     def __post_init__(self) -> None:
@@ -199,7 +199,7 @@ def decorator(prop: Any) -> Any:
         path = kwargs.pop('path', None)
         ns = kwargs.pop('ns', None)
         nsmap = kwargs.pop('nsmap', None)
-        nillable = kwargs.pop('nillable', False)
+        nillable = kwargs.pop('nillable', None)
 
         descriptor_proxy = pd.computed_field(**kwargs)(prop)
         descriptor_proxy.decorator_info = ComputedXmlEntityInfo(
@@ -245,7 +245,7 @@ def computed_element(
         tag: Optional[str] = None,
         ns: Optional[str] = None,
         nsmap: Optional[NsMap] = None,
-        nillable: bool = False,
+        nillable: Optional[bool] = None,
         **kwargs: Any,
 ) -> Union[PropertyT, Callable[[PropertyT], PropertyT]]:
     """

pydantic_xml/serializers/factories/model.py

@@ -357,7 +357,7 @@ def __init__(
             nsmap: Optional[NsMap],
             search_mode: SearchMode,
             computed: bool,
-            nillable: bool,
+            nillable: Optional[bool],
     ):
         self._model = model
         self._element_name = QName.from_alias(tag=name, ns=ns, nsmap=nsmap).uri

pydantic_xml/serializers/factories/primitive.py

@@ -36,7 +36,7 @@ def from_core_schema(cls, schema: PrimitiveTypeSchema, ctx: Serializer.Context)
 
         return cls(computed, nillable)
 
-    def __init__(self, computed: bool, nillable: bool):
+    def __init__(self, computed: bool, nillable: Optional[bool]):
         self._computed = computed
         self._nillable = nillable
 
@@ -163,7 +163,7 @@ def __init__(
             nsmap: Optional[NsMap],
             search_mode: SearchMode,
             computed: bool,
-            nillable: bool,
+            nillable: Optional[bool],
     ):
         super().__init__(computed, nillable)
 

pydantic_xml/serializers/serializer.py

@@ -133,7 +133,7 @@ def entity_nsmap(self) -> Optional[NsMap]:
             return self.entity_info.nsmap if self.entity_info is not None else None
 
         @property
-        def nillable(self) -> bool:
+        def nillable(self) -> Optional[bool]:
             return self.entity_info.nillable if self.entity_info is not None else False
 
         @property