pug-php/js-phpize-phug

Method __construct has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function __construct(ModuleContainerInterface $container)
    {
        parent::__construct($container);

        if ($container instanceof Renderer) {

Missing class import via use statement (line '114', column '23').

Open

            throw new \InvalidArgumentException(sprintf(

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Avoid assigning values to variables in if clauses and the like (line '108', column '27').

Open

    protected function getLanguageKeywordValue($value, KeywordElement $keyword, $name)
    {
        $value = trim($value, "()\"' \t\n\r\0\x0B");

        if (!in_array($value, $this->languages, true)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid assigning values to variables in if clauses and the like (line '77', column '14').

Open

    public function handleNodeEvent(NodeEvent $event)
    {
        /* @var CommentNode $node */
        if (($node = $event->getNode()) instanceof CommentNode &&
            !$node->isVisible() &&

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid assigning values to variables in if clauses and the like (line '80', column '14').

Open

    public function handleNodeEvent(NodeEvent $event)
    {
        /* @var CommentNode $node */
        if (($node = $event->getNode()) instanceof CommentNode &&
            !$node->isVisible() &&

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid assigning values to variables in if clauses and the like (line '132', column '13').

Open

    public function handleNodeLanguageKeyword($value, KeywordElement $keyword, $name)
    {
        $value = $this->getLanguageKeywordValue($value, $keyword, $name);

        if ($next = $keyword->getNextSibling()) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Call to undeclared method \JsPhpize\JsPhpizePhug::getOption

Open

        if ($this->getOption('language') === 'php') {

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \JsPhpize\JsPhpizePhug::setOptionsRecursive

Open

        $this->setOptionsRecursive($compiler->getOption(['module_options', 'jsphpize']));

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \JsPhpize\JsPhpizePhug::getOption

Open

            $next->appendChild(new KeywordElement('language', $this->getOption('language'), $keyword->getOriginNode()));

• Exclude checks
  • Disable engine
  • Disable check

Possibly zero references to use statement for classlike/namespace Compiler (\Phug\Compiler)

Open

use Phug\Compiler;

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (compiler) is \Phug\Util\ModuleContainerInterface but \JsPhpize\JsPhpizePhug::getJsPhpizeEngine() takes \Phug\CompilerInterface defined at /code/src/JsPhpize/Traits/Compilation.php:22

Open

                    return $this->transformExpression($this->getJsPhpizeEngine($compiler), $code, $compiler->getPath());

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \JsPhpize\JsPhpizePhug::setOptionsRecursive

Open

        $this->setOptionsRecursive([

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \JsPhpize\JsPhpizePhug::setOption

Open

        $this->setOption('language', $value);

• Exclude checks
  • Disable engine
  • Disable check