PackageYamlFileReader::__construct() - Code Metrics - Inspection of "... fix builder ** limit expansion" - ktomk/pipelines - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — test ( 0c4d14...2be48d )

by Tom

created 2019-12-10 00:47 UTC

PackageYamlFileReader::__construct() A

↳ Parent: PackageYamlFileReader

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	1
eloc	1
c	1
b	0
f	1
nc	1
nop	1
dl	0
loc	3
rs	10

<?php

/* this file is part of pipelines */

namespace Ktomk\Pipelines\Runner\Docker\Binary;

use Ktomk\Pipelines\LibFs;
use Ktomk\Pipelines\Yaml\Yaml;

/**
 * Class BinaryPackageYmlReader
 *
 * Read binary package information from a YAML file
 */
class PackageYamlFileReader implements PackageInterface
{
    /**
     * @var string
     */
    private $file;

    /**
     * BinaryPackageYmlReader constructor.
     *
     * @param string $file
     */
    public function __construct($file)
    {
        $this->file = LibFs::normalizePathSegments($file);
    }

    /**
     * @return array
     */
    public function asPackageArray()
    {
        $package = Yaml::file($this->file);

        $this->resolveUri($package['uri']);

        return $package;
    }

    /**
     * resolve local path URI relative to document location
     *
     * @param string $uri
     */
    private function resolveUri(&$uri)
    {
        if (null === $uri) {

            return;
        }

        // http/s is the only remote transport considered to be remote so far
        // as it is the only one used
        if (1 === preg_match('(^https?://)i', $uri)) {
            return;
        }

        if (LibFs::isAbsolutePath($uri)) {
            return;
        }

        $baseDir = dirname($this->file);
        $buffer = $baseDir . '/' . $uri;
        $uri = LibFs::normalizePathSegments($buffer);
    }
}


1			<?php
2
3			/* this file is part of pipelines */
4
5			namespace Ktomk\Pipelines\Runner\Docker\Binary;
6
7			use Ktomk\Pipelines\LibFs;
8			use Ktomk\Pipelines\Yaml\Yaml;
9
10			/**
11			* Class BinaryPackageYmlReader
12			*
13			* Read binary package information from a YAML file
14			*/
15			class PackageYamlFileReader implements PackageInterface
16			{
17			/**
18			* @var string
19			*/
20			private $file;
21
22			/**
23			* BinaryPackageYmlReader constructor.
24			*
25			* @param string $file
26			*/
27			public function __construct($file)
28			{
29			$this->file = LibFs::normalizePathSegments($file);
30			}
31
32			/**
33			* @return array
34			*/
35			public function asPackageArray()
36			{
37			$package = Yaml::file($this->file);
38
39			$this->resolveUri($package['uri']);
40
41			return $package;
42			}
43
44			/**
45			* resolve local path URI relative to document location
46			*
47			* @param string $uri
48			*/
49			private function resolveUri(&$uri)
50			{
51			if (null === $uri) {
			0 ignored issues – show introduced 2019-12-09 01:12 UTC by Report Bug Copy Issue Report The condition `null === $uri` is always `false`. Loading history...
52			return;
53			}
54
55			// http/s is the only remote transport considered to be remote so far
56			// as it is the only one used
57			if (1 === preg_match('(^https?://)i', $uri)) {
58			return;
59			}
60
61			if (LibFs::isAbsolutePath($uri)) {
62			return;
63			}
64
65			$baseDir = dirname($this->file);
66			$buffer = $baseDir . '/' . $uri;
67			$uri = LibFs::normalizePathSegments($buffer);
68			}
69			}
70

ktomk / pipelines

Push — test ( 0c4d14...2be48d )

PackageYamlFileReader::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like