Method register has 192 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function register(Container $app)
    {
        $app['profiler.mount_prefix'] = '/_profiler';
        $app->extend('dispatcher', function ($dispatcher, $app) {
            return new TraceableEventDispatcher($dispatcher, $app['stopwatch'], $app['logger']);

Function register has a Cognitive Complexity of 20 (exceeds 5 allowed). Consider refactoring.

Open

    public function register(Container $app)
    {
        $app['profiler.mount_prefix'] = '/_profiler';
        $app->extend('dispatcher', function ($dispatcher, $app) {
            return new TraceableEventDispatcher($dispatcher, $app['stopwatch'], $app['logger']);

• Read up
  Read up

File F9WebProfilerServiceProvider.php has 276 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

/*
 * This file is part of the Silex framework.
 *

Avoid too many return statements within this method.

Open

                    return new TwigDataCollector($app['twig.profiler.profile']);

Avoid too many return statements within this method.

Open

                return $twig;

Avoid too many return statements within this method.

Open

            return $profiler;

Avoid too many return statements within this method.

Open

            return new WebDebugToolbarListener($app['twig'], $app['web_profiler.debug_toolbar.intercept_redirects'], $mode, $app['web_profiler.debug_toolbar.position'], $app['url_generator']);

Avoid too many return statements within this method.

Open

            return $twig;

Avoid too many return statements within this method.

Open

                return new ResolvedTypeFactoryDataCollectorProxy($factory, $app['data_collectors']['form']($app));

Avoid too many return statements within this method.

Open

            return new FileProfilerStorage('file:' . $app['profiler.cache_dir']);

Avoid too many return statements within this method.

Open

                return $extensions;

Avoid too many return statements within this method.

Open

                return new \Twig_Profiler_Profile();

Avoid too many return statements within this method.

Open

                return $collectors;

Avoid too many return statements within this method.

Open

                return $collectors;

Avoid too many return statements within this method.

Open

            return new ExceptionController($app['profiler'], $app['twig'], $app['debug']);

Avoid too many return statements within this method.

Open

            return $loader;

Avoid too many return statements within this method.

Open

                    return new AjaxDataCollector();

Avoid too many return statements within this method.

Open

                return new DumpListener($app['var_dumper.cloner'], $app['var_dumper.data_collector']);

Avoid too many return statements within this method.

Open

                return $collectors;

Avoid too many return statements within this method.

Open

                    return Yaml::dump($var);

Avoid too many return statements within this method.

Open

            return new ProfilerController($app['url_generator'], $app['profiler'], $app['twig'], $app['data_collector.templates'], $app['web_profiler.debug_toolbar.position']);

Avoid too many return statements within this method.

Open

                        return $app['var_dumper.data_collector'] = new DumpDataCollector($app['stopwatch'], NULL, $app['charset'], $app['request_stack'], $dumper);

Avoid too many return statements within this method.

Open

                return $templates;

Avoid too many return statements within this method.

Open

                    return Yaml::dump($var);

Avoid too many return statements within this method.

Open

                return $collectors;

Avoid too many return statements within this method.

Open

                    return new $sec_data_collector($app['security.token_storage'], $roleHierarchy, $logoutUrlGenerator);

Avoid too many return statements within this method.

Open

            return dirname(dirname($r->getFileName())) . '/Resources/views';

Avoid too many return statements within this method.

Open

            return new Stopwatch();

Avoid too many return statements within this method.

Open

                    return dirname($file) . '/Resources/views';

Avoid too many return statements within this method.

Open

            return new RouterController($app['profiler'], $app['twig'], $app['request_matcher'] ?? NULL, $app['routes']);

Avoid too many return statements within this method.

Open

                $collectors['form'] = function ($app) { return new FormDataCollector($app['data_collectors.form.extractor']); };

Avoid too many return statements within this method.

Open

                return $collectors;

Avoid too many return statements within this method.

Open

                return $loader;

Avoid too many return statements within this method.

Open

                return dirname(dirname($r->getFileName())) . '/Resources/views';

Avoid too many return statements within this method.

Open

                return new ProfilerListener($app['profiler'], $app['request_stack'], $app['profiler.request_matcher'], $app['profiler.only_exceptions'], $app['profiler.only_master_requests']);

Avoid too many return statements within this method.

Open

                return new ProfilerListener($app['profiler'], $app['profiler.request_matcher'], $app['profiler.only_exceptions'], $app['profiler.only_master_requests'], $app['request_stack']);

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

Open

    public function register(Container $app)
    {
        $app['profiler.mount_prefix'] = '/_profiler';
        $app->extend('dispatcher', function ($dispatcher, $app) {
            return new TraceableEventDispatcher($dispatcher, $app['stopwatch'], $app['logger']);

• Exclude checks
  • Disable engine
  • Disable check

The method register() has an NPath complexity of 1548288. The configured NPath complexity threshold is 200.

Open

    public function register(Container $app)
    {
        $app['profiler.mount_prefix'] = '/_profiler';
        $app->extend('dispatcher', function ($dispatcher, $app) {
            return new TraceableEventDispatcher($dispatcher, $app['stopwatch'], $app['logger']);

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

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

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

Open

    public function register(Container $app)
    {
        $app['profiler.mount_prefix'] = '/_profiler';
        $app->extend('dispatcher', function ($dispatcher, $app) {
            return new TraceableEventDispatcher($dispatcher, $app['stopwatch'], $app['logger']);

• 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;
            }
        }
    }
}

The class F9WebProfilerServiceProvider has a coupling between objects value of 41. Consider to reduce the number of dependencies under 13.

Open

class F9WebProfilerServiceProvider implements ServiceProviderInterface, ControllerProviderInterface, BootableProvider, EventListenerProviderInterface
{
    public function boot($app)
    {
        $app->mount($app['profiler.mount_prefix'], $this->connect($app));

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

class Foo {
    /**
     * @var \foo\bar\X
     */
    private $x = null;

    /**
     * @var \foo\bar\Y
     */
    private $y = null;

    /**
     * @var \foo\bar\Z
     */
    private $z = null;

    public function setFoo(\Foo $foo) {}
    public function setBar(\Bar $bar) {}
    public function setBaz(\Baz $baz) {}

    /**
     * @return \SplObjectStorage
     * @throws \OutOfRangeException
     * @throws \InvalidArgumentException
     * @throws \ErrorException
     */
    public function process(\Iterator $it) {}

    // ...
}

Missing class import via use statement (line '171', column '28').

Open

                return new \Twig_Profiler_Profile();

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

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

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

Open

            throw new \LogicException('You must enable the ServiceController service provider to be able to use the WebProfiler.');

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

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

Missing class import via use statement (line '238', column '53').

Open

                $twig->addFilter('yaml_encode', new \Twig_SimpleFilter('yaml_encode', function (array $var) {

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

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

Missing class import via use statement (line '242', column '55').

Open

                $twig->addFunction('yaml_encode', new \Twig_SimpleFunction('yaml_encode', function (array $var) {

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

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

Missing class import via use statement (line '232', column '26').

Open

                $r = new \ReflectionClass('Symfony\Bundle\SecurityBundle\DataCollector\SecurityDataCollector');

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

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

Missing class import via use statement (line '330', column '22').

Open

            $r = new \ReflectionClass('Symfony\Bundle\WebProfilerBundle\EventListener\WebDebugToolbarListener');

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

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

The method register uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            else {
                /** @noinspection PhpParamsInspection */
                return new ProfilerListener($app['profiler'], $app['profiler.request_matcher'], $app['profiler.only_exceptions'], $app['profiler.only_master_requests'], $app['request_stack']);
            }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

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

Open

    public function register(Container $app)
    {
        $app['profiler.mount_prefix'] = '/_profiler';
        $app->extend('dispatcher', function ($dispatcher, $app) {
            return new TraceableEventDispatcher($dispatcher, $app['stopwatch'], $app['logger']);

• 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 variables with short names like $r. Configured minimum length is 3.

Open

                $r = new \ReflectionClass('Symfony\Bundle\SecurityBundle\DataCollector\SecurityDataCollector');

• 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 $sec_data_collector is not named in camelCase.

Open

    public function register(Container $app)
    {
        $app['profiler.mount_prefix'] = '/_profiler';
        $app->extend('dispatcher', function ($dispatcher, $app) {
            return new TraceableEventDispatcher($dispatcher, $app['stopwatch'], $app['logger']);

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

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

The variable $sec_data_collector is not named in camelCase.

Open

    public function register(Container $app)
    {
        $app['profiler.mount_prefix'] = '/_profiler';
        $app->extend('dispatcher', function ($dispatcher, $app) {
            return new TraceableEventDispatcher($dispatcher, $app['stopwatch'], $app['logger']);

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

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