Rename DescriptorRegistry to DefaultDescriptorRegistry

Author: ruudk

extension.neon

@@ -75,7 +75,7 @@ services:
 	-
 		class: PHPStan\Type\Doctrine\DescriptorRegistryFactory
 	-
-		class: PHPStan\Type\Doctrine\DescriptorRegistry
+		class: PHPStan\Type\Doctrine\DefaultDescriptorRegistry
 		factory: @PHPStan\Type\Doctrine\DescriptorRegistryFactory::createRegistry
 
 	-

src/Rules/Doctrine/ORM/EntityColumnRule.php

@@ -10,7 +10,7 @@
 use PHPStan\Rules\RuleErrorBuilder;
 use PHPStan\Type\ArrayType;
 use PHPStan\Type\Doctrine\DescriptorNotRegisteredException;
-use PHPStan\Type\Doctrine\DescriptorRegistry;
+use PHPStan\Type\Doctrine\DefaultDescriptorRegistry;
 use PHPStan\Type\Doctrine\ObjectMetadataResolver;
 use PHPStan\Type\ErrorType;
 use PHPStan\Type\MixedType;
@@ -36,7 +36,7 @@ class EntityColumnRule implements Rule
 	/** @var ObjectMetadataResolver */
 	private $objectMetadataResolver;
 
-	/** @var DescriptorRegistry */
+	/** @var DefaultDescriptorRegistry */
 	private $descriptorRegistry;
 
 	/** @var ReflectionProvider */
