Debug::enable() - Code Metrics - Inspection of "Merge branch 'develop'" - ByTIC/framework - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( c5ade0...e039e4 )

by Gabriel

created 2017-07-29 13:11 UTC

Debug::enable() C

↳ Parent: Debug

Complexity

Conditions	8
Paths	13

Size

Total Lines	32
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	8
eloc	19
nc	13
nop	2
dl	0
loc	32
rs	5.3846
c	0
b	0
f	0

<?php

namespace Nip\Debug;

use Symfony\Component\Debug\BufferingLogger;
use Symfony\Component\Debug\Debug as SymfonyDebug;

/**
 * Class Debug
 * @package Nip\Debug
 */
class Debug extends SymfonyDebug
{
    private static $enabled = false;


    /**
     * Enables the debug tools.
     *
     * This method registers an error handler and an exception handler.
     *
     * If the Symfony ClassLoader component is available, a special
     * class loader is also registered.
     *
     * @param int $errorReportingLevel The level of error reporting you want
     * @param bool $displayErrors Whether to display errors (for development) or just log them (for production)
     */
    public static function enable($errorReportingLevel = E_ALL, $displayErrors = true)
    {
        if (static::$enabled) {
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
            return;
        }

        static::$enabled = true;
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}

        if (null !== $errorReportingLevel) {
            error_reporting($errorReportingLevel);
        } else {
            error_reporting(E_ALL);
        }

        if ('cli' !== PHP_SAPI) {
            ini_set('display_errors', 0);
            ExceptionHandler::register($displayErrors);
        } elseif ($displayErrors && (!ini_get('log_errors') || ini_get('error_log'))) {
            // CLI - display errors only if they're not already logged to STDERR
            ini_set('display_errors', 1);
        }

        if ($displayErrors) {
            $handler = ErrorHandler::register(new ErrorHandler(new BufferingLogger()));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        } else {
            $handler = ErrorHandler::register();
            $handler->throwAt(0, true);
        }

        app('container')->share(ErrorHandler::class, $handler);
//        DebugClassLoader::enable();

    }
}


1			<?php
2
3			namespace Nip\Debug;
4
5			use Symfony\Component\Debug\BufferingLogger;
6			use Symfony\Component\Debug\Debug as SymfonyDebug;
7
8			/**
9			* Class Debug
10			* @package Nip\Debug
11			*/
12			class Debug extends SymfonyDebug
13			{
14			private static $enabled = false;
			0 ignored issues – show Comprehensibility introduced 2017-07-29 13:16 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
15
16			/**
17			* Enables the debug tools.
18			*
19			* This method registers an error handler and an exception handler.
20			*
21			* If the Symfony ClassLoader component is available, a special
22			* class loader is also registered.
23			*
24			* @param int $errorReportingLevel The level of error reporting you want
25			* @param bool $displayErrors Whether to display errors (for development) or just log them (for production)
26			*/
27			public static function enable($errorReportingLevel = E_ALL, $displayErrors = true)
28			{
29			if (static::$enabled) {
			0 ignored issues – show Bug introduced 2017-07-29 13:16 UTC by Report Bug Copy Issue Report Since `$enabled` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$enabled` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
30			return;
31			}
32
33			static::$enabled = true;
			0 ignored issues – show Bug introduced 2017-07-29 13:16 UTC by Report Bug Copy Issue Report Since `$enabled` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$enabled` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
34
35			if (null !== $errorReportingLevel) {
36			error_reporting($errorReportingLevel);
37			} else {
38			error_reporting(E_ALL);
39			}
40
41			if ('cli' !== PHP_SAPI) {
42			ini_set('display_errors', 0);
43			ExceptionHandler::register($displayErrors);
44			} elseif ($displayErrors && (!ini_get('log_errors') \|\| ini_get('error_log'))) {
45			// CLI - display errors only if they're not already logged to STDERR
46			ini_set('display_errors', 1);
47			}
48
49			if ($displayErrors) {
50			$handler = ErrorHandler::register(new ErrorHandler(new BufferingLogger()));
			0 ignored issues – show Documentation introduced 2017-07-29 13:16 UTC by Report Bug Copy Issue Report `new \Nip\Debug\ErrorHand...ebug\BufferingLogger())` is of type `object<Nip\Debug\ErrorHandler>`, but the function expects a `null\|object<self>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
51			} else {
52			$handler = ErrorHandler::register();
53			$handler->throwAt(0, true);
54			}
55
56			app('container')->share(ErrorHandler::class, $handler);
57			// DebugClassLoader::enable();
			0 ignored issues – show Unused Code Comprehensibility introduced 2017-07-29 13:16 UTC by Report Bug Copy Issue Report `58%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
58			}
59			}
60

ByTIC / framework

GitHub Access Token became invalid

Push — master ( c5ade0...e039e4 )

Debug::enable() C

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like