MetadataOption::check() - Code Metrics - Inspection of "Created classes for handling options" - rosell-dk/webp-convert - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 975525...307699 )

by Bjørn

created 2019-05-30 08:24 UTC

MetadataOption::check() A

↳ Parent: MetadataOption

Complexity

Conditions	5
Paths	4

Size

Total Lines	15
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	9
nc	4
nop	0
dl	0
loc	15
rs	9.6111
c	0
b	0
f	0

<?php

namespace WebPConvert\Options;

use WebPConvert\Options\StringOption;
use WebPConvert\Options\Exceptions\InvalidOptionValueException;

/**
 * Metadata option. A Comma-separated list ('all', 'none', 'exif', 'icc', 'xmp')
 *
 * @package    WebPConvert
 * @author     Bjørn Rosell <[email protected]>
 * @since      Class available since Release 2.0.0
 */
class MetadataOption extends StringOption
{

    public function __construct($id, $defaultValue)
    {
        parent::__construct($id, $defaultValue);
    }

    public function check()
    {
        parent::check();

        $value = $this->getValue();

        if (($value == 'all') || ($value == 'none')) {
            return;
        }

        foreach (explode(',', $value) as $item) {
            if (!in_array($value, ['exif', 'icc', 'xmp'])) {
                throw new InvalidOptionValueException(
                    '"metadata" option must be "all", "none" or a comma-separated list of "exif", "icc" or "xmp". ' .
                    'It was however set to: "' . $value . '"'
                );
            }
        }

        //$this->checkType('string');
    }
}


1			<?php
2
3			namespace WebPConvert\Options;
4
5			use WebPConvert\Options\StringOption;
6			use WebPConvert\Options\Exceptions\InvalidOptionValueException;
7
8			/**
9			* Metadata option. A Comma-separated list ('all', 'none', 'exif', 'icc', 'xmp')
10			*
11			* @package WebPConvert
12			* @author Bjørn Rosell <[email protected]>
13			* @since Class available since Release 2.0.0
14			*/
15			class MetadataOption extends StringOption
16			{
17
18			public function __construct($id, $defaultValue)
19			{
20			parent::__construct($id, $defaultValue);
21			}
22
23			public function check()
24			{
25			parent::check();
26
27			$value = $this->getValue();
28
29			if (($value == 'all') \|\| ($value == 'none')) {
30			return;
31			}
32
33			foreach (explode(',', $value) as $item) {
34			if (!in_array($value, ['exif', 'icc', 'xmp'])) {
35			throw new InvalidOptionValueException(
36			'"metadata" option must be "all", "none" or a comma-separated list of "exif", "icc" or "xmp". ' .
37			'It was however set to: "' . $value . '"'
38			);
39			}
40			}
41
42			//$this->checkType('string');
43			}
44			}
45

rosell-dk / webp-convert

Push — master ( 975525...307699 )

MetadataOption::check() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like