Skip to content

Commit 4f51333

Browse files
committed
Fixes after PHPStan update
1 parent 261b19d commit 4f51333

13 files changed

+33
-11
lines changed

src/Rules/Doctrine/ORM/EntityColumnRule.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
use PHPStan\Type\MixedType;
1717
use PHPStan\Type\NeverType;
1818
use PHPStan\Type\ObjectType;
19-
use PHPStan\Type\ParserNodeTypeToPHPStanType;
2019
use PHPStan\Type\Type;
2120
use PHPStan\Type\TypeCombinator;
2221
use PHPStan\Type\TypehintHelper;
@@ -176,7 +175,7 @@ public function processNode(Node $node, Scope $scope): array
176175
}
177176

178177
$phpDocType = $node->getPhpDocType();
179-
$nativeType = $node->getNativeType() !== null ? ParserNodeTypeToPHPStanType::resolve($node->getNativeType(), $scope->getClassReflection()) : new MixedType();
178+
$nativeType = $node->getNativeType() ?? new MixedType();
180179
$propertyType = TypehintHelper::decideType($nativeType, $phpDocType);
181180

182181
if (get_class($propertyType) === MixedType::class || $propertyType instanceof ErrorType || $propertyType instanceof NeverType) {

src/Rules/Doctrine/ORM/EntityRelationRule.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
use PHPStan\Type\MixedType;
1414
use PHPStan\Type\NeverType;
1515
use PHPStan\Type\ObjectType;
16-
use PHPStan\Type\ParserNodeTypeToPHPStanType;
1716
use PHPStan\Type\TypeCombinator;
1817
use PHPStan\Type\TypehintHelper;
1918
use PHPStan\Type\VerbosityLevel;
@@ -96,7 +95,7 @@ public function processNode(Node $node, Scope $scope): array
9695
}
9796

9897
$phpDocType = $node->getPhpDocType();
99-
$nativeType = $node->getNativeType() !== null ? ParserNodeTypeToPHPStanType::resolve($node->getNativeType(), $scope->getClassReflection()) : new MixedType();
98+
$nativeType = $node->getNativeType() ?? new MixedType();
10099
$propertyType = TypehintHelper::decideType($nativeType, $phpDocType);
101100

102101
$errors = [];

