Module::requireClasses() - Code Metrics - Inspection of "Updated" - samsonos/php_core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 14f560...15f305 )

unknown

created 2016-10-04 09:22 UTC

Module::requireClasses() C

↳ Parent: Module

Complexity

Conditions	12
Paths	3

Size

Total Lines	23
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	12
eloc	14
c	0
b	0
f	0
nc	3
nop	0
dl	0
loc	23
rs	5.2987

How to fix Complexity

<?php declare(strict_types=1);

namespace samsonphp\core\loader\module;

use samsonframework\container\ContainerConfigurableInterface;

/**
 * Class Module
 *
 * @package samsonphp\core\loader\module
 */
class Module
{
    /** @var string Interface which mark as class configure the module */
    public static $containerConfigurableInterface = ContainerConfigurableInterface::class;

    /** @var string Module name */
    public $name;
    /** @var string Module class name */
    public $className;
    /** @var string Module path name */
    public $pathName;
    /** @var string Module path */
    public $path;
    /** @var array List of classes */
    public $classes = [];
    /** @var bool Is module has custom dependency injection configurator */
    public $isContainerConfigurable = false;
    /** @var string Configure dependency injection class */
    public $containerConfigurableClassName;
    public $composerParameters;

    public function __construct($name, $path, $classes)
    {
        $this->name = $name;
        $this->path = $path;
        $this->classes = $classes;

//        $this->requireClasses();
//        $this->checkIsContainerConfigurable();
    }

    /**
     * Require module classes
     */
    protected function requireClasses()
    {
        foreach ($this->classes as $classPath => $className) {


            $className = '\\' . ltrim($className, '\\');
//            if (preg_match('/dbQuery/', $className)) {
//                trace('sdf');
//            }
            if (file_exists($classPath)
                && !preg_match('/\/api\/generated\//', $className)
                && !preg_match('/activerecord\/dbQuery/', $className)
                && !preg_match('/samson\/activerecord\//', $className)
                && !preg_match('/Field.php$/', $classPath)
                && !preg_match('/Navigation.php$/', $classPath)
                && !preg_match('/TableVirtualCollection.php$/', $classPath)
                && !preg_match('/TableVirtualEntity.php$/', $classPath)
                && !preg_match('/TableVirtualQuery.php$/', $classPath)
                && !class_exists($className)
            ) {
                require_once($classPath);
            }
        }
    }

    /**
     * Check if class configures the container
     */
    public function checkIsContainerConfigurable()
    {
        // Find class with correct interface
        foreach ($this->classes as $classPath => $className) {
            $reflectionClass = new \ReflectionClass($className);
            // If class implements correct interface then it is container configurable
            if (in_array($this::$containerConfigurableInterface, $reflectionClass->getInterfaceNames(), true)) {
                $this->isContainerConfigurable = true;
                $this->containerConfigurableClassName = $className;
            }
        }
    }
}


1			<?php declare(strict_types=1);
2
3			namespace samsonphp\core\loader\module;
4
5			use samsonframework\container\ContainerConfigurableInterface;
6
7			/**
8			* Class Module
9			*
10			* @package samsonphp\core\loader\module
11			*/
12			class Module
13			{
14			/** @var string Interface which mark as class configure the module */
15			public static $containerConfigurableInterface = ContainerConfigurableInterface::class;
16
17			/** @var string Module name */
18			public $name;
19			/** @var string Module class name */
20			public $className;
21			/** @var string Module path name */
22			public $pathName;
23			/** @var string Module path */
24			public $path;
25			/** @var array List of classes */
26			public $classes = [];
27			/** @var bool Is module has custom dependency injection configurator */
28			public $isContainerConfigurable = false;
29			/** @var string Configure dependency injection class */
30			public $containerConfigurableClassName;
31			public $composerParameters;
32
33			public function __construct($name, $path, $classes)
34			{
35			$this->name = $name;
36			$this->path = $path;
37			$this->classes = $classes;
38
39			// $this->requireClasses();
40			// $this->checkIsContainerConfigurable();
41			}
42
43			/**
44			* Require module classes
45			*/
46			protected function requireClasses()
47			{
48			foreach ($this->classes as $classPath => $className) {
			0 ignored issues – show Coding Style introduced 2016-10-04 09:30 UTC by Report Bug Copy Issue Report Blank line found at start of control structure Loading history...
49
50			$className = '\\' . ltrim($className, '\\');
51			// if (preg_match('/dbQuery/', $className)) {
52			// trace('sdf');
53			// }
54			if (file_exists($classPath)
55			&& !preg_match('/\/api\/generated\//', $className)
56			&& !preg_match('/activerecord\/dbQuery/', $className)
57			&& !preg_match('/samson\/activerecord\//', $className)
58			&& !preg_match('/Field.php$/', $classPath)
59			&& !preg_match('/Navigation.php$/', $classPath)
60			&& !preg_match('/TableVirtualCollection.php$/', $classPath)
61			&& !preg_match('/TableVirtualEntity.php$/', $classPath)
62			&& !preg_match('/TableVirtualQuery.php$/', $classPath)
63			&& !class_exists($className)
64			) {
65			require_once($classPath);
66			}
67			}
68			}
69
70			/**
71			* Check if class configures the container
72			*/
73			public function checkIsContainerConfigurable()
74			{
75			// Find class with correct interface
76			foreach ($this->classes as $classPath => $className) {
77			$reflectionClass = new \ReflectionClass($className);
78			// If class implements correct interface then it is container configurable
79			if (in_array($this::$containerConfigurableInterface, $reflectionClass->getInterfaceNames(), true)) {
80			$this->isContainerConfigurable = true;
81			$this->containerConfigurableClassName = $className;
82			}
83			}
84			}
85			}
86

samsonos / php_core

Push — master ( 14f560...15f305 )

Module::requireClasses() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like