Tvi\MonitorBundle\Check\php\Expression\Check

Complexity

Total Complexity

9

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Test Coverage

Coverage

90.91%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	3
A	getExpressionLanguage()	0	10	1
A	check()	0	13	5

<?php

/**
 * This file is part of the `tvi/monitor-bundle` project.
 *
 * (c) https://github.com/turnaev/monitor-bundle/graphs/contributors
 *
 * For the full copyright and license information, please view the LICENSE.md
 * file that was distributed with this source code.
 */

PHP version not specified

Missing @category tag in file comment

Missing @package tag in file comment

Missing @author tag in file comment

Missing @license tag in file comment

Missing @link tag in file comment

namespace Tvi\MonitorBundle\Check\php\Expression;

use Symfony\Component\ExpressionLanguage\Expression;
use Symfony\Component\ExpressionLanguage\ExpressionLanguage;
use ZendDiagnostics\Result\Success;
use ZendDiagnostics\Result\Warning;
use ZendDiagnostics\Result\Failure;

use Tvi\MonitorBundle\Check\CheckInterface;
use Tvi\MonitorBundle\Check\CheckTrait;

/**

Missing short description in doc comment

 * @author Vladimir Turnaev <turnaev@gmail.com>
 */

Missing @category tag in class comment

Missing @package tag in class comment

Missing @license tag in class comment

Missing @link tag in class comment

class Check extends \ZendDiagnostics\Check\AbstractCheck implements CheckInterface
{
    use CheckTrait;

    /**

Missing short description in doc comment

     * @var Expression|string
     */
    private $warningExpression;

Private member variable "warningExpression" must be prefixed with an underscore

    /**

Missing short description in doc comment

     * @var Expression|string
     */
    private $criticalExpression;

Private member variable "criticalExpression" must be prefixed with an underscore

    /**

Missing short description in doc comment

     * @var ?string
     */
    private $warningMessage;

Private member variable "warningMessage" must be prefixed with an underscore

    /**

Missing short description in doc comment

     * @var ?string
     */
    private $criticalMessage;

Private member variable "criticalMessage" must be prefixed with an underscore

    /**

Missing short description in doc comment

     * @param Expression|string $warningExpression

Missing parameter comment

     * @param Expression|string $criticalExpression

Missing parameter comment

     * @param ?string           $warningMessage

Missing parameter comment

     * @param ?string           $criticalMessage

Missing parameter comment

     *
     * @throws \Exception
     */
    public function __construct($warningExpression = null, $criticalExpression = null, $warningMessage = null, $criticalMessage = null)
    {
        if (!$warningExpression && !$criticalExpression) {
            throw new \InvalidArgumentException('Not checks set.');
        }

        $this->warningExpression = $warningExpression;
        $this->warningMessage = $warningMessage;
        $this->criticalExpression = $criticalExpression;
        $this->criticalMessage = $criticalMessage;
    }

    /**
     * {@inheritdoc}
     */

Missing @return tag in function comment

    public function check()
    {
        $language = $this->getExpressionLanguage();

        if ($this->criticalExpression && false === $language->evaluate($this->criticalExpression)) {

The condition false === $language->evaluate($this->criticalExpression) is always false.

            return new Failure($this->criticalMessage);
        }

        if ($this->warningExpression && false === $language->evaluate($this->warningExpression)) {

The condition false === $language->evaluate($this->warningExpression) is always false.

            return new Warning($this->warningMessage);
        }

        return new Success();
    }

    protected function getExpressionLanguage()

Missing doc comment for function getExpressionLanguage()

    {
        $language = new ExpressionLanguage();
        $language->register('ini', static function ($value) {

The opening parenthesis of a multi-line function call should be the last content on the line.

            return $value;
        }, static function ($arguments, $value) {

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 8.

            return ini_get($value);
        });

For multi-line function calls, the closing parenthesis should be on a new line.

        return $language;
    }
}