Skip to content

Commit f3fccb6

Browse files
committed
fixes after rebase
1 parent c87c88c commit f3fccb6

4 files changed

Lines changed: 8 additions & 15 deletions

File tree

phpstan-baseline.neon

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1473,12 +1473,6 @@ parameters:
14731473
count: 1
14741474
path: src/Type/ObjectType.php
14751475

1476-
-
1477-
rawMessage: 'Doing instanceof PHPStan\Type\ErrorType is error-prone and deprecated. Use Type::isError() instead.'
1478-
identifier: phpstanApi.instanceofType
1479-
count: 4
1480-
path: src/Type/ObjectType.php
1481-
14821476
-
14831477
rawMessage: Doing instanceof PHPStan\Type\Generic\GenericObjectType is error-prone and deprecated.
14841478
identifier: phpstanApi.instanceofType

src/Analyser/Traverser/GenericTypeTemplateTraverser.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
namespace PHPStan\Analyser\Traverser;
44

5-
use PHPStan\Type\ErrorType;
65
use PHPStan\Type\Generic\TemplateType;
76
use PHPStan\Type\Generic\TemplateTypeHelper;
87
use PHPStan\Type\Generic\TemplateTypeMap;
@@ -25,7 +24,7 @@ public function traverse(Type $type, callable $traverse): Type
2524
{
2625
if ($type instanceof TemplateType && !$type->isArgument()) {
2726
$newType = $this->resolvedTemplateTypeMap->getType($type->getName());
28-
if ($newType === null || $newType instanceof ErrorType) {
27+
if ($newType === null || $newType->isError()->yes()) {
2928
return $type->getDefault() ?? $type->getBound();
3029
}
3130

src/Type/ObjectType.php

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@
5757
use function count;
5858
use function implode;
5959
use function in_array;
60+
use function is_array;
6061
use function sprintf;
6162
use function strtolower;
6263

@@ -226,7 +227,7 @@ public function getUnresolvedPropertyPrototype(string $propertyName, ClassMember
226227
}
227228

228229
$property = RecursionGuard::run($this, static fn () => $nakedClassReflection->getProperty($propertyName, $scope));
229-
if ($property instanceof ErrorType) {
230+
if (!$property instanceof ExtendedPropertyReflection) {
230231
$property = new DummyPropertyReflection($propertyName);
231232

232233
return new CallbackUnresolvedPropertyPrototypeReflection(
@@ -332,7 +333,7 @@ public function getUnresolvedInstancePropertyPrototype(string $propertyName, Cla
332333
}
333334

334335
$property = RecursionGuard::run($this, static fn () => $nakedClassReflection->getInstanceProperty($propertyName, $scope));
335-
if ($property instanceof ErrorType) {
336+
if (! $property instanceof ExtendedPropertyReflection) {
336337
$property = new DummyPropertyReflection($propertyName);
337338

338339
return new CallbackUnresolvedPropertyPrototypeReflection(
@@ -414,7 +415,7 @@ public function getUnresolvedStaticPropertyPrototype(string $propertyName, Class
414415
}
415416

416417
$property = RecursionGuard::run($this, static fn () => $nakedClassReflection->getStaticProperty($propertyName));
417-
if ($property instanceof ErrorType) {
418+
if (!$property instanceof ExtendedPropertyReflection) {
418419
$property = new DummyPropertyReflection($propertyName);
419420

420421
return new CallbackUnresolvedPropertyPrototypeReflection(
@@ -1548,7 +1549,7 @@ public function isCallable(): TrinaryLogic
15481549
if ($parametersAcceptors === null) {
15491550
return TrinaryLogic::createNo();
15501551
}
1551-
if ($parametersAcceptors instanceof ErrorType) {
1552+
if (! is_array($parametersAcceptors)) {
15521553
return TrinaryLogic::createNo();
15531554
}
15541555

src/Type/Php/ArrayCombineHelper.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
use PHPStan\Type\ArrayType;
1212
use PHPStan\Type\Constant\ConstantArrayTypeBuilder;
1313
use PHPStan\Type\ConstantScalarType;
14-
use PHPStan\Type\ErrorType;
1514
use PHPStan\Type\MixedType;
1615
use PHPStan\Type\NeverType;
1716
use PHPStan\Type\Type;
@@ -78,7 +77,7 @@ public function getReturnAndThrowType(Expr $firstArg, Expr $secondArg, Scope $sc
7877
$itemType = $keysParamType->getIterableValueType();
7978

8079
if ($itemType->isInteger()->no()) {
81-
if ($itemType->toString() instanceof ErrorType) {
80+
if ($itemType->toString()->isError()->yes()) {
8281
return [new NeverType(), TrinaryLogic::createNo()];
8382
}
8483

@@ -116,7 +115,7 @@ private function sanitizeConstantArrayKeyTypes(array $types): ?array
116115
$sanitizedTypes = [];
117116

118117
foreach ($types as $type) {
119-
if (!$type->isInteger()->yes() && ! $type->toString() instanceof ErrorType) {
118+
if (!$type->isInteger()->yes() && ! $type->toString()->isError()->yes()) {
120119
$type = $type->toString();
121120
}
122121

0 commit comments

Comments
 (0)