@@ -53,7 +53,7 @@ class EntityColumnRule implements Rule
 
 	public function __construct(
 		ObjectMetadataResolver $objectMetadataResolver,
-		DescriptorRegistry $descriptorRegistry,
+		DefaultDescriptorRegistry $descriptorRegistry,
 		ReflectionProvider $reflectionProvider,
 		bool $reportUnknownTypes,
 		bool $allowNullablePropertyForRequiredField,

src/Type/Doctrine/CreateQueryDynamicReturnTypeExtension.php

@@ -33,10 +33,10 @@ final class CreateQueryDynamicReturnTypeExtension implements DynamicMethodReturn
 	/** @var ObjectMetadataResolver */
 	private $objectMetadataResolver;
 
-	/** @var DescriptorRegistry */
+	/** @var DefaultDescriptorRegistry */
 	private $descriptorRegistry;
 
-	public function __construct(ObjectMetadataResolver $objectMetadataResolver, DescriptorRegistry $descriptorRegistry)
+	public function __construct(ObjectMetadataResolver $objectMetadataResolver, DefaultDescriptorRegistry $descriptorRegistry)
 	{
 		$this->objectMetadataResolver = $objectMetadataResolver;
 		$this->descriptorRegistry = $descriptorRegistry;

src/Type/Doctrine/DescriptorRegistry.php renamed to src/Type/Doctrine/DefaultDescriptorRegistry.php

@@ -5,7 +5,7 @@
 use Doctrine\DBAL\Types\Type;
 use PHPStan\Type\Doctrine\Descriptors\DoctrineTypeDescriptor;
 
-class DescriptorRegistry
+class DefaultDescriptorRegistry
 {
 
 	/** @var array<class-string<Type>, DoctrineTypeDescriptor> */

src/Type/Doctrine/DescriptorRegistryFactory.php

@@ -17,9 +17,9 @@ public function __construct(Container $container)
 		$this->container = $container;
 	}
 
-	public function createRegistry(): DescriptorRegistry
+	public function createRegistry(): DefaultDescriptorRegistry
 	{
-		return new DescriptorRegistry($this->container->getServicesByTag(self::TYPE_DESCRIPTOR_TAG));
+		return new DefaultDescriptorRegistry($this->container->getServicesByTag(self::TYPE_DESCRIPTOR_TAG));
 	}
 
 }

src/Type/Doctrine/Query/QueryResultTypeWalker.php

@@ -19,7 +19,7 @@
 use PHPStan\Type\Constant\ConstantStringType;
 use PHPStan\Type\ConstantTypeHelper;
 use PHPStan\Type\Doctrine\DescriptorNotRegisteredException;
-use PHPStan\Type\Doctrine\DescriptorRegistry;
+use PHPStan\Type\Doctrine\DefaultDescriptorRegistry;
 use PHPStan\Type\FloatType;
 use PHPStan\Type\GeneralizePrecision;
 use PHPStan\Type\IntegerRangeType;
@@ -97,7 +97,7 @@ class QueryResultTypeWalker extends SqlWalker
 	/** @var QueryResultTypeBuilder */
 	private $typeBuilder;
 
-	/** @var DescriptorRegistry */
+	/** @var DefaultDescriptorRegistry */
 	private $descriptorRegistry;
 
 	/** @var bool */
@@ -109,7 +109,7 @@ class QueryResultTypeWalker extends SqlWalker
 	/**
 	 * @param Query<mixed> $query
 	 */
-	public static function walk(Query $query, QueryResultTypeBuilder $typeBuilder, DescriptorRegistry $descriptorRegistry): void
+	public static function walk(Query $query, QueryResultTypeBuilder $typeBuilder, DefaultDescriptorRegistry $descriptorRegistry): void
 	{
 		$query->setHint(Query::HINT_CUSTOM_OUTPUT_WALKER, self::class);
 		$query->setHint(self::HINT_TYPE_MAPPING, $typeBuilder);
@@ -154,11 +154,11 @@ public function __construct($query, $parserResult, array $queryComponents)
 
 		$descriptorRegistry = $this->query->getHint(self::HINT_DESCRIPTOR_REGISTRY);
 
-		if (!$descriptorRegistry instanceof DescriptorRegistry) {
+		if (!$descriptorRegistry instanceof DefaultDescriptorRegistry) {
 			throw new ShouldNotHappenException(sprintf(
 				'Expected the query hint %s to contain a %s, but got a %s',
 				self::HINT_DESCRIPTOR_REGISTRY,
-				DescriptorRegistry::class,
+				DefaultDescriptorRegistry::class,
 				is_object($descriptorRegistry) ? get_class($descriptorRegistry) : gettype($descriptorRegistry)
 			));
 		}

src/Type/Doctrine/QueryBuilder/QueryBuilderGetQueryDynamicReturnTypeExtension.php

@@ -14,7 +14,7 @@
 use PHPStan\Reflection\ParametersAcceptorSelector;
 use PHPStan\Rules\Doctrine\ORM\DynamicQueryBuilderArgumentException;
 use PHPStan\Type\Doctrine\ArgumentsProcessor;
-use PHPStan\Type\Doctrine\DescriptorRegistry;
+use PHPStan\Type\Doctrine\DefaultDescriptorRegistry;
 use PHPStan\Type\Doctrine\DoctrineTypeUtils;
 use PHPStan\Type\Doctrine\ObjectMetadataResolver;
 use PHPStan\Type\Doctrine\Query\QueryResultTypeBuilder;
@@ -62,7 +62,7 @@ class QueryBuilderGetQueryDynamicReturnTypeExtension implements DynamicMethodRet
 	/** @var string|null */
 	private $queryBuilderClass;
 
-	/** @var DescriptorRegistry */
+	/** @var DefaultDescriptorRegistry */
 	private $descriptorRegistry;
 
 	/** @var OtherMethodQueryBuilderParser */
@@ -72,7 +72,7 @@ public function __construct(
 		ObjectMetadataResolver $objectMetadataResolver,
 		ArgumentsProcessor $argumentsProcessor,
 		?string $queryBuilderClass,
-		DescriptorRegistry $descriptorRegistry,
+		DefaultDescriptorRegistry $descriptorRegistry,
 		OtherMethodQueryBuilderParser $otherMethodQueryBuilderParser
 	)
 	{

tests/Rules/Doctrine/ORM/EntityColumnRuleTest.php

@@ -8,7 +8,7 @@
 use Iterator;
 use PHPStan\Rules\Rule;
 use PHPStan\Testing\RuleTestCase;
-use PHPStan\Type\Doctrine\DescriptorRegistry;
+use PHPStan\Type\Doctrine\DefaultDescriptorRegistry;
 use PHPStan\Type\Doctrine\Descriptors\ArrayType;
 use PHPStan\Type\Doctrine\Descriptors\BigIntType;
 use PHPStan\Type\Doctrine\Descriptors\BinaryType;
@@ -58,7 +58,7 @@ protected function getRule(): Rule
 
 		return new EntityColumnRule(
 			new ObjectMetadataResolver($this->objectManagerLoader, __DIR__ . '/../../../../tmp'),
-			new DescriptorRegistry([
+			new DefaultDescriptorRegistry([
 				new ArrayType(),
 				new BigIntType(),
 				new BinaryType(),

tests/Type/Doctrine/Query/QueryResultTypeWalkerTest.php

@@ -17,7 +17,7 @@
 use PHPStan\Type\Constant\ConstantIntegerType;
 use PHPStan\Type\Constant\ConstantStringType;
 use PHPStan\Type\ConstantTypeHelper;
-use PHPStan\Type\Doctrine\DescriptorRegistry;
+use PHPStan\Type\Doctrine\DefaultDescriptorRegistry;
 use PHPStan\Type\FloatType;
 use PHPStan\Type\IntegerRangeType;
 use PHPStan\Type\IntegerType;
@@ -58,7 +58,7 @@ final class QueryResultTypeWalkerTest extends PHPStanTestCase
 	/** @var EntityManagerInterface */
 	private static $em;
 
-	/** @var DescriptorRegistry */
+	/** @var DefaultDescriptorRegistry */
 	private $descriptorRegistry;
 
 	public static function getAdditionalConfigFiles(): array
@@ -184,7 +184,7 @@ public static function tearDownAfterClass(): void
 
 	public function setUp(): void
 	{
-		$this->descriptorRegistry = self::getContainer()->getByType(DescriptorRegistry::class);
+		$this->descriptorRegistry = self::getContainer()->getByType(DefaultDescriptorRegistry::class);
 	}
 
 	/** @dataProvider getTestData */