Merge 1.8.x up into 2.0.x

.github/workflows/coding-standards.yml

@@ -11,4 +11,4 @@ on:
 jobs:
   coding-standards:
     name: "Coding Standards"
-    uses: "doctrine/.github/.github/workflows/coding-standards.yml@5.2.0"
+    uses: "doctrine/.github/.github/workflows/coding-standards.yml@6.0.0"

.github/workflows/documentation.yml

@@ -17,4 +17,4 @@ on:
 jobs:
   documentation:
     name: "Documentation"
-    uses: "doctrine/.github/.github/workflows/documentation.yml@5.2.0"
+    uses: "doctrine/.github/.github/workflows/documentation.yml@6.0.0"

.github/workflows/release-on-milestone-closed.yml

@@ -8,7 +8,7 @@ on:
 jobs:
   release:
     name: "Git tag, release & create merge-up PR"
-    uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@5.2.0"
+    uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@6.0.0"
     secrets:
       GIT_AUTHOR_EMAIL: ${{ secrets.GIT_AUTHOR_EMAIL }}
       GIT_AUTHOR_NAME: ${{ secrets.GIT_AUTHOR_NAME }}

.github/workflows/static-analysis.yml

@@ -11,4 +11,4 @@ on:
 jobs:
   static-analysis:
     name: "Static Analysis"
-    uses: "doctrine/.github/.github/workflows/phpstan.yml@5.2.0"
+    uses: "doctrine/.github/.github/workflows/phpstan.yml@6.0.0"

phpstan-baseline.neon

@@ -1,5 +1,30 @@
 parameters:
 	ignoreErrors:
+		-
+			message: "#^Strict comparison using \\=\\=\\= between array\\<string\\> and false will always evaluate to false\\.$#"
+			count: 1
+			path: src/Event/Listener/MongoDBReferenceListener.php
+
+		-
+			message: "#^Strict comparison using \\=\\=\\= between array\\<string\\> and false will always evaluate to false\\.$#"
+			count: 1
+			path: src/Event/Listener/ORMReferenceListener.php
+
+		-
+			message: "#^Method class@anonymous/src/Executor/AbstractExecutor\\.php\\:131\\:\\:log\\(\\) has parameter \\$message with no type specified\\.$#"
+			count: 1
+			path: src/Executor/AbstractExecutor.php
+
+		-
+			message: "#^Method class@anonymous/src/Executor/AbstractExecutor\\.php\\:95\\:\\:log\\(\\) has parameter \\$message with no type specified\\.$#"
+			count: 1
+			path: src/Executor/AbstractExecutor.php
+
+		-
+			message: "#^Strict comparison using \\!\\=\\= between null and null will always evaluate to false\\.$#"
+			count: 1
+			path: src/Executor/AbstractExecutor.php
+
 		-
 			message: "#^Call to method transactional\\(\\) on an unknown class Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManagerInterface\\.$#"
 			count: 1
@@ -10,6 +35,16 @@ parameters:
 			count: 1
 			path: src/Executor/PHPCRExecutor.php
 
+		-
+			message: "#^Parameter \\#1 \\$dm of method Doctrine\\\\Common\\\\DataFixtures\\\\Purger\\\\PHPCRPurger\\:\\:setDocumentManager\\(\\) expects Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManager, Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManagerInterface given\\.$#"
+			count: 1
+			path: src/Executor/PHPCRExecutor.php
+
+		-
+			message: "#^Parameter \\#1 \\$manager of method Doctrine\\\\Common\\\\DataFixtures\\\\Executor\\\\AbstractExecutor\\:\\:__construct\\(\\) expects Doctrine\\\\Persistence\\\\ObjectManager, Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManagerInterface given\\.$#"
+			count: 1
+			path: src/Executor/PHPCRExecutor.php
+
 		-
 			message: "#^Parameter \\$dm of method Doctrine\\\\Common\\\\DataFixtures\\\\Executor\\\\PHPCRExecutor\\:\\:__construct\\(\\) has invalid type Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManagerInterface\\.$#"
 			count: 2
