With nullable typed readonly properties (from PHP 8.1+) we face situations where properties are not set after deserialization (Typed property...must not be accessed before initialization). This subscriber helps to handle these cases by adding properties to the payload so that they end up being null.

Author: mathielen

src/CXml/Model/ItemDetail.php

@@ -48,7 +48,7 @@ protected function __construct(
         private readonly MoneyWrapper $unitPrice,
         #[Serializer\SerializedName('PriceBasisQuantity')]
         #[Serializer\XmlElement(cdata: false)]
-        private readonly ?PriceBasisQuantity $priceBasisQuantity,
+        private readonly ?PriceBasisQuantity $priceBasisQuantity = null,
     ) {
     }
 

src/CXml/Model/Message/PunchOutOrderMessageHeader.php

@@ -23,7 +23,7 @@ class PunchOutOrderMessageHeader
     final public const OPERATION_INSPECT = 'inspect';
 
     #[Serializer\XmlAttribute]
-    private readonly ?string $operationAllowed;
+    private ?string $operationAllowed = null; /* cant be 'readonly' bc must be initialized with null -> jms deserialization */
 
     #[Serializer\SerializedName('ShipTo')]
     private ?ShipTo $shipTo = null;

src/CXml/Model/MultilanguageString.php

@@ -10,7 +10,7 @@ class MultilanguageString
 {
     public function __construct(
         #[Serializer\XmlValue(cdata: false)]
-        private readonly ?string $value,
+        private readonly ?string $value = null,
         #[Serializer\XmlAttribute]
         private readonly ?string $type = null,
         #[Serializer\XmlAttribute(namespace: 'http://www.w3.org/XML/1998/namespace')]

src/CXml/Model/Request/OrderRequestHeader.php

@@ -43,16 +43,18 @@ class OrderRequestHeader
     #[Serializer\Type('array<CXml\Model\BusinessPartner>')]
     private array $businessPartners;
 
+    #[Serializer\XmlElement]
+    #[Serializer\SerializedName('ShipTo')]
+    private ?ShipTo $shipTo = null; /* cant be 'readonly' bc must be initialized with null -> jms deserialization */
+
     protected function __construct(
         #[Serializer\XmlAttribute]
         #[Serializer\SerializedName('orderID')]
         private readonly string $orderId,
         #[Serializer\XmlAttribute]
         #[Serializer\SerializedName('orderDate')]
         private readonly DateTimeInterface $orderDate,
-        #[Serializer\XmlElement]
-        #[Serializer\SerializedName('ShipTo')]
-        private readonly ?ShipTo $shipTo,
+        ?ShipTo $shipTo, /* cant be 'readonly' bc must be initialized with null -> jms deserialization */
         #[Serializer\XmlElement]
         #[Serializer\SerializedName('BillTo')]
         private readonly BillTo $billTo,
@@ -67,15 +69,11 @@ protected function __construct(
         #[Serializer\XmlList(entry: 'Contact', inline: true)]
         private ?array $contacts = null,
     ) {
-        if (null === $contacts) {
-            return;
-        }
+        $this->shipTo = $shipTo;
 
-        if ([] === $contacts) {
-            return;
+        if (null !== $contacts) {
+            Assertion::allIsInstanceOf($contacts, Contact::class);
         }
-
-        Assertion::allIsInstanceOf($contacts, Contact::class);
     }
 
     public static function create(

tests/CXmlTest/Model/SerializerTest.php

@@ -410,4 +410,54 @@ public function testDeserializeOneRowXml(): void
 
         $this->assertXmlStringEqualsXmlString($xml, $resultingXml);
     }
+
+    public function testDeserializeNullProperty(): void
+    {
+        $xml =
+            '<?xml version="1.0" encoding="UTF-8"?>
+			<cXML payloadID="payload-id" timestamp="2000-01-01T00:00:00+00:00">
+			<Header>
+			<From>
+			<Credential domain="AribaNetworkUserId">
+			<Identity>admin@acme.com</Identity>
+			</Credential>
+			</From>
+			<To>
+			<Credential domain="DUNS">
+			<Identity>012345678</Identity>
+			</Credential>
+			</To>
+			<Sender>
+			<Credential domain="AribaNetworkUserId">
+			<Identity>sysadmin@buyer.com</Identity>
+			<SharedSecret>abracadabra</SharedSecret>
+			</Credential>
+			<UserAgent>Network Hub 1.1</UserAgent>
+			</Sender>
+			</Header>
+			<Request>
+				<OrderRequest>
+				  <OrderRequestHeader orderDate="2000-01-01" orderID="order-id" type="new">
+					<Total>
+					  <Money currency="EUR">0.00</Money>
+					</Total>
+					<BillTo>
+					  <Address>
+						<Name xml:lang="en">name</Name>
+					  </Address>
+					</BillTo>
+				  </OrderRequestHeader>
+				</OrderRequest>
+			</Request>
+			</cXML>';
+
+        $cxml = Serializer::create()->deserialize($xml);
+
+        /** @var OrderRequest $orderRequest */
+        $orderRequest = $cxml->getRequest()->getPayload();
+
+        // Error: Typed property CXml\Model\Request\OrderRequestHeader::$shipTo must not be accessed before initialization
+        $shipTo = $orderRequest->getOrderRequestHeader()->getShipTo();
+        $this->assertNull($shipTo);
+    }
 }