Artifacts::parse() - Code Metrics - Inspection of "prepare release" - ktomk/pipelines - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 0cfc7e...b88fc1 )

by Tom

created 2018-01-29 07:32 UTC

Artifacts::parse() B

↳ Parent: Artifacts

Complexity

Conditions	5
Paths	6

Size

Total Lines	17
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	5

Importance

Changes

Metric	Value
cc	5
eloc	8
nc	6
nop	1
dl	0
loc	17
ccs	9
cts	9
cp	1
crap	5
rs	8.8571
c	0
b	0
f	0

<?php

/* this file is part of pipelines */

namespace Ktomk\Pipelines\File;

/**
 * Artifacts of a pipelines step
 *
 * @package Ktomk\Pipelines\File
 */
class Artifacts
{
    /**
     * @var array|string[]
     */
    private $artifacts;

    /**
     * Artifacts constructor.
     *
     * @param array|string[] $artifacts
     */
    public function __construct($artifacts)
    {
        $this->parse($artifacts);
    }

    /**
     * @return array|string[]
     */
    public function getPatterns()
    {
        return $this->artifacts;
    }

    /**
     * @param array|string[] $artifacts
     */
    private function parse($artifacts)
    {
        // quick validation: requires a list of strings
        if (!is_array($artifacts) || !count($artifacts)) {
            ParseException::__("'artifacts' requires a list");
        }

        foreach ($artifacts as $index => $string) {
            if (!is_string($string)) {
                ParseException::__(sprintf(
                    "'artifacts' requires a list of strings, #%d is not a string",
                    $index
                ));
            }
        }

        $this->artifacts = $artifacts;
    }
}


1		<?php
2
3		/* this file is part of pipelines */
4
5		namespace Ktomk\Pipelines\File;
6
7		/**
8		* Artifacts of a pipelines step
9		*
10		* @package Ktomk\Pipelines\File
11		*/
12		class Artifacts
13		{
14		/**
15		* @var array\|string[]
16		*/
17		private $artifacts;
18
19		/**
20		* Artifacts constructor.
21		*
22		* @param array\|string[] $artifacts
23		*/
24	4	public function __construct($artifacts)
25		{
26	4	$this->parse($artifacts);
27	2	}
28
29		/**
30		* @return array\|string[]
31		*/
32	1	public function getPatterns()
33		{
34	1	return $this->artifacts;
35		}
36
37		/**
38		* @param array\|string[] $artifacts
39		*/
40	4	private function parse($artifacts)
41		{
42		// quick validation: requires a list of strings
43	4	if (!is_array($artifacts) \|\| !count($artifacts)) {
44	1	ParseException::__("'artifacts' requires a list");
45		}
46
47	3	foreach ($artifacts as $index => $string) {
48	3	if (!is_string($string)) {
49	1	ParseException::__(sprintf(
50	1	"'artifacts' requires a list of strings, #%d is not a string",
51	3	$index
52		));
53		}
54		}
55
56	2	$this->artifacts = $artifacts;
57	2	}
58		}
59

ktomk / pipelines

Push — master ( 0cfc7e...b88fc1 )

Artifacts::parse() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like