Equip\Command\OptionsTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	2
A	ensureRequired()	0	12	2
A	valid()	0	4	1
A	toArray()	0	4	1

<?php

namespace Equip\Command;

use Equip;
use Equip\Data\Traits\ProtectedValueObjectTrait;

trait OptionsTrait /* implements OptionsInterface */
{
    use ProtectedValueObjectTrait;

    /**
     * Check that all required options are defined, then hydrate.
     *
     * @param array $values
     */
    public function __construct(array $values)
    {
        $values = Equip\grab($values, $this->valid());

        $this->ensureRequired($values);

        foreach ($values as $key => $value) {
            $this->$key = $value;
        }
    }

    /**
     * Ensure that all required options are included in the given values.
     *
     * @param array $values
     *
     * @return void
     *
     * @throws CommandException
     *  If any required options have not been defined.
     */
    private function ensureRequired(array $values)
    {
        $defined = array_filter($values, static function ($value) {
            return $value !== null;
        });

        $missing = array_diff($this->required(), array_keys($defined));

The method required() does not exist on Equip\Command\OptionsTrait. Did you maybe mean ensureRequired()?

        if ($missing) {

The expression $missing of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            throw CommandException::missingOptions($missing);
        }
    }

    /**
     * @inheritdoc
     */
    public function valid()
    {
        return array_keys($this->toArray());
    }

    /**
     * @inheritdoc
     */
    public function toArray()
    {
        return get_object_vars($this);
    }
}