Skip to content

Commit 877d900

Browse files
committed
Less aggressive dumbing down arrays
1 parent 630430f commit 877d900

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

src/Type/TypeCombinator.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -708,10 +708,6 @@ private static function countConstantArrayValueTypes(array $types): int
708708
{
709709
$constantArrayValuesCount = 0;
710710
foreach ($types as $type) {
711-
if ($type instanceof ConstantArrayType) {
712-
$constantArrayValuesCount += count($type->getValueTypes());
713-
}
714-
715711
TypeTraverser::map($type, static function (Type $type, callable $traverse) use (&$constantArrayValuesCount): Type {
716712
if ($type instanceof ConstantArrayType) {
717713
$constantArrayValuesCount += count($type->getValueTypes());

tests/PHPStan/Analyser/data/bug-8004.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public function getErrorsOnInvalidQuestions(array $importQuiz, int $key): array
7373
}
7474
}
7575

76-
assertType("list<non-empty-array<literal-string&non-falsy-string, 'empty_answer'|'empty_question'|'invalid_answer_1_too_long'|'invalid_answer_1_type'|'invalid_answer_2_too_long'|'invalid_answer_2_type'|'invalid_answer_3_too_long'|'invalid_answer_3_type'|'invalid_answer_4_too_long'|'invalid_answer_4_type'|'invalid_question_too_long'|'invalid_right_answer'|int>&oversized-array>&oversized-array", $errors);
76+
assertType("list<array{line: int, type: 'empty_answer'|'empty_question'|'invalid_answer_1_too_long'|'invalid_answer_1_type'|'invalid_answer_2_too_long'|'invalid_answer_2_type'|'invalid_answer_3_too_long'|'invalid_answer_3_type'|'invalid_answer_4_too_long'|'invalid_answer_4_type'|'invalid_question_too_long'|'invalid_right_answer', value: int}>&oversized-array", $errors);
7777

7878
return $errors;
7979
}

0 commit comments

Comments
 (0)