@@ -25,6 +60,36 @@ parameters:
 			count: 1
 			path: src/Loader.php
 
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Loader\\:\\:addFixture\\(\\) has no return type specified\\.$#"
+			count: 1
+			path: src/Loader.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Loader\\:\\:createFixture\\(\\) should return Doctrine\\\\Common\\\\DataFixtures\\\\FixtureInterface but returns object\\.$#"
+			count: 1
+			path: src/Loader.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Loader\\:\\:getUnsequencedClasses\\(\\) should return array\\<class\\-string\\<Doctrine\\\\Common\\\\DataFixtures\\\\FixtureInterface\\>\\> but returns array\\<int\\<0, max\\>, class\\-string\\<Doctrine\\\\Common\\\\DataFixtures\\\\DependentFixtureInterface\\>\\|class\\-string\\<Doctrine\\\\Common\\\\DataFixtures\\\\FixtureInterface\\>\\>\\.$#"
+			count: 1
+			path: src/Loader.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Loader\\:\\:loadFromDirectory\\(\\) return type has no value type specified in iterable type array\\.$#"
+			count: 1
+			path: src/Loader.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Loader\\:\\:loadFromFile\\(\\) return type has no value type specified in iterable type array\\.$#"
+			count: 1
+			path: src/Loader.php
+
+		-
+			message: "#^Parameter \\#1 \\$sequences of method Doctrine\\\\Common\\\\DataFixtures\\\\Loader\\:\\:getUnsequencedClasses\\(\\) expects array\\<class\\-string\\<Doctrine\\\\Common\\\\DataFixtures\\\\DependentFixtureInterface\\>, int\\>, array\\<class\\-string\\<Doctrine\\\\Common\\\\DataFixtures\\\\DependentFixtureInterface\\>\\|class\\-string\\<Doctrine\\\\Common\\\\DataFixtures\\\\FixtureInterface\\>, int\\> given\\.$#"
+			count: 2
+			path: src/Loader.php
+
 		-
 			message: "#^Call to an undefined method Doctrine\\\\Persistence\\\\ObjectManager\\:\\:getReference\\(\\)\\.$#"
 			count: 2
@@ -35,6 +100,46 @@ parameters:
 			count: 1
 			path: src/ProxyReferenceRepository.php
 
+		-
+			message: "#^Expression on left side of \\?\\? is not nullable\\.$#"
+			count: 1
+			path: src/Purger/ORMPurger.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Purger\\\\ORMPurger\\:\\:getAssociationTables\\(\\) has parameter \\$classes with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata but does not specify its types\\: T$#"
+			count: 1
+			path: src/Purger/ORMPurger.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Purger\\\\ORMPurger\\:\\:getCommitOrder\\(\\) has parameter \\$classes with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata but does not specify its types\\: T$#"
+			count: 1
+			path: src/Purger/ORMPurger.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Purger\\\\ORMPurger\\:\\:getCommitOrder\\(\\) return type with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata does not specify its types\\: T$#"
+			count: 1
+			path: src/Purger/ORMPurger.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Purger\\\\ORMPurger\\:\\:getJoinTableName\\(\\) has parameter \\$class with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata but does not specify its types\\: T$#"
+			count: 1
+			path: src/Purger/ORMPurger.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Purger\\\\ORMPurger\\:\\:getTableName\\(\\) has parameter \\$class with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata but does not specify its types\\: T$#"
+			count: 1
+			path: src/Purger/ORMPurger.php
+
+		-
+			message: "#^Parameter \\#1 \\$assoc of method Doctrine\\\\Common\\\\DataFixtures\\\\Purger\\\\ORMPurger\\:\\:getJoinTableName\\(\\) expects array\\|Doctrine\\\\ORM\\\\Mapping\\\\ManyToManyOwningSideMapping, Doctrine\\\\ORM\\\\Mapping\\\\ManyToManyInverseSideMapping\\|Doctrine\\\\ORM\\\\Mapping\\\\ManyToManyOwningSideMapping\\|Doctrine\\\\ORM\\\\Mapping\\\\ManyToOneAssociationMapping\\|Doctrine\\\\ORM\\\\Mapping\\\\OneToManyAssociationMapping\\|Doctrine\\\\ORM\\\\Mapping\\\\OneToOneInverseSideMapping\\|Doctrine\\\\ORM\\\\Mapping\\\\OneToOneOwningSideMapping given\\.$#"
+			count: 1
+			path: src/Purger/ORMPurger.php
+
+		-
+			message: "#^Parameter \\#1 \\$association of method Doctrine\\\\ORM\\\\Mapping\\\\QuoteStrategy\\:\\:getJoinTableName\\(\\) expects Doctrine\\\\ORM\\\\Mapping\\\\ManyToManyOwningSideMapping, array\\|Doctrine\\\\ORM\\\\Mapping\\\\ManyToManyOwningSideMapping given\\.$#"
+			count: 1
+			path: src/Purger/ORMPurger.php
+
 		-
 			message: "#^Call to method getPhpcrSession\\(\\) on an unknown class Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManagerInterface\\.$#"
 			count: 1
