fix phpstan errors

Author: krzysztof-ciszewski

src/Mapping/ClassMetadata.php

@@ -487,8 +487,8 @@ final class ClassMetadata implements BaseClassMetadata
 
     /**
      * The name of the mongo collection the document is mapped to.
-     *     */
-    public private(set) string $collection = '';
+     */
+    public private(set) ?string $collection = null;
 
     /**
      * The name of the GridFS bucket the document is mapped to.
@@ -576,18 +576,18 @@ final class ClassMetadata implements BaseClassMetadata
     /**
      * The name of the document class.
      *
-     * @var class-string<T>
+     * @var class-string<T>|null
      */
-    public private(set) string $name = '';
+    public private(set) ?string $name = null;
 
     /**
      * The name of the document class that is at the root of the mapped document inheritance
      * hierarchy. If the document is not part of a mapped inheritance hierarchy this is the same
      * as {@link $documentName}.
      *
-     * @var class-string
+     * @var class-string|null
      */
-    public private(set) string $rootDocumentName = '';
+    public private(set) ?string $rootDocumentName = null;
 
     /**
      * The name of the custom repository class used for the document class.
@@ -972,7 +972,7 @@ public function invokeLifecycleCallbacks(string $event, object $document, ?array
             return;
         }
 
-        if (! $document instanceof $this->name) {
+        if ($this->name !== null && ! $document instanceof $this->name) {
             throw new InvalidArgumentException(sprintf('Expected document class "%s"; found: "%s"', $this->name, $document::class));
         }
 

src/Persisters/DocumentPersister.php

@@ -1354,7 +1354,7 @@ private function prepareQueryExpression(array $expression, ClassMetadata $class)
             // Process query operators whose argument arrays need type conversion
             if (in_array($k, ['$in', '$nin', '$all']) && is_array($v)) {
                 foreach ($v as $k2 => $v2) {
-                    if ($v2 instanceof $class->name) {
+                    if ($class->name !== null && $v2 instanceof $class->name) {
                         // If a value in a query is a target document, e.g. ['referenceField' => $targetDocument],
                         // retreive id from target document and convert this id using it's type
                         $expression[$k][$k2] = $class->getDatabaseIdentifierValue($class->getIdentifierValue($v2));
@@ -1381,7 +1381,7 @@ private function prepareQueryExpression(array $expression, ClassMetadata $class)
                 continue;
             }
 
-            if ($v instanceof $class->name) {
+            if ($class->name !== null && $v instanceof $class->name) {
                 $expression[$k] = $class->getDatabaseIdentifierValue($class->getIdentifierValue($v));
             } else {
                 $expression[$k] = $class->getDatabaseIdentifierValue($v);

tests/Tests/Functional/VectorSearchTest.php

@@ -11,6 +11,7 @@
 use MongoDB\Driver\WriteConcern;
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\RequiresPhpExtension;
+use ReflectionProperty;
 
 #[Group('atlas')]
 class VectorSearchTest extends BaseTestCase
@@ -105,8 +106,11 @@ public function testAtlasVectorSearchWithBinaryType(): void
     {
         $cm = $this->dm->getClassMetadata(VectorEmbedding::class);
 
-        $cm->fieldMappings['vectorFloat']['type'] = Type::VECTOR_FLOAT32;
-        $cm->fieldMappings['vectorInt']['type']   = Type::VECTOR_INT8;
+        $fieldMappings                        = $cm->fieldMappings;
+        $fieldMappings['vectorFloat']['type'] = Type::VECTOR_FLOAT32;
+        $fieldMappings['vectorInt']['type']   = Type::VECTOR_INT8;
+
+        new ReflectionProperty($cm, 'fieldMappings')->setValue($cm, $fieldMappings);
 
         // Change the collection name to avoid conflicts with asynchronous index building
         $cm->collection .= '_binary_type';