Redbox\Hooks\Annotations\Filter

Complexity

Total Complexity

8

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

87.5%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	29	6
A	getPropertyName()	0	4	1
A	getDataType()	0	4	1

<?php
namespace Redbox\Hooks\Annotations;

/**
 * @Annotation
 */
class Filter
{
    private $propertyName;
    private $dataType = 'string';
    private $options = [];
    public $priority = 0;

    public function __construct($options = [])
    {
        if (isset($options['value'])) {
            $options['propertyName'] = $options['value'];
            unset($options['value']);
        }

        $this->priority = 10;

        $default = [
            'priority' => 10,
        ];

        foreach ($default as $key => $value) {
            if (isset($options[$key]) == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

                $options[$key] = $value;
            }
        }

        foreach ($options as $key => $value) {
            if (!property_exists($this, $key)) {
                throw new \InvalidArgumentException(sprintf('Property "%s" does not exist', $key));
            }

            $this->$key = $value;
        }

        $this->options = $options;
    }

    public function getPropertyName()
    {
        return $this->propertyName;
    }

    public function getDataType()
    {
        return $this->dataType;
    }
}