@@ -50,6 +155,11 @@ parameters:
 			count: 1
 			path: src/Purger/PHPCRPurger.php
 
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Purger\\\\PHPCRPurger\\:\\:setDocumentManager\\(\\) has no return type specified\\.$#"
+			count: 1
+			path: src/Purger/PHPCRPurger.php
+
 		-
 			message: "#^Parameter \\$dm of method Doctrine\\\\Common\\\\DataFixtures\\\\Purger\\\\PHPCRPurger\\:\\:__construct\\(\\) has invalid type Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManagerInterface\\.$#"
 			count: 1
@@ -85,11 +195,76 @@ parameters:
 			count: 2
 			path: src/ReferenceRepository.php
 
+		-
+			message: "#^Call to an undefined method object\\:\\:getDocumentId\\(\\)\\.$#"
+			count: 1
+			path: src/ReferenceRepository.php
+
+		-
+			message: "#^Call to an undefined method object\\:\\:getDocumentIdentifier\\(\\)\\.$#"
+			count: 1
+			path: src/ReferenceRepository.php
+
 		-
 			message: "#^Class Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManager not found\\.$#"
 			count: 2
 			path: src/ReferenceRepository.php
 
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\ReferenceRepository\\:\\:getIdentifier\\(\\) return type has no value type specified in iterable type array\\.$#"
+			count: 1
+			path: src/ReferenceRepository.php
+
+		-
+			message: "#^Parameter \\#1 \\$className of method Doctrine\\\\Persistence\\\\ObjectManager\\:\\:getClassMetadata\\(\\) expects class\\-string\\<object\\>, string given\\.$#"
+			count: 2
+			path: src/ReferenceRepository.php
+
+		-
+			message: "#^Property Doctrine\\\\Common\\\\DataFixtures\\\\ReferenceRepository\\:\\:\\$identitiesByClass \\(array\\<class\\-string, array\\<string, mixed\\>\\>\\) does not accept array\\<string, array\\<string, mixed\\>\\>\\.$#"
+			count: 2
+			path: src/ReferenceRepository.php
+
+		-
+			message: "#^Property Doctrine\\\\Common\\\\DataFixtures\\\\ReferenceRepository\\:\\:\\$referencesByClass \\(array\\<class\\-string, array\\<string, object\\>\\>\\) does not accept array\\<string, array\\<string, mixed\\>\\>\\.$#"
+			count: 1
+			path: src/ReferenceRepository.php
+
+		-
+			message: "#^Property Doctrine\\\\Common\\\\DataFixtures\\\\ReferenceRepository\\:\\:\\$referencesByClass \\(array\\<class\\-string, array\\<string, object\\>\\>\\) does not accept array\\<string, array\\<string, object\\>\\>\\.$#"
+			count: 1
+			path: src/ReferenceRepository.php
+
+		-
+			message: "#^Unable to resolve the template type T in call to method Doctrine\\\\Persistence\\\\ObjectManager\\:\\:getClassMetadata\\(\\)$#"
+			count: 2
+			path: src/ReferenceRepository.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Sorter\\\\TopologicalSorter\\:\\:addNode\\(\\) has parameter \\$node with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata but does not specify its types\\: T$#"
+			count: 1
+			path: src/Sorter/TopologicalSorter.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Sorter\\\\TopologicalSorter\\:\\:sort\\(\\) return type with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata does not specify its types\\: T$#"
+			count: 1
+			path: src/Sorter/TopologicalSorter.php
+
+		-
+			message: "#^Property Doctrine\\\\Common\\\\DataFixtures\\\\Sorter\\\\TopologicalSorter\\:\\:\\$sortedNodeList with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata does not specify its types\\: T$#"
+			count: 1
+			path: src/Sorter/TopologicalSorter.php
+
+		-
+			message: "#^Method Doctrine\\\\Common\\\\DataFixtures\\\\Sorter\\\\Vertex\\:\\:__construct\\(\\) has parameter \\$value with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata but does not specify its types\\: T$#"
+			count: 1
+			path: src/Sorter/Vertex.php
+
+		-
+			message: "#^Property Doctrine\\\\Common\\\\DataFixtures\\\\Sorter\\\\Vertex\\:\\:\\$value with generic class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata does not specify its types\\: T$#"
+			count: 1
+			path: src/Sorter/Vertex.php
+
 		-
 			message: "#^Method Doctrine\\\\Tests\\\\Common\\\\DataFixtures\\\\FixtureWithUnexistentDependency\\:\\:getDependencies\\(\\) should return array\\<class\\-string\\<Doctrine\\\\Common\\\\DataFixtures\\\\FixtureInterface\\>\\> but returns array\\<int, string\\>\\.$#"
 			count: 1
