BretRZaun\StatusPage\StatusChecker

Complexity

Total Complexity

9

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	addCheck()	0	8	2
A	addGroup()	0	4	1
A	check()	0	6	2
A	getResults()	0	4	1
A	hasErrors()	0	11	3

<?php
namespace BretRZaun\StatusPage;

use BretRZaun\StatusPage\Check\AbstractCheck;

class StatusChecker implements StatusCheckerInterface
{
    /**
     * registered ungrouped checks
     * @var StatusCheckerGroup[]
     */
    protected $ungroupedChecks;

    /**
     * @var StatusCheckerGroup[]
     */
    protected $results = array();

    public function addCheck(AbstractCheck $checker)
    {
        if (!$this->ungroupedChecks) {

The expression $this->ungroupedChecks of type BretRZaun\StatusPage\StatusCheckerGroup[] 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.

            $this->ungroupedChecks = new StatusCheckerGroup('');

It seems like new \BretRZaun\StatusPage\StatusCheckerGroup('') of type object<BretRZaun\StatusPage\StatusCheckerGroup> is incompatible with the declared type array<integer,object<BretRZaun\StatusPage\StatusCheckerGroup>> of property $ungroupedChecks.

            $this->results[] = $this->ungroupedChecks;
        }
        $this->ungroupedChecks->addCheck($checker);
    }

    public function addGroup(StatusCheckerGroup $group)
    {
        $this->results[] = $group;
    }

    public function check()
    {
        foreach($this->results as $group) {
            $group->check();
        }
    }

    public function getResults(): array
    {
        return $this->results;
    }

    public function hasErrors(): bool
    {
        $error = false;
        foreach($this->results as $group) {
            if ($group->hasErrors()) {
                $error = true;
                break;
            }
        }
        return $error;
    }
}