Sensorario\Resources\Validators\Validators\AllowedProperties

Complexity

Total Complexity

12

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	buildAllowedProperties()	0	7	1
A	check()	0	8	2
A	ensurePropertyIsAllowed()	0	10	3
A	checkShouldBeSkipped()	0	8	3
A	mustSkip()	0	6	3

<?php

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

namespace Sensorario\Resources\Validators\Validators;

use Sensorario\Resources\Resource;
use Sensorario\Resources\Validators\Interfaces\Validator;

final class AllowedProperties implements Validator
{
    private $resource;

    private $allowed;

    public function check(Resource $resource)
    {
        $this->resource = $resource;
        $this->buildAllowedProperties();
        if (!$this->checkShouldBeSkipped()) {

The expression $this->checkShouldBeSkipped() of type boolean|null is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

            $this->ensurePropertyIsAllowed();
        }
    }

    private function buildAllowedProperties()
    {
        $this->allowed = array_merge(
            $this->resource->allowed(),
            $this->resource->mandatory()
        );
    }

    private function checkShouldBeSkipped()
    {
        foreach ($this->allowed as $kk => $vv) {
            if ($this->mustSkip($kk, $vv)) {
                return true;
            }
        }
    }

    public function ensurePropertyIsAllowed()
    {
        foreach ($this->resource->properties() as $key => $value) {
            if (!in_array($key, $this->allowed)) {
                throw new \Sensorario\Resources\Exceptions\NotAllowedKeyException(
                    "Key `" . get_class($this->resource) . "::\$$key` with value `" . $value . "` is not allowed"
                );
            }
        }
    }

    public function mustSkip($key, $value)
    {
        return !is_numeric($key) &&
            isset($value['when']) &&
            $this->resource->hasProperty($value['when']['property']);
    }
}