helpers.php ➔ kernel() - Code Metrics - Inspection of "revert back `env` helper" - fondbot/framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 6d232f...71788a )

by Vladimir

created 2017-07-18 21:32 UTC

helpers.php ➔ kernel() A

↳ Parent: Project

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	1
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	0
dl	0
loc	4
ccs	1
cts	1
cp	1
crap	1
rs	10
c	0
b	0
f	0

<?php

declare(strict_types=1);

if (!function_exists('env')) {
    /**
     * Get environment variable value.
     *
     * @param string $key
     * @param null   $default
     *
     * @return mixed
     */
    function env(string $key, $default = null)
    {
        $value = $_ENV[$key] ?? null;
        if ($value === null) {
            return $default;
        }
        switch (mb_strtolower($value)) {
            case 'true':
switch ($expr) {
    case "A": { //wrong
        doSomething();
        break;
    }
    case "B"; //wrong
        doSomething();
        break;
    case "C": //right
        doSomething();
        break;
}
                return true;
            case 'false':
switch ($expr) {
    case "A": { //wrong
        doSomething();
        break;
    }
    case "B"; //wrong
        doSomething();
        break;
    case "C": //right
        doSomething();
        break;
}
                return false;
            case 'null':
                return null;
        }

        return $value;
    }
}

if (!function_exists('kernel')) {
    /**
     * Get kernel instance.
     *
     * @return \FondBot\Foundation\Kernel
     */
    function kernel(): \FondBot\Foundation\Kernel
    {
        return FondBot\Foundation\Kernel::getInstance();
    }
}

if (!function_exists('resolve')) {
    /**
     * Resolve an alias from container.
     *
     * @param string $alias
     * @param array  $args
     *
     * @return mixed
     */
    function resolve(string $alias, array $args = [])
    {
        return kernel()->resolve($alias, $args);
    }
}

if (!function_exists('session')) {
    /**
     * Get session.
     *
     * @return FondBot\Conversation\Session
     */
    function session(): FondBot\Conversation\Session
    {
        return kernel()->getSession();
    }
}

if (!function_exists('context')) {
    /**
     * Get context.
     *
     * @return FondBot\Conversation\Context
     */
    function context(): FondBot\Conversation\Context
    {
        return kernel()->getContext();
    }
}

if (!function_exists('path')) {

    /**
     * Get path.
     *
     * @param string $postfix
     *
     * @return string
     */
    function path(string $postfix = null): string
    {
        $path = resolve('base_path');

        if ($postfix === null) {
            return $path;
        }

        return $path.'/'.$postfix;
    }
}

if (!function_exists('resources')) {

    /**
     * Get resources path.
     *
     * @param string $postfix
     *
     * @return string
     */
    function resources(string $postfix = null): string
    {
        $path = resolve('resources_path');

        if ($postfix === null) {
            return $path;
        }

        return $path.'/'.$postfix;
    }
}

if (!function_exists('logger')) {
    /**
     * Get logger.
     *
     * @return Monolog\Logger|Psr\Log\LoggerInterface
     */
    function logger(): Monolog\Logger
    {
        return resolve(Monolog\Logger::class);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			if (!function_exists('env')) {
6			/**
7			* Get environment variable value.
8			*
9			* @param string $key
10			* @param null $default
11			*
12			* @return mixed
13			*/
14			function env(string $key, $default = null)
15			{
16	3		$value = $_ENV[$key] ?? null;
17	3		if ($value === null) {
18	1		return $default;
19			}
20	3		switch (mb_strtolower($value)) {
21	3		case 'true':
			0 ignored issues – show Coding Style introduced 2017-05-12 16:02 UTC by Report Bug Copy Issue Report `case` statements should be defined using a colon. As per the PSR-2 coding standard, case statements should not be wrapped in curly braces. There is no need for braces, since each case is terminated by the next `break`. There is also the option to use a semicolon instead of a colon, this is discouraged because many programmers do not even know it works and the colon is universal between programming languages. switch ($expr) { case "A": { //wrong doSomething(); break; } case "B"; //wrong doSomething(); break; case "C": //right doSomething(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
22	1		return true;
23	3		case 'false':
			0 ignored issues – show Coding Style introduced 2017-05-12 16:02 UTC by Report Bug Copy Issue Report `case` statements should be defined using a colon. As per the PSR-2 coding standard, case statements should not be wrapped in curly braces. There is no need for braces, since each case is terminated by the next `break`. There is also the option to use a semicolon instead of a colon, this is discouraged because many programmers do not even know it works and the colon is universal between programming languages. switch ($expr) { case "A": { //wrong doSomething(); break; } case "B"; //wrong doSomething(); break; case "C": //right doSomething(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
24	1		return false;
25	3		case 'null':
26	1		return null;
27			}
28
29	3		return $value;
30			}
31			}
32
33			if (!function_exists('kernel')) {
34			/**
35			* Get kernel instance.
36			*
37			* @return \FondBot\Foundation\Kernel
38			*/
39			function kernel(): \FondBot\Foundation\Kernel
40			{
41	33		return FondBot\Foundation\Kernel::getInstance();
42			}
43			}
44
45			if (!function_exists('resolve')) {
46			/**
47			* Resolve an alias from container.
48			*
49			* @param string $alias
50			* @param array $args
51			*
52			* @return mixed
53			*/
54			function resolve(string $alias, array $args = [])
55			{
56	28		return kernel()->resolve($alias, $args);
57			}
58			}
59
60			if (!function_exists('session')) {
61			/**
62			* Get session.
63			*
64			* @return FondBot\Conversation\Session
65			*/
66			function session(): FondBot\Conversation\Session
67			{
68	2		return kernel()->getSession();
69			}
70			}
71
72			if (!function_exists('context')) {
73			/**
74			* Get context.
75			*
76			* @return FondBot\Conversation\Context
77			*/
78			function context(): FondBot\Conversation\Context
79			{
80	2		return kernel()->getContext();
81			}
82			}
83
84		View Code Duplication	if (!function_exists('path')) {
			0 ignored issues – show Duplication introduced 2017-05-23 11:38 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
85			/**
86			* Get path.
87			*
88			* @param string $postfix
89			*
90			* @return string
91			*/
92			function path(string $postfix = null): string
93			{
94	1		$path = resolve('base_path');
95
96	1		if ($postfix === null) {
97	1		return $path;
98			}
99
100	1		return $path.'/'.$postfix;
101			}
102			}
103
104		View Code Duplication	if (!function_exists('resources')) {
			0 ignored issues – show Duplication introduced 2017-05-23 11:38 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
105			/**
106			* Get resources path.
107			*
108			* @param string $postfix
109			*
110			* @return string
111			*/
112			function resources(string $postfix = null): string
113			{
114	1		$path = resolve('resources_path');
115
116	1		if ($postfix === null) {
117	1		return $path;
118			}
119
120	1		return $path.'/'.$postfix;
121			}
122			}
123
124			if (!function_exists('logger')) {
125			/**
126			* Get logger.
127			*
128			* @return Monolog\Logger\|Psr\Log\LoggerInterface
129			*/
130			function logger(): Monolog\Logger
131			{
132	1		return resolve(Monolog\Logger::class);
133			}
134			}
135

fondbot / framework

Push — master ( 6d232f...71788a )

helpers.php ➔ kernel() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like