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 #1574 - detect docblock additions to the first statement of a method when diffing

Browse Source
This commit is contained in:
Matthew Brown 2019-04-22 21:50:58 -04:00
parent 473c8cb832
commit c657a45132
3 changed files with 101 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
src/Psalm/Internal/Diff/ClassStatementsDiffer.php
View File

@ -82,7 +82,9 @@ class ClassStatementsDiffer extends AstDiffer
$a_size = $a_end - $a_start;
$b_size = $b_end - $b_start;
if (substr($a_code, $a_start, $a_size) === substr($b_code, $b_start, $b_size)) {
if ($a_size === $b_size
&& substr($a_code, $a_start, $a_size) === substr($b_code, $b_start, $b_size)
) {
$start_diff = $b_start - $a_start;
$line_diff = $b->getLine() - $a->getLine();
@ -103,11 +105,34 @@ class ClassStatementsDiffer extends AstDiffer
return false;
}
$a_stmts_start = $a->stmts ? (int) $a->stmts[0]->getAttribute('startFilePos') : $a_end;
$b_stmts_start = $b->stmts ? (int) $b->stmts[0]->getAttribute('startFilePos') : $b_end;
if ($a->stmts) {
$first_stmt = $a->stmts[0];
$a_stmts_start = (int) $first_stmt->getAttribute('startFilePos');
$body_change = substr($a_code, $a_stmts_start, $a_end - $a_stmts_start)
!== substr($b_code, $b_stmts_start, $b_end - $b_stmts_start);
if ($a_stmt_comments = $first_stmt->getComments()) {
$a_stmts_start = $a_stmt_comments[0]->getFilePos();
}
} else {
$a_stmts_start = $a_end;
}
if ($b->stmts) {
$first_stmt = $b->stmts[0];
$b_stmts_start = (int) $first_stmt->getAttribute('startFilePos');
if ($b_stmt_comments = $first_stmt->getComments()) {
$b_stmts_start = $b_stmt_comments[0]->getFilePos();
}
} else {
$b_stmts_start = $b_end;
}
$a_body_size = $a_end - $a_stmts_start;
$b_body_size = $b_end - $b_stmts_start;
$body_change = $a_body_size !== $b_body_size
|| substr($a_code, $a_stmts_start, $a_end - $a_stmts_start)
!== substr($b_code, $b_stmts_start, $b_end - $b_stmts_start);
if (!$signature_change) {
$a_signature = substr($a_code, $a_start, $a_stmts_start - $a_start);

29
tests/FileDiffTest.php
View File

@ -1497,6 +1497,35 @@ class FileDiffTest extends TestCase
['use:Exception'],
[[-36, -2]],
],
'addDocblockToFirstFunctionStatement' => [
'<?php
namespace Foo;
class C {
public function foo(array $a) : void {
foreach ($a as $b) {
$b->bar();
}
}
}',
'<?php
namespace Foo;
class C {
public function foo(array $a) : void {
/**
* @psalm-suppress MixedAssignment
*/
foreach ($a as $b) {
$b->bar();
}
}
}',
[],
['foo\c::foo'],
[],
[],
],
'vimeoDiff' => [
'<?php
namespace C;

42
tests/FileUpdates/ErrorFixTest.php
View File

@ -326,6 +326,48 @@ class ErrorFixTest extends \Psalm\Tests\TestCase
],
'error_counts' => [1, 0, 0],
],
'addSuppressions' => [
'start_files' => [
getcwd() . DIRECTORY_SEPARATOR . 'A.php' => '<?php
class C {
public function foo(array $a) : void {
foreach ($a as $b) {
$b->bar();
}
}
}',
],
'middle_files' => [
getcwd() . DIRECTORY_SEPARATOR . 'A.php' => '<?php
class C {
public function foo(array $a) : void {
/**
* @psalm-suppress MixedAssignment
*/
foreach ($a as $b) {
$b->bar();
}
}
}',
],
'end_files' => [
getcwd() . DIRECTORY_SEPARATOR . 'A.php' => '<?php
class C {
public function foo(array $a) : void {
/**
* @psalm-suppress MixedAssignment
*/
foreach ($a as $b) {
/**
* @psalm-suppress MixedMethodCall
*/
$b->bar();
}
}
}',
],
'error_counts' => [2, 1, 0],
],
];
}
}