@@ -105,7 +280,62 @@ parameters:
 			count: 2
 			path: tests/Common/DataFixtures/Executor/PHPCRExecutorTest.php
 
+		-
+			message: "#^Parameter \\#1 \\$dm of class Doctrine\\\\Common\\\\DataFixtures\\\\Executor\\\\PHPCRExecutor constructor expects Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManagerInterface, Doctrine\\\\ODM\\\\PHPCR\\\\DocumentManager&PHPUnit\\\\Framework\\\\MockObject\\\\MockObject given\\.$#"
+			count: 5
+			path: tests/Common/DataFixtures/Executor/PHPCRExecutorTest.php
+
 		-
 			message: "#^Call to an undefined method Doctrine\\\\ODM\\\\MongoDB\\\\DocumentManager\\:\\:getConnection\\(\\)\\.$#"
 			count: 1
 			path: tests/Common/DataFixtures/Purger/MongoDBPurgerTest.php
+
+		-
+			message: "#^Parameter \\#1 \\$name of class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata constructor expects class\\-string\\<1\\>, string given\\.$#"
+			count: 6
+			path: tests/Common/DataFixtures/Sorter/TopologicalSorterTest.php
+
+		-
+			message: "#^Parameter \\#1 \\$name of class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata constructor expects class\\-string\\<2\\>, string given\\.$#"
+			count: 5
+			path: tests/Common/DataFixtures/Sorter/TopologicalSorterTest.php
+
+		-
+			message: "#^Parameter \\#1 \\$name of class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata constructor expects class\\-string\\<3\\>, string given\\.$#"
+			count: 5
+			path: tests/Common/DataFixtures/Sorter/TopologicalSorterTest.php
+
+		-
+			message: "#^Parameter \\#1 \\$name of class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata constructor expects class\\-string\\<4\\>, string given\\.$#"
+			count: 3
+			path: tests/Common/DataFixtures/Sorter/TopologicalSorterTest.php
+
+		-
+			message: "#^Parameter \\#1 \\$name of class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata constructor expects class\\-string\\<5\\>, string given\\.$#"
+			count: 3
+			path: tests/Common/DataFixtures/Sorter/TopologicalSorterTest.php
+
+		-
+			message: "#^Parameter \\#1 \\$name of class Doctrine\\\\ORM\\\\Mapping\\\\ClassMetadata constructor expects class\\-string\\<\\\\Sample\\\\Entity\\>, string given\\.$#"
+			count: 1
+			path: tests/Common/DataFixtures/Sorter/VertexTest.php
+
+		-
+			message: "#^Method Doctrine\\\\Tests\\\\Common\\\\DataFixtures\\\\TestEntity\\\\Quoted\\:\\:getSelects\\(\\) return type with generic interface Doctrine\\\\Common\\\\Collections\\\\Collection does not specify its types\\: TKey, T$#"
+			count: 1
+			path: tests/Common/DataFixtures/TestEntity/Quoted.php
+
+		-
+			message: "#^Method Doctrine\\\\Tests\\\\Common\\\\DataFixtures\\\\TestEntity\\\\Quoted\\:\\:setSelects\\(\\) has parameter \\$selects with generic interface Doctrine\\\\Common\\\\Collections\\\\Collection but does not specify its types\\: TKey, T$#"
+			count: 1
+			path: tests/Common/DataFixtures/TestEntity/Quoted.php
+
+		-
+			message: "#^Property Doctrine\\\\Tests\\\\Common\\\\DataFixtures\\\\TestEntity\\\\Quoted\\:\\:\\$selects with generic interface Doctrine\\\\Common\\\\Collections\\\\Collection does not specify its types\\: TKey, T$#"
+			count: 1
+			path: tests/Common/DataFixtures/TestEntity/Quoted.php
+
+		-
+			message: "#^Method Doctrine\\\\Tests\\\\Common\\\\DataFixtures\\\\TestValueObjects\\\\Uuid\\:\\:__unserialize\\(\\) has parameter \\$data with no value type specified in iterable type array\\.$#"
+			count: 1
+			path: tests/Common/DataFixtures/TestValueObjects/Uuid.php

phpstan.neon.dist

@@ -1,6 +1,6 @@
 parameters:
     phpVersion: 80200
-    level: 3
+    level: 7
     paths:
         - src
         - tests