Sonata\NotificationBundle\Iterator\AMQPMessageIterator

Complexity

Total Complexity

8

Size/Duplication

Total Lines	148
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
A	current()	0	4	1
A	next()	0	23	2
A	key()	0	4	1
A	valid()	0	4	1
A	rewind()	0	5	1
A	convertToAmqpLibMessage()	0	16	1

<?php

/*
 * This file is part of the Sonata Project package.
 *
 * (c) Thomas Rabaix <thomas.rabaix@sonata-project.org>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sonata\NotificationBundle\Iterator;

use Interop\Amqp\AmqpConsumer;
use PhpAmqpLib\Channel\AMQPChannel;
use PhpAmqpLib\Message\AMQPMessage;
use Sonata\NotificationBundle\Model\Message;

class AMQPMessageIterator implements MessageIteratorInterface
{
    /**
     * @deprecated since 3.2, will be removed in 4.x
     *
     * @var AMQPChannel
     */
    protected $channel;

    /**
     * @var mixed
     */
    protected $message;

    /**
     * @deprecated since 3.2, will be removed in 4.x
     *
     * @var AMQPMessage
     */
    protected $AMQMessage;

    /**
     * @deprecated since 3.2, will be removed in 4.x
     *
     * @var string
     */
    protected $queue;

    /**
     * @var int
     */
    protected $counter;

    /**
     * @var \Interop\Amqp\AmqpMessage
     */
    private $interopMessage;

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

    /**
     * @var int
     */
    private $timeout;

    /**
     * @var AmqpConsumer
     */
    private $consumer;

    /**
     * @var bool
     */
    private $isValid;

    public function __construct(AMQPChannel $channel, AmqpConsumer $consumer)
    {
        $this->consumer = $consumer;
        $this->counter = 0;
        $this->timeout = 0;
        $this->isValid = true;

        $this->channel = $channel;

The property Sonata\NotificationBundle\Iterator\AMQPMessageIterator::$channel has been deprecated with message: since 3.2, will be removed in 4.x

        $this->queue = $consumer->getQueue()->getQueueName();

The property Sonata\NotificationBundle\Iterator\AMQPMessageIterator::$queue has been deprecated with message: since 3.2, will be removed in 4.x

    }

    /**
     * {@inheritdoc}
     */
    public function current()
    {
        return $this->message;
    }

    /**
     * {@inheritdoc}
     */
    public function next()
    {
        $this->isValid = false;

        if ($amqpMessage = $this->consumer->receive($this->timeout)) {
            $this->AMQMessage = $this->convertToAmqpLibMessage($amqpMessage);

$amqpMessage of type object<Interop\Queue\PsrMessage> is not a sub-type of object<Interop\Amqp\AmqpMessage>. It seems like you assume a child interface of the interface Interop\Queue\PsrMessage to be always present.

The property Sonata\NotificationBundle\Iterator\AMQPMessageIterator::$AMQMessage has been deprecated with message: since 3.2, will be removed in 4.x

The method Sonata\NotificationBundle\Iterator\AMQPMessageIterator::convertToAmqpLibMessage() has been deprecated with message: since 3.2, will be removed in 4.x

            $data = json_decode($amqpMessage->getBody(), true);
            $data['body']['interopMessage'] = $amqpMessage;

            // @deprecated
            $data['body']['AMQMessage'] = $this->AMQMessage;

The property Sonata\NotificationBundle\Iterator\AMQPMessageIterator::$AMQMessage has been deprecated with message: since 3.2, will be removed in 4.x

            $message = new Message();
            $message->setBody($data['body']);
            $message->setType($data['type']);
            $message->setState($data['state']);
            $this->message = $message;

            ++$this->counter;
            $this->isValid = true;
        }
    }

    /**
     * {@inheritdoc}
     */
    public function key()
    {
        $this->counter;
    }

    /**
     * {@inheritdoc}
     */
    public function valid()
    {
        return $this->isValid;
    }

    /**
     * {@inheritdoc}
     */
    public function rewind()
    {
        $this->isValid = true;
        $this->next();
    }

    /**
     * @deprecated since 3.2, will be removed in 4.x
     *
     * @param \Interop\Amqp\AmqpMessage $amqpMessage
     *
     * @return AMQPMessage
     */
    private function convertToAmqpLibMessage(\Interop\Amqp\AmqpMessage $amqpMessage)
    {
        $amqpLibProperties = $amqpMessage->getHeaders();
        $amqpLibProperties['application_headers'] = $amqpMessage->getProperties();

        $amqpLibMessage = new AMQPMessage($amqpMessage->getBody(), $amqpLibProperties);
        $amqpLibMessage->delivery_info = [
            'consumer_tag' => $this->consumer->getConsumerTag(),
            'delivery_tag' => $amqpMessage->getDeliveryTag(),
            'redelivered' => $amqpMessage->isRedelivered(),
            'routing_key' => $amqpMessage->getRoutingKey(),
            'channel' => $this->channel,

The property Sonata\NotificationBundle\Iterator\AMQPMessageIterator::$channel has been deprecated with message: since 3.2, will be removed in 4.x

        ];

        return $amqpLibMessage;
    }
}