Ccovey\RabbitMQ\Channel

Complexity

Total Complexity

15

Size/Duplication

Total Lines	164
Duplicated Lines	18.9 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

93.1%

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	wait()	0	4	1
A	acknowledge()	0	4	1
A	nack()	0	4	1
A	consume()	0	16	1
A	publish()	0	19	2
A	basicQos()	0	6	1
A	declareQueue()	15	15	1
A	bindQueue()	0	13	1
A	declareExchange()	16	16	1
A	getCallbacks()	0	4	1
A	setCallbacks()	0	4	1
A	getMessage()	0	4	1
A	getQueueSize()	0	15	1

<?php

namespace Ccovey\RabbitMQ;

use Ccovey\RabbitMQ\Consumer\Consumable;
use Ccovey\RabbitMQ\Producer\Publishable;
use PhpAmqpLib\Channel\AMQPChannel;
use PhpAmqpLib\Message\AMQPMessage;

class Channel implements ChannelInterface
{
    /**
     * @var AMQPChannel
     */
    private $channel;

    public function __construct(AMQPChannel $channel)
    {
        $this->channel = $channel;
    }

    public function wait()
    {
        return $this->channel->wait();
    }

    public function acknowledge(string $deliveryTag, bool $multiple = false)
    {
        return $this->channel->basic_ack($deliveryTag, $multiple);
    }

    public function nack(string $deliveryTag, bool $multiple = false, bool $requeue = false)
    {
        return $this->channel->basic_nack($deliveryTag, $multiple, $requeue);
    }

    public function consume(Consumable $consumable) : ChannelInterface
    {
        $this->channel->basic_consume(
            $consumable->getQueueName(),
            $consumable->getConsumerTag(),
            $consumable->isNoLocal(),
            $consumable->isNoAck(),
            $consumable->isExclusive(),
            $consumable->isNoWait(),
            $consumable->getCallback(),
            $consumable->getTicket(),
            $consumable->getArguments()
        );

        return $this;
    }

    public function publish(Publishable $message) : ChannelInterface
    {
        if ($message->isDelayed()) {
            $queue = new Queue($message->getQueueName(), $message->getDelayedExchanged());
            $message->setExchange($message->getDelayedExchanged());

The method setExchange() does not seem to exist on object<Ccovey\RabbitMQ\Producer\Publishable>.

            $this->bindQueue($queue);
        }

        $this->channel->basic_publish(
            $message->getMessage(),
            $message->getExchange(),
            $message->getQueueName(),
            $message->isMandatory(),
            $message->isImmediate(),
            $message->getTicket()
        );

        return $this;
    }

    /**
     * @param     $prefetchSize
     * @param int $prefetchCount
     * @param     $aGlobal
     *
     * @return mixed
     */
    public function basicQos($prefetchSize = null, int $prefetchCount = 1, $aGlobal = null)
    {
        $this->channel->basic_qos($prefetchSize, $prefetchCount, $aGlobal);

        return $this;
    }

    public function declareQueue(Queue $queue) : ChannelInterface

This method seems to be duplicated in your project.

    {
        $this->channel->queue_declare(
            $queue->getQueueName(),
            $queue->isPassive(),
            $queue->isDurable(),
            $queue->isExclusive(),
            $queue->isAutoDelete(),
            $queue->isNoWait(),
            $queue->getArguments(),

It seems like $queue->getArguments() targeting Ccovey\RabbitMQ\Queue::getArguments() can also be of type object<PhpAmqpLib\Wire\AMQPTable>; however, PhpAmqpLib\Channel\AMQPChannel::queue_declare() does only seem to accept array|null, maybe add an additional type check?

            $queue->getTicket()
        );

        return $this;
    }

    public function bindQueue(Queue $queue) : ChannelInterface
    {
        $this->channel->queue_bind(
            $queue->getQueueName(),
            $queue->getExchange(),
            $queue->getQueueName(),
            $queue->isNoWait(),
            $queue->getArguments(),

It seems like $queue->getArguments() targeting Ccovey\RabbitMQ\Queue::getArguments() can also be of type object<PhpAmqpLib\Wire\AMQPTable>; however, PhpAmqpLib\Channel\AMQPChannel::queue_bind() does only seem to accept array|null, maybe add an additional type check?

            $queue->getTicket()
        );

        return $this;
    }

    public function declareExchange(Exchange $exchange) : ChannelInterface

This method seems to be duplicated in your project.

    {
        $this->channel->exchange_declare(
            $exchange->getExchange(),
            $exchange->getType(),
            $exchange->isPassive(),
            $exchange->isDurable(),
            $exchange->isAutoDelete(),
            $exchange->isInternal(),
            $exchange->isNoWait(),
            $exchange->getArguments(),
            $exchange->getTicket()
        );

        return $this;
    }

    /**
     * @return array
     */
    public function getCallbacks() : array
    {
        return $this->channel->callbacks;
    }

    public function setCallbacks(array $callbacks = [])
    {
        $this->channel->callbacks = $callbacks;
    }

    /**
     * @param Consumable $consumable
     *
     * @return AMQPMessage|null
     */
    public function getMessage(Consumable $consumable)
    {
        return $this->channel->basic_get($consumable->getQueueName(), $consumable->isNoAck(), $consumable->getTicket());
    }

    public function getQueueSize(Queue $queue) : int
    {
        $declaredQueue = $this->channel->queue_declare(
            $queue->getQueueName(),
            true,
            $queue->isDurable(),
            $queue->isExclusive(),
            $queue->isAutoDelete(),
            $queue->isNoWait(),
            $queue->getArguments(),

It seems like $queue->getArguments() targeting Ccovey\RabbitMQ\Queue::getArguments() can also be of type object<PhpAmqpLib\Wire\AMQPTable>; however, PhpAmqpLib\Channel\AMQPChannel::queue_declare() does only seem to accept array|null, maybe add an additional type check?

            $queue->getTicket()
        );

        return $declaredQueue[1] ?? 0;
    }
}