Freshcells\StateInspector\StateInspector

Complexity

Total Complexity

14

Size/Duplication

Total Lines	101
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

83.33%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	3
A	inspect()	0	8	2
B	inspection()	0	31	6
A	getIssues()	0	4	1
A	addInspection()	0	9	2

<?php

namespace Freshcells\StateInspector;

use Freshcells\StateInspector\Exception\StateInspectorException;
use Freshcells\StateInspector\Inspection\InspectionInterface;
use Freshcells\StateInspector\Issue\Issue;

class StateInspector implements StateInspectorInterface
{
    /**
     * @var InspectionInterface[]
     */
    private $inspections = [];

    /**
     * @var IssueInterface[]
     */
    private $issues = [];

    /**
     * StateInspectorManager constructor.
     * @param InspectorInterface[] $inspections
     */
    public function __construct(array $inspections = [])
    {
        foreach ($inspections as $name => $inspection) {
            if(is_numeric($name)){
                $name = null;
            }
            $this->addInspection($inspection, $name);
        }
    }

    /**
     * @param mixed $object
     * @param bool $bubble
     */
    final public function inspect($object, $bubble = false)
    {
        $this->issues = []; //reset
        foreach ($this->inspections as $name => $inspection) {
            $issues       = $this->inspection($name, $object, $bubble);
            $this->issues = array_merge($this->issues, $issues);

It seems like array_merge($this->issues, $issues) of type array is incompatible with the declared type array<integer,object<Freshcells\StateInspector\IssueInterface>> of property $issues.

        }
    }

    /**
     * @param string $name
     * @param mixed $object
     * @param bool $bubble
     * @return InspectionInterface[]
     * @throws \Exception
     */
    final public function inspection(string $name, $object, $bubble = false): array
    {
        if (isset($this->inspections[$name]) === false) {
            $msg = $name.' no such inspection!';
            throw new StateInspectorException(
                new Issue($msg, $name.' is no inspection.', 'Please create new Inspection or use other name.'),
                $msg
            );
        }
        $inspection = $this->inspections[$name];

        if ($inspection->supports($object) === false) {
            $msg = $name.' inspection doesnt support object type!';
            throw new StateInspectorException(
                new Issue($msg, get_class($object).' is not supported.', 'Please create new Inspection for this type.'),
                $msg
            );
        }

        try {
            $inspection->inspect($object)
                ? $inspection->success()
                : $inspection->failure();
        } catch (\Exception $e) {
            if ($bubble) {
                throw $e;
            }
        }

        return $inspection->getIssues();
    }

    /**
     * @return IssueInterface[]
     */
    final public function getIssues(): array
    {
        return $this->issues;
    }

    /**
     * @param InspectionInterface $inspection
     * @param string|null $name
     * @return StateInspectorInterface
     */
    final public function addInspection(InspectionInterface $inspection, string $name = null): StateInspectorInterface
    {
        if(null === $name){
            $name = get_class($inspection);
        }
        $this->inspections[$name] = $inspection;

        return $this;
    }
}