Subset::getOptions() - Code Metrics - Inspection of "Fix #117 issue, InRange SRP violation" - yiisoft/validator - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#119)

unknown

created 2021-03-01 18:10 UTC

Subset::getOptions() A

↳ Parent: Subset

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	6
c	1
b	0
f	0
nc	1
nop	0
dl	0
loc	9
ccs	0
cts	7
cp	0
crap	2
rs	10

<?php

declare(strict_types=1);

namespace Yiisoft\Validator\Rule;

use Yiisoft\Arrays\ArrayHelper;
use Yiisoft\Validator\Result;
use Yiisoft\Validator\Rule;
use Yiisoft\Validator\ValidationContext;

class Subset extends Rule
{
    /**
     * @var iterable
     */
    private iterable $range;
    /**
     * @var bool whether the comparison is strict (both type and value must be the same)
     */
    private bool $strict = false;

    private string $iterableMessage = 'Value must be iterable';

    private string $subsetMessage = 'Value must be subset of...';

    public function __construct(iterable $range)
    {
        $this->range = $range;
    }

    protected function validateValue($value, ValidationContext $context = null): Result
    {
        $result = new Result();

        if (!is_iterable($value)) {
            $result->addError($this->formatMessage($this->iterableMessage));
            return $result;
        }

        if (!ArrayHelper::isSubset($value, $this->range, $this->strict)) {
            $result->addError($this->formatMessage($this->subsetMessage));
        }

        return $result;
    }

    public function strict(): self
    {
        $new = clone $this;
        $new->strict = true;
        return $new;
    }

    public function getOptions(): array
    {
        return array_merge(
            parent::getOptions(),
            [
                'iterableMessage' => $this->formatMessage($this->iterableMessage),
                'subsetMessage' => $this->formatMessage($this->subsetMessage),
                'range' => $this->range,
                'strict' => $this->strict,
            ],
        );
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Yiisoft\Validator\Rule;
6
7		use Yiisoft\Arrays\ArrayHelper;
8		use Yiisoft\Validator\Result;
9		use Yiisoft\Validator\Rule;
10		use Yiisoft\Validator\ValidationContext;
11
12		class Subset extends Rule
13		{
14		/**
15		* @var iterable
16		*/
17		private iterable $range;
18		/**
19		* @var bool whether the comparison is strict (both type and value must be the same)
20		*/
21		private bool $strict = false;
22
23		private string $iterableMessage = 'Value must be iterable';
24
25		private string $subsetMessage = 'Value must be subset of...';
26
27	4	public function __construct(iterable $range)
28		{
29	4	$this->range = $range;
30	4	}
31
32	3	protected function validateValue($value, ValidationContext $context = null): Result
33		{
34	3	$result = new Result();
35
36	3	if (!is_iterable($value)) {
37		$result->addError($this->formatMessage($this->iterableMessage));
38		return $result;
39		}
40
41	3	if (!ArrayHelper::isSubset($value, $this->range, $this->strict)) {
42	1	$result->addError($this->formatMessage($this->subsetMessage));
43		}
44
45	3	return $result;
46		}
47
48		public function strict(): self
49		{
50		$new = clone $this;
51		$new->strict = true;
52		return $new;
53		}
54
55		public function getOptions(): array
56		{
57		return array_merge(
58		parent::getOptions(),
59		[
60		'iterableMessage' => $this->formatMessage($this->iterableMessage),
61		'subsetMessage' => $this->formatMessage($this->subsetMessage),
62		'range' => $this->range,
63		'strict' => $this->strict,
64		],
65		);
66		}
67		}
68

yiisoft / validator

Pull Request — master (#119)

Subset::getOptions() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like