w() - Code Metrics - Inspection of "Php81" - Ecodev/felix - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — master (#11)

by Adrien

created 2022-02-17 11:53 UTC

w() A

↳ Parent: Project

Complexity

Conditions	6
Paths	1

Size

Total Lines	9
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	6
eloc	7
c	1
b	0
f	0
nc	1
nop	0
dl	0
loc	9
ccs	0
cts	7
cp	0
crap	42
rs	9.2222

<?php

declare(strict_types=1);

use Doctrine\ORM\EntityManager;
use Ecodev\Felix\Debug;
use GraphQL\Doctrine\Types;
use Laminas\Log\LoggerInterface;

/**
 * Returns the type registry.
 */
function _types(): Types
{
    global $container;

    return $container->get(Types::class);
}

/**
 * Returns the Entity Manager.
 */
function _em(): EntityManager
{
    global $container;

    return $container->get(EntityManager::class);
}

/**
 * Returns logger.
 */
function _log(): LoggerInterface
{
    global $container;

    return $container->get(LoggerInterface::class);
}

/**
 * Export variables omitting array keys that are strictly numeric.
 *
 * By default will output result
 *
 * @param mixed $data
 *
 * @return string string representation of variable
 */
function ve($data, bool $return = false): string
{
    return Debug::export($data, $return);
}

/**
 * Dump all arguments.
 */
function v(): void
{
    var_dump(func_get_args());

}

/**
 * Dump all arguments and die.
 */
function w(): never
filter:
    dependency_paths: ["lib/*"]
{
    $isHtml = (PHP_SAPI !== 'cli');
    echo "\n_________________________________________________________________________________________________________________________" . ($isHtml ? '</br>' : '') . "\n";
    var_dump(func_get_args());

    echo "\n" . ($isHtml ? '</br>' : '') . '_________________________________________________________________________________________________________________________' . ($isHtml ? '<pre>' : '') . "\n";
    debug_print_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
    echo '' . ($isHtml ? '</pre>' : '') . '_________________________________________________________________________________________________________________________' . ($isHtml ? '</br>' : '') . "\n";
    exit("script aborted on purpose.\n");
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}
}


1		<?php
2
3		declare(strict_types=1);
4
5		use Doctrine\ORM\EntityManager;
6		use Ecodev\Felix\Debug;
7		use GraphQL\Doctrine\Types;
8		use Laminas\Log\LoggerInterface;
9
10		/**
11		* Returns the type registry.
12		*/
13		function _types(): Types
14		{
15		global $container;
16
17		return $container->get(Types::class);
18		}
19
20		/**
21		* Returns the Entity Manager.
22		*/
23		function _em(): EntityManager
24		{
25		global $container;
26
27	7	return $container->get(EntityManager::class);
28		}
29
30		/**
31		* Returns logger.
32		*/
33		function _log(): LoggerInterface
34		{
35		global $container;
36
37		return $container->get(LoggerInterface::class);
38		}
39
40		/**
41		* Export variables omitting array keys that are strictly numeric.
42		*
43		* By default will output result
44		*
45		* @param mixed $data
46		*
47		* @return string string representation of variable
48		*/
49		function ve($data, bool $return = false): string
50		{
51		return Debug::export($data, $return);
52		}
53
54		/**
55		* Dump all arguments.
56		*/
57		function v(): void
58		{
59		var_dump(func_get_args());
		0 ignored issues – show Security Debugging Code introduced 2020-05-07 11:05 UTC by Report Bug Copy Issue Report `var_dump(func_get_args())` looks like debug code. Are you sure you do not want to remove it? Loading history...
60		}
61
62		/**
63		* Dump all arguments and die.
64		*/
65		function w(): never
		1 ignored issue – show Bug introduced 2022-02-17 12:09 UTC by Report Bug Copy Issue Report The type `never` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
66		{
67		$isHtml = (PHP_SAPI !== 'cli');
68		echo "\n_________________________________________________________________________________________________________________________" . ($isHtml ? '</br>' : '') . "\n";
69		var_dump(func_get_args());
		0 ignored issues – show Security Debugging Code introduced 2020-05-07 11:05 UTC by Report Bug Copy Issue Report `var_dump(func_get_args())` looks like debug code. Are you sure you do not want to remove it? Loading history...
70		echo "\n" . ($isHtml ? '</br>' : '') . '_________________________________________________________________________________________________________________________' . ($isHtml ? '<pre>' : '') . "\n";
71		debug_print_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
72		echo '' . ($isHtml ? '</pre>' : '') . '_________________________________________________________________________________________________________________________' . ($isHtml ? '</br>' : '') . "\n";
73		exit("script aborted on purpose.\n");
		0 ignored issues – show Best Practice introduced 2020-05-07 11:05 UTC by Report Bug Copy Issue Report Using `exit` here is not recommended. In general, usage of exit should be done with care and only when running in a scripting context like a CLI script. Loading history... Bug Best Practice introduced 2022-02-17 12:09 UTC by Report Bug Copy Issue Report In this branch, the function will implicitly return `null` which is incompatible with the type-hinted return `never`. Consider adding a return statement or allowing `null` as return value. For hinted functions/methods where all return statements with the correct type are only reachable via conditions, ?null? gets implicitly returned which may be incompatible with the hinted type. Let?s take a look at an example: interface ReturnsInt { public function returnsIntHinted(): int; } class MyClass implements ReturnsInt { public function returnsIntHinted(): int { if (foo()) { return 123; } // here: null is implicitly returned } } Loading history...
74		}
75

Ecodev / felix

Pull Request — master (#11)

w() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like