AccessRestrictedElementTrait::setAccess() - Code Metrics - Inspection of "Merge branch 'feature/issue-11' into develop" - WsdlToPhp/PhpGenerator - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( 6b3bb3...b964e0 )

by Mikaël

created 2021-02-04 14:45 UTC

AccessRestrictedElementTrait::setAccess() A

↳ Parent: AccessRestrictedElementTrait

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	2
eloc	4
nc	2
nop	1
dl	0
loc	8
ccs	5
cts	5
cp	1
crap	2
rs	10
c	1
b	0
f	1

<?php

declare(strict_types=1);

namespace WsdlToPhp\PhpGenerator\Element;

use InvalidArgumentException;

trait AccessRestrictedElementTrait
{
    protected string $access;

    public function setAccess(?string $access): AbstractElement
    {
        if (!static::accessIsValid($access)) {
            throw new InvalidArgumentException(sprintf('Access "%s" is invalid, please provide one of these accesses: %s', $access, implode(', ', AccessRestrictedElementInterface::ACCESSES)));
        }
        $this->access = $access;

        return $this;

    }

    public function getAccess(): string
    {
        return $this->access;
    }

    public static function accessIsValid(?string $access): bool
    {
        return '' === $access || in_array($access, AccessRestrictedElementInterface::ACCESSES, true);
    }

    protected function getPhpAccess(): string
    {
        return '' === $this->getAccess() ? '' : sprintf('%s ', $this->getAccess());
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace WsdlToPhp\PhpGenerator\Element;
6
7		use InvalidArgumentException;
8
9		trait AccessRestrictedElementTrait
10		{
11		protected string $access;
12
13	110	public function setAccess(?string $access): AbstractElement
14		{
15	110	if (!static::accessIsValid($access)) {
16	2	throw new InvalidArgumentException(sprintf('Access "%s" is invalid, please provide one of these accesses: %s', $access, implode(', ', AccessRestrictedElementInterface::ACCESSES)));
17		}
18	110	$this->access = $access;
19
20	110	return $this;
		0 ignored issues – show Bug Best Practice introduced 2021-02-03 23:14 UTC by Report Bug Copy Issue Report The expression `return $this` returns the type `WsdlToPhp\PhpGenerator\E...sRestrictedElementTrait` which is incompatible with the type-hinted return `WsdlToPhp\PhpGenerator\Element\AbstractElement`. Loading history...
21		}
22
23	82	public function getAccess(): string
24		{
25	82	return $this->access;
26		}
27
28	110	public static function accessIsValid(?string $access): bool
29		{
30	110	return '' === $access \|\| in_array($access, AccessRestrictedElementInterface::ACCESSES, true);
31		}
32
33	82	protected function getPhpAccess(): string
34		{
35	82	return '' === $this->getAccess() ? '' : sprintf('%s ', $this->getAccess());
36		}
37		}
38

WsdlToPhp / PhpGenerator

Push — develop ( 6b3bb3...b964e0 )

AccessRestrictedElementTrait::setAccess() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like