Describe some types more accurately

Downstream builds are broken because the last release of persistence
describes its inputs more accurately, but not its outputs.

lib/Doctrine/Persistence/Mapping/Driver/AnnotationDriver.php

@@ -61,6 +61,7 @@ abstract class AnnotationDriver implements MappingDriver
      * Cache for AnnotationDriver#getAllClassNames().
      *
      * @var string[]|null
+     * @psalm-var list<class-string>|null
      */
     protected $classNames;
 

lib/Doctrine/Persistence/Mapping/Driver/DefaultFileLocator.php

@@ -136,7 +136,9 @@ public function getAllClassNames($globalBasename)
                     }
 
                     // NOTE: All files found here means classes are not transient!
-                    $classes[] = str_replace('.', '\\', $fileName);
+                    /** @psalm-var class-string */
+                    $class     = str_replace('.', '\\', $fileName);
+                    $classes[] = $class;
                 }
             }
         }

lib/Doctrine/Persistence/Mapping/Driver/FileDriver.php

@@ -8,6 +8,7 @@
 use function array_keys;
 use function array_merge;
 use function array_unique;
+use function array_values;
 use function is_file;
 use function str_replace;
 
@@ -26,7 +27,7 @@ abstract class FileDriver implements MappingDriver
 
     /**
      * @var ClassMetadata[]|null
-     * @psalm-var ClassMetadata<object>[]|null
+     * @psalm-var array<class-string, ClassMetadata<object>>|null
      */
     protected $classCache;
 
@@ -77,6 +78,7 @@ public function getGlobalBasename()
      * This will lazily load the mapping file if it is not loaded yet.
      *
      * @param string $className
+     * @psalm-param class-string $className
      *
      * @return ClassMetadata The element of schema meta data.
      * @psalm-return ClassMetadata<object>
@@ -132,10 +134,10 @@ public function getAllClassNames()
             return (array) $this->locator->getAllClassNames($this->globalBasename);
         }
 
-        return array_unique(array_merge(
+        return array_values(array_unique(array_merge(
             array_keys($this->classCache),
             (array) $this->locator->getAllClassNames($this->globalBasename)
-        ));
+        )));
     }
 
     /**

lib/Doctrine/Persistence/Mapping/Driver/FileLocator.php

@@ -25,6 +25,7 @@ public function findMappingFile($className);
      * @param string|null $globalBasename Passed to allow excluding the basename.
      *
      * @return string[]
+     * @psalm-return list<class-string>
      */
     public function getAllClassNames($globalBasename);
 

lib/Doctrine/Persistence/Mapping/Driver/MappingDriver.php

@@ -13,6 +13,7 @@ interface MappingDriver
      * Loads the metadata for the specified class into the provided container.
      *
      * @param string $className
+     * @psalm-param class-string $className
      * @psalm-param ClassMetadata<object> $metadata
      *
      * @return void
@@ -23,6 +24,7 @@ public function loadMetadataForClass($className, ClassMetadata $metadata);
      * Gets the names of all mapped classes known to this driver.
      *
      * @return string[] The names of all mapped classes known to this driver.
+     * @psalm-return list<class-string>
      */
     public function getAllClassNames();
 

lib/Doctrine/Persistence/Mapping/Driver/StaticPHPDriver.php

@@ -33,6 +33,7 @@ class StaticPHPDriver implements MappingDriver
      * Map of all class names.
      *
      * @var string[]
+     * @psalm-var list<class-string>
      */
     private $classNames;
 

lib/Doctrine/Persistence/Mapping/Driver/SymfonyFileLocator.php

@@ -194,10 +194,14 @@ public function getAllClassNames($globalBasename = null)
                             '\\'
                         );
 
-                        $classes[] = $this->prefixes[$path] . str_replace(DIRECTORY_SEPARATOR, '\\', $nsSuffix) . '\\' . str_replace($this->nsSeparator, '\\', $fileName);
+                        /** @psalm-var class-string */
+                        $class = $this->prefixes[$path] . str_replace(DIRECTORY_SEPARATOR, '\\', $nsSuffix) . '\\' . str_replace($this->nsSeparator, '\\', $fileName);
                     } else {
-                        $classes[] = str_replace($this->nsSeparator, '\\', $fileName);
+                        /** @psalm-var class-string */
+                        $class = str_replace($this->nsSeparator, '\\', $fileName);
                     }
+
+                    $classes[] = $class;
                 }
             }
         }

lib/Doctrine/Persistence/ObjectManager.php

@@ -22,7 +22,7 @@ interface ObjectManager
      * @return object|null The found object.
      * @psalm-return T|null
      *
-     * @template T
+     * @template T of object
      */
     public function find($className, $id);
 

phpstan.neon

@@ -9,4 +9,4 @@ parameters:
         - tests
 
     excludes_analyse:
-        - tests/Doctrine/Tests/Persistence/Mapping/_files/TestEntity.php
+        - tests/Doctrine/Tests/Persistence/Mapping/_files/Doctrine.Tests.Persistence.Mapping.PHPTestEntity.php

psalm.xml

@@ -11,7 +11,8 @@
         <directory name="tests/Doctrine" />
         <ignoreFiles>
             <directory name="vendor" />
-            <file name="tests/Doctrine/Tests/Persistence/Mapping/_files/TestEntity.php" />
+            <!-- this one is a mapping file in written in PHP -->
+            <file name="tests/Doctrine/Tests/Persistence/Mapping/_files/Doctrine.Tests.Persistence.Mapping.PHPTestEntity.php" />
         </ignoreFiles>
     </projectFiles>
     <issueHandlers>

tests/Doctrine/Tests/Persistence/Mapping/PHPDriverTest.php

@@ -14,6 +14,10 @@ public function testLoadMetadata(): void
         $metadata->expects($this->once())->method('getFieldNames');
 
         $driver = new PHPDriver([__DIR__ . '/_files']);
-        $driver->loadMetadataForClass('TestEntity', $metadata);
+        $driver->loadMetadataForClass(PHPTestEntity::class, $metadata);
     }
 }
+
+class PHPTestEntity
+{
+}