Translation\Extractor\Visitor\Php\Symfony\FlashMessage

Complexity

Total Complexity

14

Size/Duplication

Total Lines	40
Duplicated Lines	7.5 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

95.45%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	beforeTraverse()	0	3	1
A	leaveNode()	0	3	1
A	afterTraverse()	0	3	1
C	enterNode()	3	25	11

<?php

/*
 * This file is part of the PHP Translation package.
 *
 * (c) PHP Translation team <tobias.nyholm@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Translation\Extractor\Visitor\Php\Symfony;

use PhpParser\Node;
use PhpParser\NodeVisitor;
use Translation\Extractor\Model\SourceLocation;
use Translation\Extractor\Visitor\Php\BasePHPVisitor;

class FlashMessage extends BasePHPVisitor implements NodeVisitor
{
    public function beforeTraverse(array $nodes)
    {
    }

    public function enterNode(Node $node)
    {
        if ($node instanceof Node\Expr\MethodCall) {
            if (!is_string($node->name)) {
                return;
            }

            $name = $node->name;
            $caller = $node->var;
            // $caller might be "Node\Expr\New_"
            $callerName = isset($caller->name) ? $caller->name : '';

The property name does not seem to exist in PhpParser\Node\Expr.

            /*
             * Make sure the caller is from a variable named "this" or a function called "getFlashbag"
             */
            //If $this->addFlash()  or xxx->getFlashbag()->add()

64% of this comment could be valid code. Did you maybe forget this after debugging?

            if (('addFlash' === $name && $callerName === 'this' && $caller instanceof Node\Expr\Variable) ||
                ('add' === $name && $callerName === 'getFlashBag' && $caller instanceof Node\Expr\MethodCall)
            ) {
                if (null !== $label = $this->getStringArgument($node, 1)) {

This code seems to be duplicated across your project.

                    $this->collection->addLocation(new SourceLocation($label, $this->getAbsoluteFilePath(), $node->getAttribute('startLine')));
                }
            }
        }
    }

    public function leaveNode(Node $node)
    {
    }

    public function afterTraverse(array $nodes)
    {
    }
}