src/Type/Doctrine/DBAL/RowCountMethodDynamicReturnTypeExtension.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
class RowCountMethodDynamicReturnTypeExtension implements DynamicMethodReturnTypeExtension
1717
{
1818

19+
/** @var class-string */
1920
private string $class;
2021

2122
private ObjectMetadataResolver $objectMetadataResolver;
@@ -24,6 +25,9 @@ class RowCountMethodDynamicReturnTypeExtension implements DynamicMethodReturnTyp
2425

2526
private ReflectionProvider $reflectionProvider;
2627

28+
/**
29+
* @param class-string $class
30+
*/
2731
public function __construct(
2832
string $class,
2933
ObjectMetadataResolver $objectMetadataResolver,

src/Type/Doctrine/GetRepositoryDynamicReturnTypeExtension.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,10 +35,14 @@ class GetRepositoryDynamicReturnTypeExtension implements DynamicMethodReturnType
3535

3636
private ?string $odmRepositoryClass = null;
3737

38+
/** @var class-string */
3839
private string $managerClass;
3940

4041
private ObjectMetadataResolver $metadataResolver;
4142

43+
/**
44+
* @param class-string $managerClass
45+
*/
4246
public function __construct(
4347
ReflectionProvider $reflectionProvider,
4448
?string $repositoryClass,

src/Type/Doctrine/QueryBuilder/Expr/NewExprDynamicReturnTypeExtension.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,14 @@ class NewExprDynamicReturnTypeExtension implements DynamicStaticMethodReturnType
2020

2121
private ArgumentsProcessor $argumentsProcessor;
2222

23+
/** @var class-string */
2324
private string $class;
2425

2526
private ReflectionProvider $reflectionProvider;
2627

28+
/**
29+
* @param class-string $class
30+
*/
2731
public function __construct(
2832
ArgumentsProcessor $argumentsProcessor,
2933
string $class,

src/Type/Doctrine/QueryBuilder/QueryBuilderGetDqlDynamicReturnTypeExtension.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,12 @@
1313
class QueryBuilderGetDqlDynamicReturnTypeExtension implements DynamicMethodReturnTypeExtension
1414
{
1515

16+
/** @var class-string|null */
1617
private ?string $queryBuilderClass = null;
1718

19+
/**
20+
* @param class-string|null $queryBuilderClass
21+
*/
1822
public function __construct(
1923
?string $queryBuilderClass
2024
)

src/Type/Doctrine/QueryBuilder/QueryBuilderGetQueryDynamicReturnTypeExtension.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ class QueryBuilderGetQueryDynamicReturnTypeExtension implements DynamicMethodRet
5959

6060
private ArgumentsProcessor $argumentsProcessor;
6161

62+
/** @var class-string|null */
6263
private ?string $queryBuilderClass = null;
6364

6465
private DescriptorRegistry $descriptorRegistry;
@@ -67,6 +68,9 @@ class QueryBuilderGetQueryDynamicReturnTypeExtension implements DynamicMethodRet
6768

6869
private DriverDetector $driverDetector;
6970

71+
/**
72+
* @param class-string|null $queryBuilderClass
73+
*/
7074
public function __construct(
7175
ObjectMetadataResolver $objectMetadataResolver,
7276
ArgumentsProcessor $argumentsProcessor,

src/Type/Doctrine/QueryBuilder/QueryBuilderMethodDynamicReturnTypeExtension.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,12 @@ class QueryBuilderMethodDynamicReturnTypeExtension implements DynamicMethodRetur
2222

2323
private const MAX_COMBINATIONS = 16;
2424

25+
/** @var class-string|null */
2526
private ?string $queryBuilderClass = null;
2627

28+
/**
29+
* @param class-string|null $queryBuilderClass
30+
*/
2731
public function __construct(
2832
?string $queryBuilderClass
2933
)

src/Type/Doctrine/QueryBuilder/QueryBuilderTypeSpecifyingExtension.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,14 @@ class QueryBuilderTypeSpecifyingExtension implements MethodTypeSpecifyingExtensi
2424

2525
private const MAX_COMBINATIONS = 16;
2626

27+
/** @var class-string|null */
2728
private ?string $queryBuilderClass = null;
2829

2930
private TypeSpecifier $typeSpecifier;
3031

32+
/**
33+
* @param class-string|null $queryBuilderClass
34+
*/
3135
public function __construct(?string $queryBuilderClass)
3236
{
3337
$this->queryBuilderClass = $queryBuilderClass;

stubs/MongoClassMetadataInfo.stub

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,6 @@ class ClassMetadata implements BaseClassMetadata
1515
/** @var string|null */
1616
public $customRepositoryClassName;
1717

18-
/**
19-
* @var class-string<T>
20-
*/
21-
public $name;
22-
2318
/**
2419
* @param class-string<T> $documentName
2520
*/

tests/DoctrineIntegration/ORM/EntityRepositoryDynamicReturnIntegrationTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ final class EntityRepositoryDynamicReturnIntegrationTest extends LevelsTestCase
1010
/**
1111
* @return string[][]
1212
*/
13-
public function dataTopics(): array
13+
public static function dataTopics(): array
1414
{
1515
return [
1616
['entityRepositoryDynamicReturn'],

tests/DoctrineIntegration/ORM/EntityRepositoryWithoutObjectManagerLoaderDynamicReturnIntegrationTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ final class EntityRepositoryWithoutObjectManagerLoaderDynamicReturnIntegrationTe
1010
/**
1111
* @return string[][]
1212
*/
13-
public function dataTopics(): array
13+
public static function dataTopics(): array
1414
{
1515
return [
1616
['entityRepositoryDynamicReturn'],

tests/Rules/Doctrine/ORM/FakeTestingUuidType.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ public function convertToDatabaseValue($value, AbstractPlatform $platform): ?str
4747
return null;
4848
}
4949

50+
/** @throws ConversionException */
5051
return (string) $value;
5152
}
5253

0 commit comments

Comments
 (0)