octoleo/rector
1
1
Fork 0
mirror of https://github.com/rectorphp/rector.git synced 2024-06-28 21:53:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

remove comments

Browse Source
This commit is contained in:
TomasVotruba 2017-11-01 22:37:57 +01:00
parent d2ae52e417
commit 452e6748fa
5 changed files with 50 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
packages/NodeTraverserQueue/src/BetterNodeFinder.php
View File

@ -76,4 +76,12 @@ final class BetterNodeFinder
{ {
return $this->nodeFinder->find($nodes, $filter); return $this->nodeFinder->find($nodes, $filter);
} }
/**
* @param Node|Node[] $nodes
*/
public function findFirst($nodes, callable $filter): ?Node
{
return $this->nodeFinder->findFirst($nodes, $filter);
}
} }

18
packages/NodeTypeResolver/src/NodeVisitor/CallerTypeResolver.php
View File

@ -70,12 +70,24 @@ final class CallerTypeResolver extends NodeVisitorAbstract
private function processMethodCallNode(MethodCall $methodCallNode): void private function processMethodCallNode(MethodCall $methodCallNode): void
{ {
$node = $this->betterNodeFinder->findFirstInstanceOfAny( $callerNode = $this->betterNodeFinder->findFirst(
$methodCallNode, $methodCallNode,
[Variable::class, PropertyFetch::class] function (Node $node) use ($methodCallNode) {
if ($node instanceof Variable || $node instanceof PropertyFetch /* || $node instanceof MethodCall*/) {
return $node;
}
if ($node === $methodCallNode) {
return null;
}
}
); );
$nodeTypes = (array) $node->getAttribute(Attribute::TYPES); if ($callerNode === null) {
return;
}
$nodeTypes = (array) $callerNode->getAttribute(Attribute::TYPES);
if ($nodeTypes) { if ($nodeTypes) {
$methodCallNode->setAttribute(Attribute::CALLER_TYPES, $nodeTypes); $methodCallNode->setAttribute(Attribute::CALLER_TYPES, $nodeTypes);
} }

6
tests/Rector/Contrib/Nette/DI/ExpandFunctionToParametersArrayRector/Correct/correct.php.inc
View File

@ -2,8 +2,14 @@
class SomeNetteExtension extends Nette\DI\CompilerExtension class SomeNetteExtension extends Nette\DI\CompilerExtension
{ {
/**
* @var SomeBuilder
*/
private $builder;
public function loadConfiguration() public function loadConfiguration()
{ {
$builder = new SomeBuilder;
$builder->parameters['appDir']; $builder->parameters['appDir'];
$this->builder->parameters['appDir']; $this->builder->parameters['appDir'];
$this->getContainerBuilder()->parameters['appDir']; $this->getContainerBuilder()->parameters['appDir'];

6
tests/Rector/Contrib/Nette/DI/ExpandFunctionToParametersArrayRector/Wrong/wrong.php.inc
View File

@ -2,8 +2,14 @@
class SomeNetteExtension extends Nette\DI\CompilerExtension class SomeNetteExtension extends Nette\DI\CompilerExtension
{ {
/**
* @var SomeBuilder
*/
private $builder;
public function loadConfiguration() public function loadConfiguration()
{ {
$builder = new SomeBuilder;
$builder->expand('%appDir%'); $builder->expand('%appDir%');
$this->builder->expand('%appDir%'); $this->builder->expand('%appDir%');
$this->getContainerBuilder()->expand('%appDir%'); $this->getContainerBuilder()->expand('%appDir%');

30
tests/Rector/Dynamic/ArgumentReplacerRector/Test.php
View File

@ -9,25 +9,25 @@ final class Test extends AbstractConfigurableRectorTestCase
{ {
public function test(): void public function test(): void
{ {
// $this->doTestFileMatchesExpectedContent( $this->doTestFileMatchesExpectedContent(
// __DIR__ . '/wrong/wrong.php.inc', __DIR__ . '/wrong/wrong.php.inc',
// __DIR__ . '/correct/correct.php.inc' __DIR__ . '/correct/correct.php.inc'
// ); );
//
// $this->doTestFileMatchesExpectedContent( $this->doTestFileMatchesExpectedContent(
// __DIR__ . '/wrong/wrong2.php.inc', __DIR__ . '/wrong/wrong2.php.inc',
// __DIR__ . '/correct/correct2.php.inc' __DIR__ . '/correct/correct2.php.inc'
// ); );
//
$this->doTestFileMatchesExpectedContent( $this->doTestFileMatchesExpectedContent(
__DIR__ . '/wrong/wrong3.php.inc', __DIR__ . '/wrong/wrong3.php.inc',
__DIR__ . '/correct/correct3.php.inc' __DIR__ . '/correct/correct3.php.inc'
); );
//
// $this->doTestFileMatchesExpectedContent( $this->doTestFileMatchesExpectedContent(
// __DIR__ . '/wrong/wrong4.php.inc', __DIR__ . '/wrong/wrong4.php.inc',
// __DIR__ . '/correct/correct4.php.inc' __DIR__ . '/correct/correct4.php.inc'
// ); );
} }
protected function provideConfig(): string protected function provideConfig(): string