Config::load() - Code Metrics - Inspection of "rootDirectory path and default config" - pageon/stitcher-core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( 9dc7fc...abb005 )

by Brent

created 2017-12-10 14:07 UTC

Config::load() A

↳ Parent: Config

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3.0175

Importance

Changes

Metric	Value
cc	3
eloc	8
nc	3
nop	1
dl	0
loc	16
ccs	7
cts	8
cp	0.875
crap	3.0175
rs	9.4285
c	0
b	0
f	0

<?php

namespace Pageon;

use Dotenv\Dotenv;
use Dotenv\Exception\InvalidPathException;
use Illuminate\Support\Arr;
use Iterator;
use Stitcher\Exception\InvalidConfiguration;
use Stitcher\File;
use Symfony\Component\Finder\Finder;

class Config
{
    protected static $env;
    protected static $loadedConfiguration = [];

    public static function init()
    {
        self::$env = new Dotenv(File::path());

        try {
            self::$env->load();
        } catch (InvalidPathException $e) {
            throw InvalidConfiguration::dotEnvNotFound(File::path());
        }

        $configurationFiles = Finder::create()->files()->in(File::path('config'))->name('*.php')->getIterator();

        self::loadDefaults();

        $loadedConfiguration = self::load($configurationFiles);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

        self::$loadedConfiguration = array_merge(self::$loadedConfiguration, Arr::dot($loadedConfiguration));
    }

    public static function get(string $key)
    {
        return self::$loadedConfiguration[$key] ?? null;
    }

    public static function all(): array
    {
        return self::$loadedConfiguration;
    }

    protected static function loadDefaults(): void
    {
        self::$loadedConfiguration['rootDirectory'] = File::path();
        self::$loadedConfiguration['templateRenderer'] = 'twig';
    }

    protected static function load(Iterator $configurationFiles): array
    {
        $loadedConfiguration = [];

        foreach ($configurationFiles as $configurationFile) {
            $loadedFileConfiguration = require $configurationFile;

            if (!is_array($loadedFileConfiguration)) {
                continue;
            }

            $loadedConfiguration = array_merge($loadedConfiguration, $loadedFileConfiguration);
        }

        return $loadedConfiguration;
    }
}


1		<?php
2
3		namespace Pageon;
4
5		use Dotenv\Dotenv;
6		use Dotenv\Exception\InvalidPathException;
7		use Illuminate\Support\Arr;
8		use Iterator;
9		use Stitcher\Exception\InvalidConfiguration;
10		use Stitcher\File;
11		use Symfony\Component\Finder\Finder;
12
13		class Config
14		{
15		protected static $env;
16		protected static $loadedConfiguration = [];
17
18	7	public static function init()
19		{
20	7	self::$env = new Dotenv(File::path());
21
22		try {
23	7	self::$env->load();
24		} catch (InvalidPathException $e) {
25		throw InvalidConfiguration::dotEnvNotFound(File::path());
26		}
27
28	7	$configurationFiles = Finder::create()->files()->in(File::path('config'))->name('*.php')->getIterator();
29
30	7	self::loadDefaults();
31
32	7	$loadedConfiguration = self::load($configurationFiles);
		0 ignored issues – show Bug introduced 2017-12-03 08:28 UTC by Report Bug Copy Issue Report It seems like `$configurationFiles` defined by `\Symfony\Component\Finde...'.php')->getIterator()` on line `28` can also be of type `array<integer,object<Sym...nt\Finder\SplFileInfo>>`; however, `Pageon\Config::load()` does only seem to accept `object<Iterator>`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /* * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
33
34	7	self::$loadedConfiguration = array_merge(self::$loadedConfiguration, Arr::dot($loadedConfiguration));
35	7	}
36
37	4	public static function get(string $key)
38		{
39	4	return self::$loadedConfiguration[$key] ?? null;
40		}
41
42	2	public static function all(): array
43		{
44	2	return self::$loadedConfiguration;
45		}
46
47	7	protected static function loadDefaults(): void
48		{
49	7	self::$loadedConfiguration['rootDirectory'] = File::path();
50	7	self::$loadedConfiguration['templateRenderer'] = 'twig';
51	7	}
52
53	7	protected static function load(Iterator $configurationFiles): array
54		{
55	7	$loadedConfiguration = [];
56
57	7	foreach ($configurationFiles as $configurationFile) {
58	7	$loadedFileConfiguration = require $configurationFile;
59
60	7	if (!is_array($loadedFileConfiguration)) {
61		continue;
62		}
63
64	7	$loadedConfiguration = array_merge($loadedConfiguration, $loadedFileConfiguration);
65		}
66
67	7	return $loadedConfiguration;
68		}
69		}
70

pageon / stitcher-core

Push — develop ( 9dc7fc...abb005 )

Config::load() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like