code quality & restore comment

Author: johanib

library/EngineBlock/Saml2/AuthnRequestAnnotationDecorator.php

@@ -61,12 +61,12 @@ class EngineBlock_Saml2_AuthnRequestAnnotationDecorator extends EngineBlock_Saml
     /**
      * @var string|null Temporary storage for serialized XML
      */
-    private ?string $_serializableSspMessageXml;
+    private ?string $_serializableSspMessageXml = null;
 
     /**
      * @var string|null Persisted RelayState while serialized (not part of the AuthnRequest XML itself)
      */
-    private ?string $_serializableRelayState;
+    private ?string $_serializableRelayState = null;
 
     /**
      * @param AuthnRequest $request

library/EngineBlock/Saml2/ResponseAnnotationDecorator.php

@@ -86,12 +86,12 @@ class EngineBlock_Saml2_ResponseAnnotationDecorator extends EngineBlock_Saml2_Me
     /**
      * @var string Temporary storage for serialized XML
      */
-    private ?string $_serializableSspMessageXml;
+    private ?string $_serializableSspMessageXml = null;
 
     /**
      * @var string|null Persisted RelayState while serialized (not part of the AuthnRequest XML itself)
      */
-    private ?string $_serializableRelayState;
+    private ?string $_serializableRelayState = null;
 
     /**
      * @param Response $response

src/OpenConext/EngineBlockBundle/Doctrine/DqlFunction/Md5.php

@@ -29,7 +29,6 @@ class Md5 extends FunctionNode
 
     public function getSql(SqlWalker $sqlWalker): string
     {
-        // Directly emit the MD5 SQL function instead of using the deprecated platform->getMd5Expression() helper.
         return 'MD5(' . $this->value->dispatch($sqlWalker) . ')';
     }
 

src/OpenConext/EngineBlockBundle/Doctrine/Type/CollabPersonIdType.php

@@ -65,6 +65,7 @@ public function convertToPHPValue($value, AbstractPlatform $platform): mixed
         try {
             $entityId = new CollabPersonId($value);
         } catch (InvalidArgumentException $e) {
+            // get nice standard message, so we can throw it keeping the exception chain
             $doctrineExceptionMessage = sprintf(
                 'Could not convert database value "%s" to Doctrine Type %s. Expected format: %s',
                 $value,