Start writing nested expression fixer
This commit is contained in:
parent
e33da7edc8
commit
940c2bbab4
|
@ -76,7 +76,7 @@ class IssetExpressionFixer extends Plugin
|
||||||
if (!$needsFixing) {
|
if (!$needsFixing) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
switch (\get_class($workVar)) {
|
switch ($class = \get_class($workVar)) {
|
||||||
case ArrayDimFetch::class:
|
case ArrayDimFetch::class:
|
||||||
case PropertyFetch::class:
|
case PropertyFetch::class:
|
||||||
$workVar->var = self::callPoly('returnMe', $workVar->var);
|
$workVar->var = self::callPoly('returnMe', $workVar->var);
|
||||||
|
@ -97,6 +97,8 @@ class IssetExpressionFixer extends Plugin
|
||||||
new LNumber(0)
|
new LNumber(0)
|
||||||
));
|
));
|
||||||
break;
|
break;
|
||||||
|
default:
|
||||||
|
throw new \RuntimeException("Trying to fix unknown isset expression $class");
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -0,0 +1,61 @@
|
||||||
|
<?php
|
||||||
|
|
||||||
|
namespace Phabel\Plugin;
|
||||||
|
|
||||||
|
use Phabel\Plugin;
|
||||||
|
use PhpParser\Node\Expr;
|
||||||
|
use PhpParser\Node\Expr\ArrayDimFetch;
|
||||||
|
use PhpParser\Node\Expr\FuncCall;
|
||||||
|
use PhpParser\Node\Expr\Instanceof_;
|
||||||
|
use PhpParser\Node\Expr\MethodCall;
|
||||||
|
use PhpParser\Node\Expr\New_;
|
||||||
|
use PhpParser\Node\Expr\PropertyFetch;
|
||||||
|
|
||||||
|
class NestedExpressionFixer extends Plugin
|
||||||
|
{
|
||||||
|
public function enter(Expr $expr): void
|
||||||
|
{
|
||||||
|
/** @var array<string, array<class-string<Expr>, true>> */
|
||||||
|
$subNodes = $this->getConfig($class = \get_class($expr), false);
|
||||||
|
if (!$subNodes) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
foreach ($subNodes as $key => $types) {
|
||||||
|
/** @var Expr $value */
|
||||||
|
$value = &$expr->{$key};
|
||||||
|
if (!isset($types[\get_class($value)])) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
switch ($class) {
|
||||||
|
case ArrayDimFetch::class:
|
||||||
|
case PropertyFetch::class:
|
||||||
|
case MethodCall::class:
|
||||||
|
case New_::class:
|
||||||
|
case Instanceof_::class:
|
||||||
|
$value = self::callPoly('returnMe', $value);
|
||||||
|
break;
|
||||||
|
case FuncCall::class:
|
||||||
|
$expr->var = self::callPoly('returnMe', $expr->var);
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Returns the data provided.
|
||||||
|
*
|
||||||
|
* @param mixed $data Data
|
||||||
|
*
|
||||||
|
* @return mixed
|
||||||
|
*
|
||||||
|
* @template T
|
||||||
|
*
|
||||||
|
* @psalm-param T $data data
|
||||||
|
*
|
||||||
|
* @psalm-return T
|
||||||
|
*/
|
||||||
|
public static function returnMe($data)
|
||||||
|
{
|
||||||
|
return $data;
|
||||||
|
}
|
||||||
|
}
|
|
@ -264,7 +264,7 @@ foreach ($instanceArgTypes as $class => $argTypes) {
|
||||||
}
|
}
|
||||||
|
|
||||||
$keys = [];
|
$keys = [];
|
||||||
foreach ($result['isset'] as $version) {
|
foreach ($result['main'] as $version) {
|
||||||
$keys = array_merge_recursive($keys, $version);
|
$keys = array_merge_recursive($keys, $version);
|
||||||
}
|
}
|
||||||
foreach ($keys as &$values) {
|
foreach ($keys as &$values) {
|
||||||
|
|
Loading…
Reference in New Issue
Block a user