Tvi\MonitorBundle\Check\rabbitmq\QueueSize\Check

Complexity

Total Complexity

3

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Test Coverage

Coverage

75%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	check()	0	12	2
A	__construct()	0	14	1

<?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\rabbitmq\QueueSize;

use JMS\Serializer\Annotation as JMS;
use Tvi\MonitorBundle\Check\rabbitmq\RabbitMQClient;
use Tvi\MonitorBundle\Check\CheckAbstract;
use ZendDiagnostics\Result\Failure;
use ZendDiagnostics\Result\Success;

/**

Missing short description in doc comment

 * @JMS\ExclusionPolicy("all")
 *
 * @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 CheckAbstract
{
    /**

Missing short description in doc comment

     * @var RabbitMQClient
     */
    private $client;

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

    /**

Missing short description in doc comment

     * @param string     $host

Missing parameter comment

     * @param int        $port

Missing parameter comment

     * @param string     $user

Missing parameter comment

     * @param string     $password

Missing parameter comment

     * @param string     $vhost

Missing parameter comment

     * @param null|mixed $dsn

Missing parameter comment

     */
    public function __construct(
        $host = 'localhost',
        $port = 5672,
        $user = 'guest',
        $password = 'guest',
        $vhost = '/',
        $dsn = null)

The closing parenthesis of a multi-line function declaration must be on a new line

    {

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

        $this->client = new RabbitMQClient($host,

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

            $port,
            $user,
            $password,
            $vhost,
            $dsn);

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

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

    }

    /**
     * {@inheritdoc}
     */

Missing @return tag in function comment

    public function check()
    {
        try {
            $conn = $this->client->getConnect();
            $conn->channel();

            $conn->isConnected();
            $version = $conn->getServerProperties()['version'][1];

            return new Success(null, $version);
        } catch (\Exception $e) {
            return new Failure($e->getMessage());
        }
    }
}