SimpleSAML\Module\monitor\TestResult::setOutput()

Complexity

Conditions	5
Paths	4

Size

Total Lines	14
Code Lines	10

Duplication

Lines	14
Ratio	100 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	TestResult::__construct()	0	5	1

<?php

namespace SimpleSAML\Module\monitor;

final class TestResult
{
    /**
     * @var string
     */
    private $category;

    /**
     * @var string
     */
    private $subject;

    /**
     * @var string
     */
    private $message;

The property $message is not used and could be removed.

    /**
     * @var State
     */
    private $state = State::NOSTATE;

    /**
     * @param string $category
     * @param string $subject
     */
    public function __construct($category = 'Unknown category', $subject = 'Unknown subject')
    {
        $this->category = $category;
        $this->subject = $subject;
    }

    /**
     * @param State $state
     *
     * @return void
     */
    public function setState($state = State::NOSTATE)
    {
        assert($state instanceof State);
        $this->$state = $state;
    }

    /**
     * @return State
     */
    public function getState()
    {
        assert($this->state instanceof State);
        return $this->$state;

The variable $state does not exist. Did you forget to declare it?

    }
}