AssetEngineManager::addAssetEngine() - Code Metrics - Inspection of "Let's get sassy" - stakx-io/stakx - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#69)

by Vladimir

created 2018-06-26 06:21 UTC

AssetEngineManager::addAssetEngine() A

↳ Parent: AssetEngineManager

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
c	0
b	0
f	0
rs	9.8333
cc	2
nc	2
nop	1

<?php

/**
 * @copyright 2018 Vladimir Jimenez
 * @license   https://github.com/allejo/stakx/blob/master/LICENSE.md MIT
 */

namespace allejo\stakx\AssetEngine;

use __;
use allejo\stakx\Exception\UnsupportedAssetEngineException;

class AssetEngineManager
{
    private $enginesByExtension = [];
    private $foldersToWatch = [];
    private $engines;

    public function addAssetEngines(/* iterable */ $assetEngines)
    {
        foreach ($assetEngines as $assetEngine)
        {
            $this->addAssetEngine($assetEngine);
        }
    }

    public function addAssetEngine(AssetEngine $assetEngine)
    {
        $extensions = $assetEngine->getExtensions();

        $e = $this->engines[] = $assetEngine;


        foreach ($extensions as $extension)
        {
            $this->enginesByExtension[$extension] = $e;
        }

        $this->foldersToWatch[$assetEngine->getFolder()] = $e;
    }

    public function getEngineByExtension($extension)

    {
        if (isset($this->enginesByExtension[$extension]))
        {
            return $this->enginesByExtension[$extension];
        }

        throw new UnsupportedAssetEngineException($extension, 'There is no support to handle this asset type.');
    }

    public function getFoldersToWatch()
    {
        return $this->foldersToWatch;
    }
}


1			<?php
2
3			/**
4			* @copyright 2018 Vladimir Jimenez
5			* @license https://github.com/allejo/stakx/blob/master/LICENSE.md MIT
6			*/
7
8			namespace allejo\stakx\AssetEngine;
9
10			use __;
11			use allejo\stakx\Exception\UnsupportedAssetEngineException;
12
13			class AssetEngineManager
14			{
15			private $enginesByExtension = [];
16			private $foldersToWatch = [];
17			private $engines;
18
19			public function addAssetEngines(/* iterable */ $assetEngines)
20			{
21			foreach ($assetEngines as $assetEngine)
22			{
23			$this->addAssetEngine($assetEngine);
24			}
25			}
26
27			public function addAssetEngine(AssetEngine $assetEngine)
28			{
29			$extensions = $assetEngine->getExtensions();
30
31			$e = $this->engines[] = $assetEngine;
			0 ignored issues – show Comprehensibility introduced 2018-06-26 06:26 UTC by Report Bug Copy Issue Report Avoid variables with short names like $e. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
32
33			foreach ($extensions as $extension)
34			{
35			$this->enginesByExtension[$extension] = $e;
36			}
37
38			$this->foldersToWatch[$assetEngine->getFolder()] = $e;
39			}
40
41			public function getEngineByExtension($extension)
			0 ignored issues – show Documentation introduced 2018-06-26 06:26 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
42			{
43			if (isset($this->enginesByExtension[$extension]))
44			{
45			return $this->enginesByExtension[$extension];
46			}
47
48			throw new UnsupportedAssetEngineException($extension, 'There is no support to handle this asset type.');
49			}
50
51			public function getFoldersToWatch()
52			{
53			return $this->foldersToWatch;
54			}
55			}
56

stakx-io / stakx

Pull Request — master (#69)

AssetEngineManager::addAssetEngine() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like