Config::set() - Code Metrics - Inspection of "remove database & model features from framework" - zewadesign/framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#81)

unknown

created 2016-11-09 11:14 UTC

Config::set() A

↳ Parent: Config

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	5
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	3
nc	1
nop	2

<?php
namespace Zewa;

use Zewa\Exception\ConfigException;

class Config
{
    /**
     * Loaded Configuration Items
     *
     * @var array
     */
    protected $configuration = [];

    /**
     * Path to Configuration Folder
     *
     * @var string
     */
    protected $path;

    /**
     * Configuration file extension
     */
    const CONFIG_FILE_EXTENSION = ".php";

    /**
     * Config constructor.
     *
     * @param $configFolderPath
     */
    public function __construct($configFolderPath = null)
    {
        if (is_null($configFolderPath)) {
            $configFolderPath = APP_PATH . DIRECTORY_SEPARATOR . 'Config';
        }

        $this->setPath($configFolderPath);
    }

    /**
     * Sets the configuration folder path
     *
     * @param $path
     */
    protected function setPath($path)
    {
        $this->path = $path;
    }

    /**
     * Loads a configuration file in to memory
     *
     * @param $key
     * @return bool
     */
    protected function loadConfigFile(string $key) : bool
    {
        $key = strtolower($key);
        $filename = $this->path . DIRECTORY_SEPARATOR . ucfirst($key) . Config::CONFIG_FILE_EXTENSION;

        if (file_exists($filename)) {
            $this->configuration[$key] = require $filename;
            return true;
        }

        return false;
    }

    /**
     * Get Configuration Item
     *
     * @param $key
     *
     * @return \stdClass
     * @throws ConfigException when config file not found
     */
    public function get(string $key)
    {
        $key = strtolower($key);
        $value = [];
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        if (isset($this->configuration[$key]) || $this->loadConfigFile($key)) {
            $value = $this->configuration[$key];
        } else {
            throw new ConfigException($key . ' configuration file is missing.');
        }

        return $value;
    }

    /**
     * @param string $key
     * @param $value mixed array|string
     */
    public function set(string $key, $value)
    {
        $key = strtolower($key);
        $this->configuration[$key] = $value;
    }
}


1			<?php
2			namespace Zewa;
3
4			use Zewa\Exception\ConfigException;
5
6			class Config
7			{
8			/**
9			* Loaded Configuration Items
10			*
11			* @var array
12			*/
13			protected $configuration = [];
14
15			/**
16			* Path to Configuration Folder
17			*
18			* @var string
19			*/
20			protected $path;
21
22			/**
23			* Configuration file extension
24			*/
25			const CONFIG_FILE_EXTENSION = ".php";
26
27			/**
28			* Config constructor.
29			*
30			* @param $configFolderPath
31			*/
32			public function __construct($configFolderPath = null)
33			{
34			if (is_null($configFolderPath)) {
35			$configFolderPath = APP_PATH . DIRECTORY_SEPARATOR . 'Config';
36			}
37
38			$this->setPath($configFolderPath);
39			}
40
41			/**
42			* Sets the configuration folder path
43			*
44			* @param $path
45			*/
46			protected function setPath($path)
47			{
48			$this->path = $path;
49			}
50
51			/**
52			* Loads a configuration file in to memory
53			*
54			* @param $key
55			* @return bool
56			*/
57			protected function loadConfigFile(string $key) : bool
58			{
59			$key = strtolower($key);
60			$filename = $this->path . DIRECTORY_SEPARATOR . ucfirst($key) . Config::CONFIG_FILE_EXTENSION;
61
62			if (file_exists($filename)) {
63			$this->configuration[$key] = require $filename;
64			return true;
65			}
66
67			return false;
68			}
69
70			/**
71			* Get Configuration Item
72			*
73			* @param $key
74			*
75			* @return \stdClass
76			* @throws ConfigException when config file not found
77			*/
78			public function get(string $key)
79			{
80			$key = strtolower($key);
81			$value = [];
			0 ignored issues – show Unused Code introduced 2016-11-09 10:30 UTC by Report Bug Copy Issue Report `$value` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
82
83			if (isset($this->configuration[$key]) \|\| $this->loadConfigFile($key)) {
84			$value = $this->configuration[$key];
85			} else {
86			throw new ConfigException($key . ' configuration file is missing.');
87			}
88
89			return $value;
90			}
91
92			/**
93			* @param string $key
94			* @param $value mixed array\|string
95			*/
96			public function set(string $key, $value)
97			{
98			$key = strtolower($key);
99			$this->configuration[$key] = $value;
100			}
101			}
102

zewadesign / framework

Pull Request — master (#81)

Config::set() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like