MigrantUtil::getDependencies() - Code Metrics - Inspection of "Refactor migrant to work with classnames as well a..." - delboy1978uk/common - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 931184...014faa )

by Derek Stephen

created 2019-09-15 12:35 UTC

MigrantUtil::getDependencies() B

↳ Parent: MigrantUtil

Complexity

Conditions	10
Paths	17

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	37.0965

Importance

Changes

Metric	Value
dl	0
loc	33
ccs	6
cts	17
cp	0.3529
rs	7.6666
c	0
b	0
f	0
cc	10
nc	17
nop	1
crap	37.0965

How to fix Complexity

<?php

namespace Del\Common\Util;

use Exception;
use ReflectionClass;

/**
 * User: delboy1978uk
 * Date: 15/10/2016
 * Time: 16:18
 */
class MigrantUtil
{
    /**
     * @var array
     */
    private $mergedPackages;

    public function __construct()
    {
        $this->setMergedPackages([]);
    }

    /**
     * @return array
     */
    private function getMergedPackages()
    {
        return $this->mergedPackages;
    }

    /**
     * @param array $mergedPackages
     * @return MigrantUtil
     */
    private function setMergedPackages($mergedPackages)
    {
        $this->mergedPackages = $mergedPackages;
        return $this;
    }

    /**
     * @param array $packages
     * @return array
     */
    public function processDependencies(array $packages): array
    {
        foreach ($packages as $package) {
            $this->processPackage($package);
        }

        return $this->getMergedPackages();
    }

    /**
     * @param $package
     */
    private function processPackage($package)
    {
        $mergedPackages = $this->getMergedPackages();
        if (!in_array($package, $mergedPackages)) {
            $mergedPackages[] = $package;
            $this->setMergedPackages($mergedPackages);
            $packages = $this->getDependencies($package);
            if (count($packages) > 0) {
                $this->processDependencies($packages);
            }
        }
    }

    /**
     * @param $package
     * @return array
     */
    private function getDependencies(string $package): array
    {
        $srcFolder = 'vendor' . DIRECTORY_SEPARATOR . $package . DIRECTORY_SEPARATOR;

        if (file_exists($srcFolder . '.migrant')) {
            $depend = require($srcFolder . '.migrant');

            if (file_exists('.migrant_local')) {
                $depend = array_merge($depend, require_once '.migrant_local');
            }

            return isset($depend['packages']) ? $depend['packages'] : [];
        }

        try {
            if (class_exists($package)) {
                $mirror = new ReflectionClass($package);
                $location = $mirror->getFileName();
                if (false !== strpos($location, 'vendor') && preg_match('#(?<packagePath>.+)/src/.+\.php#', $location, $match)) {
                    $path = $match['packagePath'] . DIRECTORY_SEPARATOR . '.migrant';
                    if (file_exists($path)) {
                        $depend = require_once $path;

                        return isset($depend['packages']) ? $depend['packages'] : [];
                    }
                }
            }
        } catch (Exception $e) {


        }

        return [];
    }
}

1		<?php
2
3		namespace Del\Common\Util;
4
5		use Exception;
6		use ReflectionClass;
7
8		/**
9		* User: delboy1978uk
10		* Date: 15/10/2016
11		* Time: 16:18
12		*/
13		class MigrantUtil
14		{
15		/**
16		* @var array
17		*/
18		private $mergedPackages;
19
20	2	public function __construct()
21		{
22	2	$this->setMergedPackages([]);
23	2	}
24
25		/**
26		* @return array
27		*/
28	2	private function getMergedPackages()
29		{
30	2	return $this->mergedPackages;
31		}
32
33		/**
34		* @param array $mergedPackages
35		* @return MigrantUtil
36		*/
37	2	private function setMergedPackages($mergedPackages)
38		{
39	2	$this->mergedPackages = $mergedPackages;
40	2	return $this;
41		}
42
43		/**
44		* @param array $packages
45		* @return array
46		*/
47	2	public function processDependencies(array $packages): array
48		{
49	2	foreach ($packages as $package) {
50	2	$this->processPackage($package);
51		}
52
53	2	return $this->getMergedPackages();
54		}
55
56		/**
57		* @param $package
58		*/
59	2	private function processPackage($package)
60		{
61	2	$mergedPackages = $this->getMergedPackages();
62	2	if (!in_array($package, $mergedPackages)) {
63	2	$mergedPackages[] = $package;
64	2	$this->setMergedPackages($mergedPackages);
65	2	$packages = $this->getDependencies($package);
66	2	if (count($packages) > 0) {
67		$this->processDependencies($packages);
68		}
69		}
70	2	}
71
72		/**
73		* @param $package
74		* @return array
75		*/
76	2	private function getDependencies(string $package): array
77		{
78	2	$srcFolder = 'vendor' . DIRECTORY_SEPARATOR . $package . DIRECTORY_SEPARATOR;
79
80	2	if (file_exists($srcFolder . '.migrant')) {
81		$depend = require($srcFolder . '.migrant');
82
83		if (file_exists('.migrant_local')) {
84		$depend = array_merge($depend, require_once '.migrant_local');
85		}
86
87		return isset($depend['packages']) ? $depend['packages'] : [];
88		}
89
90		try {
91	2	if (class_exists($package)) {
92		$mirror = new ReflectionClass($package);
93		$location = $mirror->getFileName();
94		if (false !== strpos($location, 'vendor') && preg_match('#(?<packagePath>.+)/src/.+\.php#', $location, $match)) {
95		$path = $match['packagePath'] . DIRECTORY_SEPARATOR . '.migrant';
96		if (file_exists($path)) {
97		$depend = require_once $path;
98
99	2	return isset($depend['packages']) ? $depend['packages'] : [];
100		}
101		}
102		}
103		} catch (Exception $e) {
		0 ignored issues – show Coding Style Comprehensibility introduced 2019-09-15 12:37 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
104
105		}
106
107	2	return [];
108		}
109		}

delboy1978uk / common

Push — master ( 931184...014faa )

MigrantUtil::getDependencies() B

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like