Leankoala\HealthFoundation\Filter\Time\DailyFilter::run()

Complexity

Conditions	5
Paths	6

Size

Total Lines	27
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function run()
    {
        $result = $this->getCheck()->run();

        $currentHour = (int)date('H');

        if ($result->getStatus() == Result::STATUS_FAIL) {
            if ($currentHour == $this->hour) {
                if ($this->reason) {
                    $message = $this->reason . ' (fail reason was: ' . $result->getMessage() . ')';

Are you sure $this->reason of type true can be used in concatenation?

                } else {
                    $message = 'Passed due daily filter (fail reason was: ' . $result->getMessage() . ')';
                }

                if ($result instanceof MetricAwareResult) {
                    $newResult = new MetricAwareResult(Result::STATUS_PASS, $message);
                    $newResult->setMetric($result->getMetricValue(), $result->getMetricUnit());
                } else {
                    $newResult = new Result(Result::STATUS_PASS, $message);
                }

                return $newResult;
            } else {
                return $result;
            }
        } else {
            return $result;
        }