MagicResource::get() - Code Metrics - Inspection of "Merge pull request #53 from sensorario/master-has-..." - sensorario/resources - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 63e254...443ded )

by Simone

created 2016-07-09 10:09 UTC

MagicResource::get() B

↳ Parent: Project

Complexity

Conditions	4
Paths	4

Size

Total Lines	22
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	22
rs	8.9197
cc	4
eloc	12
nc	4
nop	1

<?php

/**
 * This file is part of sensorario/resources repository
 *
 * (c) Simone Gentili <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sensorario\Resources;

use RuntimeException;
use Sensorario\Resources\Validators\ResourcesValidator;

abstract class MagicResource
{
    protected $properties = [];

    public function __call($functionName, $arguments)
    {
        $propertyName = strtolower($functionName);

        if ($this->hasProperty($propertyName)) {
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            return $this->get($propertyName);
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        }

        if (isset($this->defaults()[$propertyName])) {
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            return $this->defaults()[$propertyName];
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        }

        throw new RuntimeException(
            'Method `' . get_class($this)
            . '::' . $functionName 
            . '()` is not yet implemented'
        );
    }

    protected function __construct(
        array $properties,
        ResourcesValidator $validator
    ) {
        $this->properties = $properties;

        foreach ($properties as $k => $v) {
            if ('object' === gettype($v) && !isset($this->rules()[$k])) {
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
                throw new RuntimeException(
                    'When property `' . $k . '` is an object class, must be defined in Resources::rules()'
                );
            }
        }

        foreach ($properties as $name => $value) {
            if ('object' !== gettype($value)) {
                $properties[$name] = utf8_encode(
                    $value
                );
            }
        }

        $validator->validate($this);

    }

    public static function __callStatic($methodName, array $args)
    {
        $methodWhiteList = [
            'box',
            'allowedValues'
        ];

        $isMethodAllowed = in_array(
            $methodName,
            $methodWhiteList
        );

        if ($isMethodAllowed) {
            return new static(
                isset($args[0])
                ? $args[0]
                : [],
                new ResourcesValidator()
            );
        }

        throw new RuntimeException(
            'Invalid factory method `' . $methodName . '`'
        );
    }
}


1			<?php
2
3			/**
4			* This file is part of sensorario/resources repository
5			*
6			* (c) Simone Gentili <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Sensorario\Resources;
13
14			use RuntimeException;
15			use Sensorario\Resources\Validators\ResourcesValidator;
16
17			abstract class MagicResource
18			{
19			protected $properties = [];
20
21			public function __call($functionName, $arguments)
22			{
23			$propertyName = strtolower($functionName);
24
25			if ($this->hasProperty($propertyName)) {
			0 ignored issues – show Documentation Bug introduced 2016-07-09 09:54 UTC by Report Bug Copy Issue Report The method `hasProperty` does not exist on `object<Sensorario\Resources\MagicResource>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
26			return $this->get($propertyName);
			0 ignored issues – show Documentation Bug introduced 2016-07-09 09:54 UTC by Report Bug Copy Issue Report The method `get` does not exist on `object<Sensorario\Resources\MagicResource>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
27			}
28
29			if (isset($this->defaults()[$propertyName])) {
			0 ignored issues – show Documentation Bug introduced 2016-07-09 09:54 UTC by Report Bug Copy Issue Report The method `defaults` does not exist on `object<Sensorario\Resources\MagicResource>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
30			return $this->defaults()[$propertyName];
			0 ignored issues – show Documentation Bug introduced 2016-07-09 09:54 UTC by Report Bug Copy Issue Report The method `defaults` does not exist on `object<Sensorario\Resources\MagicResource>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
31			}
32
33			throw new RuntimeException(
34			'Method `' . get_class($this)
35			. '::' . $functionName
36			. '()` is not yet implemented'
37			);
38			}
39
40			protected function __construct(
41			array $properties,
42			ResourcesValidator $validator
43			) {
44			$this->properties = $properties;
45
46			foreach ($properties as $k => $v) {
47			if ('object' === gettype($v) && !isset($this->rules()[$k])) {
			0 ignored issues – show Documentation Bug introduced 2016-07-09 09:54 UTC by Report Bug Copy Issue Report The method `rules` does not exist on `object<Sensorario\Resources\MagicResource>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
48			throw new RuntimeException(
49			'When property `' . $k . '` is an object class, must be defined in Resources::rules()'
50			);
51			}
52			}
53
54			foreach ($properties as $name => $value) {
55			if ('object' !== gettype($value)) {
56			$properties[$name] = utf8_encode(
57			$value
58			);
59			}
60			}
61
62			$validator->validate($this);
			1 ignored issue – show Compatibility introduced 2016-07-09 09:54 UTC by Report Bug Copy Issue Report `$this` of type `object<Sensorario\Resources\MagicResource>` is not a sub-type of `object<Sensorario\Resources\Resource>`. It seems like you assume a child class of the class `Sensorario\Resources\MagicResource` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
63			}
64
65			public static function __callStatic($methodName, array $args)
66			{
67			$methodWhiteList = [
68			'box',
69			'allowedValues'
70			];
71
72			$isMethodAllowed = in_array(
73			$methodName,
74			$methodWhiteList
75			);
76
77			if ($isMethodAllowed) {
78			return new static(
79			isset($args[0])
80			? $args[0]
81			: [],
82			new ResourcesValidator()
83			);
84			}
85
86			throw new RuntimeException(
87			'Invalid factory method `' . $methodName . '`'
88			);
89			}
90			}
91

sensorario / resources

Push — master ( 63e254...443ded )

MagicResource::get() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like