PHPSA\Compiler\Expression\AssignRef::compile()

Complexity

Conditions	5
Paths	7

Size

Total Lines	40
Code Lines	27

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	24
CRAP Score	5.1281

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function compile($expr, Context $context)
    {
        $compiler = $context->getExpressionCompiler();
        if ($expr->var instanceof VariableNode) {
            $name = $expr->var->name;

            $compiledExpression = $compiler->compile($expr->expr);
            
            $symbol = $context->getSymbol($name);
            if ($symbol) {
                $symbol->modify($compiledExpression->getType(), $compiledExpression->getValue());
            } else {
                $symbol = new \PHPSA\Variable(
                    $name,

It seems like $name defined by $expr->var->name on line 26 can also be of type object<PhpParser\Node\Expr>; however, PHPSA\Variable::__construct() does only seem to accept string, maybe add an additional type check?

                    $compiledExpression->getValue(),
                    $compiledExpression->getType(),
                    $context->getCurrentBranch()
                );
                $context->addVariable($symbol);
            }

            if ($expr->expr instanceof VariableNode) {
                $rightVarName = $expr->expr->name;

                $rightSymbol = $context->getSymbol($rightVarName);
                if ($rightSymbol) {
                    $rightSymbol->incUse();
                    $symbol->setReferencedTo($rightSymbol);
                } else {
                    $context->debug('Cannot fetch variable by name: ' . $rightVarName);
                }
            }

            $symbol->incSets();
            return $compiledExpression;
        }

        $context->debug('Unknown how to pass symbol by ref');
        return new CompiledExpression();
    }
}