Psi\Component\Description\ValidatedDescription

Complexity

Total Complexity

5

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	get()	0	6	1
A	has()	0	6	1
A	all()	0	4	1
A	set()	0	5	1

<?php

declare(strict_types=1);

namespace Psi\Component\Description;

use Psi\Component\Description\Schema\Schema;

/**
 * Decorator for description which validates the descriptions based
 * on the given Schema.
 *
 * Note that for performance reasons this should only be used in a development
 * environemnt.
 */
class ValidatedDescription implements DescriptionInterface
{
    /**
     * @var Description
     */
    private $description;

    /**
     * @var Scehma
     */
    private $schema;

    public function __construct(DescriptionInterface $description, Schema $schema)
    {
        $this->description = $description;

$description is of type object<Psi\Component\Description\DescriptionInterface>, but the property $description was declared to be of type object<Psi\Component\Description\Description>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->schema = $schema;

It seems like $schema of type object<Psi\Component\Description\Schema\Schema> is incompatible with the declared type object<Psi\Component\Description\Scehma> of property $schema.

    }

    /**
     * {@inheritdoc}
     */
    public function get($descriptorKey): DescriptorInterface
    {
        $this->schema->validateKey($descriptorKey);

        return $this->description->get($descriptorKey);
    }

    /**
     * {@inheritdoc}
     */
    public function has($descriptor): bool
    {
        $this->schema->validateKey($descriptor);

        return $this->description->has($descriptor);
    }

    /**
     * {@inheritdoc}
     */
    public function all(): array
    {
        return $this->description->all();
    }

    /**
     * {@inheritdoc}
     */
    public function set(string $key, DescriptorInterface $descriptor, int $priority = 0)
    {
        $this->schema->validate($key, $descriptor);
        $this->description->set($key, $descriptor, $priority);
    }
}