Thinktomorrow\Chief\HealthMonitor\Monitor::check()

Complexity

Conditions	3
Paths	3

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public static function check()
    {
        foreach(static::$checks as $check => $notifier)

Since $checks is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $checks to at least protected.

        {
            $checker = (new $check);
            $notifier = (new $notifier);
            
            if(!$checker->check())
            {
                return $notifier->notify($checker->notify());
            }