danog/psalm
1
0
Fork 0
mirror of https://github.com/danog/psalm.git synced 2025-01-21 21:31:13 +01:00
Code Issues Projects Releases Wiki Activity

Fix addition of int and string type

Browse Source
This commit is contained in:
Matt Brown 2020-10-07 18:01:41 -04:00
parent 8b7657f379
commit 443025eab8
2 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Psalm/Internal/Analyzer/Statements/Expression/BinaryOp/NonDivArithmeticOpAnalyzer.php
View File

@ -668,10 +668,8 @@ class NonDivArithmeticOpAnalyzer
if ($parent instanceof PhpParser\Node\Expr\BinaryOp\Mod) {
$result_type = Type::getInt();
} elseif (!$result_type) {
$result_type = Type::getFloat();
} else {
$result_type = Type::combineUnionTypes(Type::getFloat(), $result_type);
$result_type = new Type\Union([new Type\Atomic\TInt, new Type\Atomic\TFloat]);
}
$has_valid_right_operand = true;

8
tests/BinaryOperationTest.php
View File

@ -296,6 +296,14 @@ class BinaryOperationTest extends TestCase
*/
function test(int $tickedTimes): void {}'
],
'numericPlusIntegerIsIntOrFloat' => [
'<?php
/** @param numeric-string $s */
function foo(string $s) : void {
$s = $s + 1;
if (is_int($s)) {}
}'
],
];
}