mirror of https://github.com/rectorphp/rector.git
167 lines
5.1 KiB
PHP
167 lines
5.1 KiB
PHP
<?php
|
|
|
|
declare (strict_types=1);
|
|
namespace Rector\CodingStyle\Rector\ClassMethod;
|
|
|
|
use PhpParser\Node;
|
|
use PhpParser\Node\Expr\Assign;
|
|
use PhpParser\Node\Expr\Closure;
|
|
use PhpParser\Node\Expr\MethodCall;
|
|
use PhpParser\Node\Expr\StaticCall;
|
|
use PhpParser\Node\Expr\Variable;
|
|
use PhpParser\Node\Stmt;
|
|
use PhpParser\Node\Stmt\ClassMethod;
|
|
use PhpParser\Node\Stmt\Expression;
|
|
use PhpParser\Node\Stmt\Function_;
|
|
use PhpParser\Node\Stmt\Nop;
|
|
use Rector\Rector\AbstractRector;
|
|
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
|
|
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
|
|
/**
|
|
* @see \Rector\Tests\CodingStyle\Rector\ClassMethod\NewlineBeforeNewAssignSetRector\NewlineBeforeNewAssignSetRectorTest
|
|
*/
|
|
final class NewlineBeforeNewAssignSetRector extends AbstractRector
|
|
{
|
|
/**
|
|
* @var string|null
|
|
*/
|
|
private $previousStmtVariableName;
|
|
/**
|
|
* @var string|null
|
|
*/
|
|
private $previousPreviousStmtVariableName;
|
|
public function getRuleDefinition() : RuleDefinition
|
|
{
|
|
return new RuleDefinition('Add extra space before new assign set', [new CodeSample(<<<'CODE_SAMPLE'
|
|
final class SomeClass
|
|
{
|
|
public function run()
|
|
{
|
|
$value = new Value;
|
|
$value->setValue(5);
|
|
$value2 = new Value;
|
|
$value2->setValue(1);
|
|
}
|
|
}
|
|
CODE_SAMPLE
|
|
, <<<'CODE_SAMPLE'
|
|
final class SomeClass
|
|
{
|
|
public function run()
|
|
{
|
|
$value = new Value;
|
|
$value->setValue(5);
|
|
|
|
$value2 = new Value;
|
|
$value2->setValue(1);
|
|
}
|
|
}
|
|
CODE_SAMPLE
|
|
)]);
|
|
}
|
|
/**
|
|
* @return array<class-string<Node>>
|
|
*/
|
|
public function getNodeTypes() : array
|
|
{
|
|
return [ClassMethod::class, Function_::class, Closure::class];
|
|
}
|
|
/**
|
|
* @param ClassMethod|Function_|Closure $node
|
|
*/
|
|
public function refactor(Node $node) : ?Node
|
|
{
|
|
// skip methods with no bodies (e.g interface methods)
|
|
if ($node->stmts === null) {
|
|
return null;
|
|
}
|
|
$this->reset();
|
|
$hasChanged = \false;
|
|
$newStmts = [];
|
|
foreach ($node->stmts as $key => $stmt) {
|
|
$currentStmtVariableName = $this->resolveCurrentStmtVariableName($stmt);
|
|
if ($this->shouldAddEmptyLine($currentStmtVariableName, $node, $key)) {
|
|
$hasChanged = \true;
|
|
// insert newline before stmt
|
|
$newStmts[] = new Nop();
|
|
}
|
|
$newStmts[] = $stmt;
|
|
$this->previousPreviousStmtVariableName = $this->previousStmtVariableName;
|
|
$this->previousStmtVariableName = $currentStmtVariableName;
|
|
}
|
|
$node->stmts = $newStmts;
|
|
return $hasChanged ? $node : null;
|
|
}
|
|
private function reset() : void
|
|
{
|
|
$this->previousStmtVariableName = null;
|
|
$this->previousPreviousStmtVariableName = null;
|
|
}
|
|
private function resolveCurrentStmtVariableName(Stmt $stmt) : ?string
|
|
{
|
|
if (!$stmt instanceof Expression) {
|
|
return null;
|
|
}
|
|
$stmtExpr = $stmt->expr;
|
|
if ($stmtExpr instanceof Assign || $stmtExpr instanceof MethodCall) {
|
|
if ($this->shouldSkipLeftVariable($stmtExpr)) {
|
|
return null;
|
|
}
|
|
if (!$stmtExpr->var instanceof MethodCall && !$stmtExpr->var instanceof StaticCall) {
|
|
return $this->getName($stmtExpr->var);
|
|
}
|
|
}
|
|
return null;
|
|
}
|
|
/**
|
|
* @param \PhpParser\Node\Stmt\ClassMethod|\PhpParser\Node\Stmt\Function_|\PhpParser\Node\Expr\Closure $node
|
|
*/
|
|
private function shouldAddEmptyLine(?string $currentStmtVariableName, $node, int $key) : bool
|
|
{
|
|
if (!$this->isNewVariableThanBefore($currentStmtVariableName)) {
|
|
return \false;
|
|
}
|
|
// this is already empty line before
|
|
return !$this->isPrecededByEmptyLine($node, $key);
|
|
}
|
|
/**
|
|
* @param \PhpParser\Node\Expr\Assign|\PhpParser\Node\Expr\MethodCall $node
|
|
*/
|
|
private function shouldSkipLeftVariable($node) : bool
|
|
{
|
|
if (!$node->var instanceof Variable) {
|
|
return \false;
|
|
}
|
|
// local method call
|
|
return $this->nodeNameResolver->isName($node->var, 'this');
|
|
}
|
|
private function isNewVariableThanBefore(?string $currentStmtVariableName) : bool
|
|
{
|
|
if ($this->previousPreviousStmtVariableName === null) {
|
|
return \false;
|
|
}
|
|
if ($this->previousStmtVariableName === null) {
|
|
return \false;
|
|
}
|
|
if ($currentStmtVariableName === null) {
|
|
return \false;
|
|
}
|
|
if ($this->previousStmtVariableName !== $this->previousPreviousStmtVariableName) {
|
|
return \false;
|
|
}
|
|
return $this->previousStmtVariableName !== $currentStmtVariableName;
|
|
}
|
|
/**
|
|
* @param \PhpParser\Node\Stmt\ClassMethod|\PhpParser\Node\Stmt\Function_|\PhpParser\Node\Expr\Closure $node
|
|
*/
|
|
private function isPrecededByEmptyLine($node, int $key) : bool
|
|
{
|
|
if ($node->stmts === null) {
|
|
return \false;
|
|
}
|
|
$previousNode = $node->stmts[$key - 1];
|
|
$currentNode = $node->stmts[$key];
|
|
return \abs($currentNode->getLine() - $previousNode->getLine()) >= 2;
|
|
}
|
|
}
|