Config::getJsConfig() - Code Metrics - Inspection of "Simplifying miniPHP" - OmarElGabry/miniPHP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 17ed34...f4bd34 )

by Omar El

created 2016-04-14 22:27 UTC

Config::getJsConfig() A

↳ Parent: Config

Complexity

Conditions	4
Paths	6

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	13
rs	9.2
cc	4
eloc	8
nc	6
nop	1

<?php

/**
 * Config class.
 * Gets a configuration value
 *
 * @license    http://opensource.org/licenses/MIT The MIT License (MIT)
 * @author     Omar El Gabry <[email protected]>
 */

class Config{
namespace YourVendor;

class YourClass { }

    /**
     * Array of configurations
     *
     * @var array
     */
    public static $config = null;

    /**
     * Array of javascript configurations
     *
     * @var array
     */
    public static $jsConfig = null;

    /**
     * Gets a configuration value
     *
     * @param $key string
     * @return string|null
     * @throws Exception if configuration file doesn't exist
     */
    public static function get($key){

        if (!self::$config) {


	        $config_file = APP . 'config/config.php';

			if (!file_exists($config_file)) {
				throw new Exception("Configuration file doesn't exist");
			}

	        self::$config = require $config_file . "";
        }

        return isset(self::$config[$key])? self::$config[$key]: null;
    }

    /**
     * Loads javascript configurations
     *
     * @param $key string
     * @return string|null
     * @throws Exception if configuration file doesn't exist
     */
    private static function loadJsConfig(){

        if (!self::$jsConfig) {


            $config_file = APP . 'config/javascript.php';

            if (!file_exists($config_file)) {
                throw new Exception("JavaScript Configuration file doesn't exist");
            }

            self::$jsConfig = require $config_file . "";
        }
    }

    /**
     * Gets javascript configuration value(s)
     *
     * @param $key string
     * @return string|array|null
     * @throws Exception if configuration file doesn't exist
     */
    public static function getJsConfig($key = ""){
        
        if (!self::$jsConfig) {

            self::loadJsConfig();
        }

        if(empty($key)){
            return self::$jsConfig;
        }else if(isset(self::$jsConfig[$key])){
            return self::$jsConfig[$key];
        }
        return null;
    }

    /**
     * Adds a new variable to javascript configurations
     *
     * @param string $key
     * @param mixed  $value
     */
    public static function addJsConfig($key, $value){
        if (!self::$jsConfig) {

            self::loadJsConfig();
        }
        self::$jsConfig[$key] = $value;
    }
}


1		<?php
2
3		/**
4		* Config class.
5		* Gets a configuration value
6		*
7		* @license http://opensource.org/licenses/MIT The MIT License (MIT)
8		* @author Omar El Gabry <[email protected]>
9		*/
10
11		class Config{
		0 ignored issues – show Coding Style Compatibility introduced 2015-11-27 07:13 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
12
13		/**
14		* Array of configurations
15		*
16		* @var array
17		*/
18		public static $config = null;
19
20		/**
21		* Array of javascript configurations
22		*
23		* @var array
24		*/
25		public static $jsConfig = null;
26
27		/**
28		* Gets a configuration value
29		*
30		* @param $key string
31		* @return string\|null
32		* @throws Exception if configuration file doesn't exist
33		*/
34		public static function get($key){
35
36	View Code Duplication	if (!self::$config) {
		0 ignored issues – show Bug Best Practice introduced 2015-11-27 07:13 UTC by Report Bug Copy Issue Report The expression `self::$config` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history... Duplication introduced 2016-04-14 22:30 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...
37
38		$config_file = APP . 'config/config.php';
39
40		if (!file_exists($config_file)) {
41		throw new Exception("Configuration file doesn't exist");
42		}
43
44		self::$config = require $config_file . "";
45		}
46
47		return isset(self::$config[$key])? self::$config[$key]: null;
48		}
49
50		/**
51		* Loads javascript configurations
52		*
53		* @param $key string
54		* @return string\|null
55		* @throws Exception if configuration file doesn't exist
56		*/
57		private static function loadJsConfig(){
58
59	View Code Duplication	if (!self::$jsConfig) {
		0 ignored issues – show Bug Best Practice introduced 2016-04-14 22:30 UTC by Report Bug Copy Issue Report The expression `self::$jsConfig` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history... Duplication introduced 2016-04-14 22:30 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...
60
61		$config_file = APP . 'config/javascript.php';
62
63		if (!file_exists($config_file)) {
64		throw new Exception("JavaScript Configuration file doesn't exist");
65		}
66
67		self::$jsConfig = require $config_file . "";
68		}
69		}
70
71		/**
72		* Gets javascript configuration value(s)
73		*
74		* @param $key string
75		* @return string\|array\|null
76		* @throws Exception if configuration file doesn't exist
77		*/
78		public static function getJsConfig($key = ""){
79
80		if (!self::$jsConfig) {
		0 ignored issues – show Bug Best Practice introduced 2016-04-14 22:30 UTC by Report Bug Copy Issue Report The expression `self::$jsConfig` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
81		self::loadJsConfig();
82		}
83
84		if(empty($key)){
85		return self::$jsConfig;
86		}else if(isset(self::$jsConfig[$key])){
87		return self::$jsConfig[$key];
88		}
89		return null;
90		}
91
92		/**
93		* Adds a new variable to javascript configurations
94		*
95		* @param string $key
96		* @param mixed $value
97		*/
98		public static function addJsConfig($key, $value){
99		if (!self::$jsConfig) {
		0 ignored issues – show Bug Best Practice introduced 2016-04-14 22:30 UTC by Report Bug Copy Issue Report The expression `self::$jsConfig` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
100		self::loadJsConfig();
101		}
102		self::$jsConfig[$key] = $value;
103		}
104		}
105

OmarElGabry / miniPHP

GitHub Access Token became invalid

Push — master ( 17ed34...f4bd34 )

Config::getJsConfig() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like