danog/psalm
1
0
Fork 0
mirror of https://github.com/danog/psalm.git synced 2025-01-22 05:41:20 +01:00
Code Issues Projects Releases Wiki Activity

Don't throw UnnecesseryVarAnnotation when hinting a mixed template var

Browse Source
This commit is contained in:
RobChett 2023-04-19 21:11:38 +01:00
parent 5efddb4201
commit 0a40286830
4 changed files with 39 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Psalm/Internal/Analyzer/Statements/Block/ForeachAnalyzer.php
View File

@ -191,7 +191,7 @@ class ForeachAnalyzer
&& $type_location
&& isset($context->vars_in_scope[$var_comment->var_id])
&& $context->vars_in_scope[$var_comment->var_id]->getId() === $comment_type->getId()
&& !$comment_type->isMixed()
&& !$comment_type->isMixed(true)
) {
$project_analyzer = $statements_analyzer->getProjectAnalyzer();

2
src/Psalm/Internal/Analyzer/Statements/Expression/AssignmentAnalyzer.php
View File

@ -270,7 +270,7 @@ class AssignmentAnalyzer
&& $extended_var_id
&& (!$not_ignored_docblock_var_ids || isset($not_ignored_docblock_var_ids[$extended_var_id]))
&& $temp_assign_value_type->getId() === $comment_type->getId()
&& !$comment_type->isMixed()
&& !$comment_type->isMixed(true)
) {
if ($codebase->alter_code
&& isset($statements_analyzer->getProjectAnalyzer()->getIssuesToFix()['UnnecessaryVarAnnotation'])

13
src/Psalm/Type/UnionTrait.php
View File

@ -795,9 +795,18 @@ trait UnionTrait
/**
* @psalm-mutation-free
*/
public function isMixed(): bool
public function isMixed(bool $check_templates = false): bool
{
return isset($this->types['mixed']) && count($this->types) === 1;
return count(
array_filter(
$this->types,
static fn($type): bool => $type instanceof TMixed
|| ($check_templates
&& $type instanceof TTemplateParam
&& $type->as->isMixed()
)
),
) === count($this->types);
}
/**

26
tests/Template/ClassTemplateTest.php
View File

@ -4112,6 +4112,32 @@ class ClassTemplateTest extends TestCase
'$t===' => '\'\'',
],
],
'mixedAssignment' => [
'code' => '<?php
/** @template T */
abstract class Foo {
/** @psalm-var T */
protected $value;
/** @psalm-param T $value */
public function __construct($value)
{
/** @var T */
$value = $this->normalize($value);
$this->value = $value;
}
/**
* @psalm-param T $value
* @psalm-return T
*/
protected function normalize($value)
{
return $value;
}
}
',
],
];
}