Fix: Cast numeric write-concern values from XML mapping to integers (#2750)

When using write-concern="1" in XML mapping, the value was being cast to string,
causing MongoDB to interpret it as a named write concern mode instead of a numeric level.
This led to errors like: "No write concern mode named '1' found in replica set configuration".

This commit ensures that numeric write concern values are properly cast to integers,
while still allowing named modes such as "majority" to be used as strings.

---------

Co-authored-by: Jérôme Tamarelle <[email protected]>

Author: ikallali

lib/Doctrine/ODM/MongoDB/Mapping/Driver/XmlDriver.php

@@ -158,7 +158,12 @@ public function loadMetadataForClass($className, \Doctrine\Persistence\Mapping\C
         }
 
         if (isset($xmlRoot['write-concern'])) {
-            $metadata->setWriteConcern((string) $xmlRoot['write-concern']);
+            $writeConcern = (string) $xmlRoot['write-concern'];
+            if (is_numeric($writeConcern)) {
+                $writeConcern = (int) $writeConcern;
+            }
+
+            $metadata->setWriteConcern($writeConcern);
         }
 
         if (isset($xmlRoot['inheritance-type'])) {

tests/Doctrine/ODM/MongoDB/Tests/Mapping/AbstractAnnotationDriverTestCase.php

@@ -148,7 +148,7 @@ public function testEmbeddedClassCantHaveShardKey(): void
     public function testDocumentAnnotationCanSpecifyWriteConcern(): void
     {
         $cm = $this->dm->getClassMetadata(AnnotationDriverTestWriteConcernMajority::class);
-        self::assertEquals('majority', $cm->writeConcern);
+        self::assertSame('majority', $cm->writeConcern);
 
         $cm = $this->dm->getClassMetadata(AnnotationDriverTestWriteConcernUnacknowledged::class);
         self::assertSame(0, $cm->writeConcern);

tests/Doctrine/ODM/MongoDB/Tests/Mapping/AbstractMappingDriverTestCase.php

@@ -101,7 +101,7 @@ public function testDocumentLevelReadPreference(ClassMetadata $class): ClassMeta
     #[Depends('testDocumentCollectionNameAndInheritance')]
     public function testDocumentLevelWriteConcern(ClassMetadata $class): ClassMetadata
     {
-        self::assertEquals(1, $class->getWriteConcern());
+        self::assertSame(1, $class->getWriteConcern());
 
         return $class;
     }