Percy\Store\AbstractStore

Complexity

Total Complexity

5

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	4
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	validate()	0	18	4

<?php

namespace Percy\Store;

use Aura\Filter\Exception\FilterFailed;
use Aura\Filter\FilterFactory;
use InvalidArgumentException;
use Percy\Decorator\DecoratorTrait;
use Percy\Entity\Collection;

abstract class AbstractStore implements StoreInterface
{
    use DecoratorTrait;

    /**
     * Construct.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(FilterFactory $filter)
    {
        $this->filter = $filter;

The property filter does not exist. Did you maybe forget to declare it?

    }

    /**
     * Iterate collection and validate data.
     *
     * @param \Percy\Entity\Collection $collection
     *
     * @throws \Percy\Exception\ValidationException when first validation failure occurs
     *
     * @return boolean
     */
    public function validate(Collection $collection)
    {
        foreach ($collection->getIterator() as $entity) {
            if (is_null($collection->getValidator())) {

The method getValidator() does not seem to exist on object<Percy\Entity\Collection>.

                continue;
            }

            $filter = $this->filter->newSubjectFilter($entity->getValidator());

            try {
                $filter($entity->toArray());
            } catch (FilterFailed $e) {

The class Aura\Filter\Exception\FilterFailed does not exist. Did you forget a USE statement, or did you not list all dependencies?

                throw new ValidationException($e->getMessage());
            }
        }

        return true;
    }
}