The method getTemplateFile() contains an eval expression.

Open

        eval("?>$__php");

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

class Foo {
    public function bar($param)  {
        if ($param === 42) {
            eval('$param = 23;');
        }
    }
}

Function dumpCoverage has a Cognitive Complexity of 94 (exceeds 5 allowed). Consider refactoring.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

• Read up
  Read up

File Coverage.php has 412 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

namespace Phug\Tester;

use Phug\Ast\NodeInterface as AstNodeInterface;

Method dumpCoverage has 146 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

Coverage has 29 functions (exceeds 20 allowed). Consider refactoring.

Open

class Coverage
{
    const VERSION = '0.1.0';

    /**

The class Coverage has an overall complexity of 101 which is very high. The configured complexity threshold is 50.

Open

class Coverage
{
    const VERSION = '0.1.0';

    /**

• Exclude checks
  • Disable engine
  • Disable check

Function getLocationPath has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    private function getLocationPath(string $path): string
    {
        foreach ($this->getPaths() as $base) {
            $realBase = realpath($base);
            if ($realBase) {

• Read up
  Read up

Function emptyDirectory has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    protected static function emptyDirectory(string $dir)
    {
        if (!is_dir($dir)) {
            return;
        }

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                                if (!($node instanceof DocumentNode) && ($location = $node->getSourceLocation())) {
                                    $locationPath = $location->getPath();
                                    if (!isset($counts[$locationPath])) {
                                        $counts[$locationPath] = $this->countFileNodes($locationPath);
                                    }

The method dumpCoverage() has an NPath complexity of 3646296. The configured NPath complexity threshold is 200.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method dumpCoverage() has 153 lines of code. Current threshold is set to 100. Avoid really long methods.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

• Exclude checks
  • Disable engine
  • Disable check

The method dumpCoverage() has a Cyclomatic Complexity of 38. The configured cyclomatic complexity threshold is 10.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

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

Open

            throw new \BadFunctionCallException('You need to install XDebug to use coverage feature.');

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

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

The method dumpCoverage has a boolean flag argument $output, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)

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

class Foo {
    public function bar($flag = true) {
    }
}

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

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

• 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
            // ...
        }
    }
}

Assigning null to property but \Phug\Tester\Coverage::$singleton is \Phug\Tester\Coverage

Open

        static::$singleton = null;

• Exclude checks
  • Disable engine
  • Disable check

Call to method getSourceLocation from undeclared class \Phug\Parser\NodeInterface

Open

                                if (!($node instanceof DocumentNode) && ($location = $node->getSourceLocation())) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (input) is int but \str_pad() takes string

Open

                str_pad($coveredNodes, 3, ' ', STR_PAD_LEFT).' / '.

• Exclude checks
  • Disable engine
  • Disable check

Call to method getOption from undeclared class \Phug\Renderer

Open

        static::removeDirectory($this->renderer->getOption('cache_dir'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method getOption from undeclared class \Phug\Renderer

Open

        $cache = $this->renderer->getOption('cache_dir');

• Exclude checks
  • Disable engine
  • Disable check

Call to method getCompiler from undeclared class \Phug\Renderer

Open

        $document = $this->renderer->getCompiler()->getParser()->parse($contents, $file);

• Exclude checks
  • Disable engine
  • Disable check

Checking instanceof against undeclared class \Phug\Parser\NodeInterface

Open

        if ($node instanceof NodeInterface && !($node instanceof DocumentNode) && !$list->offsetExists($node)) {

• Exclude checks
  • Disable engine
  • Disable check

Checking instanceof against undeclared class \Phug\Ast\NodeInterface

Open

        if ($node instanceof AstNodeInterface) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (input) is int but \str_pad() takes string

Open

                    str_pad($_nodes, 3, ' ', STR_PAD_LEFT).'   '.

• Exclude checks
  • Disable engine
  • Disable check

Call to method getOuterNode from undeclared class \Phug\Parser\NodeInterface

Open

                                $node = $node->getOuterNode() ?: $node->getParent()) {

• Exclude checks
  • Disable engine
  • Disable check

Checking instanceof against undeclared class \Phug\Parser\Node\DocumentNode

Open

                                if (!($node instanceof DocumentNode) && ($location = $node->getSourceLocation())) {

• Exclude checks
  • Disable engine
  • Disable check

Parameter $location has undeclared type ?\Phug\Util\SourceLocationInterface

Open

    protected function recordLocation(SourceLocationInterface $location = null, int $covered = 0)

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (input) is 0|int but \str_pad() takes string

Open

                    str_pad($_coveredNodes, 3, ' ', STR_PAD_LEFT).' / '.

• Exclude checks
  • Disable engine
  • Disable check

Call to method getPath from undeclared class \Phug\Util\SourceLocationInterface

Open

            $locationPath = realpath($location->getPath());

• Exclude checks
  • Disable engine
  • Disable check

Call to method getOption from undeclared class \Phug\Renderer

Open

        return $this->renderer->getOption('paths');

• Exclude checks
  • Disable engine
  • Disable check

Call to method scanDirectory from undeclared class \Phug\Renderer

Open

            foreach ($this->renderer->scanDirectory($path) as $file) {

• Exclude checks
  • Disable engine
  • Disable check

Checking instanceof against undeclared class \Phug\Parser\Node\DocumentNode

Open

        if ($node instanceof NodeInterface && !($node instanceof DocumentNode) && !$list->offsetExists($node)) {

• Exclude checks
  • Disable engine
  • Disable check

Return type of createRenderer() is undeclared type \Phug\Renderer

Open

    public function createRenderer($renderer, array $options = []): Renderer

• Exclude checks
  • Disable engine
  • Disable check

Call to method getCompiler from undeclared class \Phug\Renderer

Open

        $formatter = $this->renderer->getCompiler()->getFormatter();

• Exclude checks
  • Disable engine
  • Disable check

Call to method getParent from undeclared class \Phug\Parser\NodeInterface

Open

                                $node = $node->getOuterNode() ?: $node->getParent()) {

• Exclude checks
  • Disable engine
  • Disable check

Return type of getRenderer() is undeclared type \Phug\Renderer

Open

    public function getRenderer(): Renderer

• Exclude checks
  • Disable engine
  • Disable check

Static property \Phug\Tester\Coverage::$singleton is declared to have type static, but the only instance is shared among all subclasses (Did you mean \Phug\Tester\Coverage)

Open

     * @var static

• Exclude checks
  • Disable engine
  • Disable check

Call to method getChildren from undeclared class \Phug\Ast\NodeInterface

Open

            foreach ($node->getChildren() as $child) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method getCompiler from undeclared class \Phug\Renderer

Open

        $compiler = $this->renderer->getCompiler();

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (input) is int but \str_pad() takes string

Open

                        str_pad($id, $pad, ' ', STR_PAD_LEFT).

• Exclude checks
  • Disable engine
  • Disable check

Call to method getOption from undeclared class \Phug\Renderer

Open

        static::emptyDirectory($this->renderer->getOption('cache_dir'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method getLine from undeclared class \Phug\Util\SourceLocationInterface

Open

            $locationLine = $location->getLine() - 1;

• Exclude checks
  • Disable engine
  • Disable check

Call to method getSourceLocation from undeclared class \Phug\Parser\NodeInterface

Open

            $this->recordLocation($node->getSourceLocation());

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (input) is int but \str_pad() takes string

Open

                str_pad($nodes, 3, ' ', STR_PAD_LEFT).'   '.

• Exclude checks
  • Disable engine
  • Disable check

Property \Phug\Tester\Coverage->renderer has undeclared type \Phug\Renderer

Open

    protected $renderer;

• Exclude checks
  • Disable engine
  • Disable check

Call to method setOption from undeclared class \Phug\Renderer

Open

        $this->renderer->setOption('cache_dir', realpath($cache));

• Exclude checks
  • Disable engine
  • Disable check

Call to method getOffset from undeclared class \Phug\Util\SourceLocationInterface

Open

            $this->coverage[$locationPath][$locationLine][$location->getOffset() - 1] = $covered;

• Exclude checks
  • Disable engine
  • Disable check

Call to method getOption from undeclared class \Phug\Renderer

Open

        $cache = realpath($this->renderer->getOption('cache_dir')).DIRECTORY_SEPARATOR;

• Exclude checks
  • Disable engine
  • Disable check

Checking instanceof against undeclared class \Phug\Parser\NodeInterface

Open

                                $node instanceof NodeInterface;

• Exclude checks
  • Disable engine
  • Disable check

Argument #2 of this call to \str_pad is typically a literal or constant but isn't, but argument #1 (which is typically a variable) is a literal or constant. The arguments may be in the wrong order.

Open

                str_pad('Total', $pad, ' ', STR_PAD_RIGHT).

• Exclude checks
  • Disable engine
  • Disable check

Avoid excessively long variable names like $coverageStoppingAllowed. Keep variable name length under 20.

Open

    protected $coverageStoppingAllowed = true;

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

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid variables with short names like $id. Configured minimum length is 3.

Open

                    $id = $number + 1;

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

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

The variable $_nodes is not named in camelCase.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $_coveredNodes is not named in camelCase.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $__php is not named in camelCase.

Open

    protected function getTemplateFile(string $file, array $vars): string
    {
        $__php = file_get_contents(__DIR__."/../../template/$file");
        extract($vars);
        ob_start();

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $_coveredNodes is not named in camelCase.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $_coveredNodes is not named in camelCase.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $_nodes is not named in camelCase.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $_coveredNodes is not named in camelCase.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $_nodes is not named in camelCase.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $__php is not named in camelCase.

Open

    protected function getTemplateFile(string $file, array $vars): string
    {
        $__php = file_get_contents(__DIR__."/../../template/$file");
        extract($vars);
        ob_start();

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $_nodes is not named in camelCase.

Open

    public function dumpCoverage(bool $output = false, string $directory = null)
    {
        if ($directory) {
            static::addEmptyDirectory($directory);
            foreach (static::$assets as $asset) {

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}