contamobi / CmobiRabbitmqBundle

Transport/Rpc/RpcQueueBag.php

Doc Comments +2 added lines, -2 removed lines

@@ -274,7 +274,7 @@ 
     }
 
     /**
-     * @return string
+     * @return boolean
      */
     public function getExchange()
     {
@@ -282,7 +282,7 @@ 
     }
 
     /**
-     * @return string
+     * @return boolean
      */
     public function getType()
     {

Transport/Worker/Task.php

Doc Comments +5 added lines, -2 removed lines

@@ -15,6 +15,9 @@ 
     private $fromName;
     private $queueName;
 
+    /**
+     * @param string $queueName
+     */
     public function __construct($queueName, ConnectionManager $manager, $fromName = '')
     {
         $this->queueName = $queueName;
@@ -87,7 +90,7 @@ 
     }
 
     /**
-     * @return string
+     * @return boolean
      */
     public function getExchange()
     {
@@ -95,7 +98,7 @@ 
     }
 
     /**
-     * @return string
+     * @return boolean
      */
     public function getExchangeType()
     {

Spacing +2 added lines, -2 removed lines

@@ -34,13 +34,13 @@
     {
         $this->refreshChannel();
 
-        if (! $this->queueHasExists()) {
+        if (!$this->queueHasExists()) {
             throw new QueueNotFoundException("Queue $this->queueName not declared.");
         }
         $queueBag = new WorkerQueueBag($this->getQueueName());
         $this->getChannel()->queueDeclare($queueBag->getQueueDeclare());
         $msg = new CmobiAMQPMessage(
-            (string) $data,
+            (string)$data,
             [
                 'delivery_mode' => 2, // make message persistent
                 'priority' => $priority,

Transport/Worker/WorkerQueueCallback.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     public function toClosure()
     {
-        return function (AMQPMessage $message) {
+        return function(AMQPMessage $message) {
             $this->getQueueService()->handle($message);
 
             $message->delivery_info['channel']->basic_ack($message->delivery_info['delivery_tag']);

Tests/Transport/Rpc/RpcClientTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
 
     /**
      * @param string $msg
-     * @param null   $correlationId
+     * @param string   $correlationId
      *
      * @return CmobiAMQPMessage
      */

Connection/ConnectionFactory.php

Spacing +3 added lines, -3 removed lines

@@ -62,10 +62,10 @@
             $this->parameters['user'],
             $this->parameters['password'],
             $this->parameters['vhost'],
-            false,      // insist
+            false, // insist
             'AMQPLAIN', // login_method
-            null,       // login_response
-            'en_US',    // locale
+            null, // login_response
+            'en_US', // locale
             $this->parameters['connection_timeout'],
             $this->parameters['read_write_timeout'],
             $this->parameters['ssl_context'],

Transport/Rpc/RpcQueueCallback.php

Spacing +2 added lines, -2 removed lines

@@ -29,11 +29,11 @@
      */
     public function toClosure()
     {
-        return function (AMQPMessage $message) {
+        return function(AMQPMessage $message) {
             $response = $this->getQueueService()->handle($message);
 
             $amqpResponse = new CmobiAMQPMessage(
-                (string) $response,
+                (string)$response,
                 ['correlation_id' => $message->get('correlation_id')]
             );
 

Tests/Queue/QueueTest.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
             ->disableOriginalConstructor()
             ->getMock();
         $callbackMock->method('toClosure')
-            ->willReturn(function () {});
+            ->willReturn(function() {});
 
         return $callbackMock;
     }

Transport/Worker/WorkerBuilder.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Cmobi\RabbitmqBundle\Transport\Worker;
 
 use Cmobi\RabbitmqBundle\Connection\ConnectionManager;
-use Cmobi\RabbitmqBundle\Connection\Exception\InvalidAMQPChannelException;
 use Cmobi\RabbitmqBundle\Queue\Queue;
 use Cmobi\RabbitmqBundle\Queue\QueueBagInterface;
 use Cmobi\RabbitmqBundle\Queue\QueueBuilderInterface;

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     }
 
     /**
-     * @param $queueName
+     * @param string $queueName
      * @param QueueServiceInterface $queueService
      * @param QueueBagInterface $queueBag
      * @return Queue

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      */
     public function buildQueue($queueName, QueueServiceInterface $queueService, QueueBagInterface $queueBag)
     {
-        if (! $queueBag instanceof WorkerQueueBag) {
+        if (!$queueBag instanceof WorkerQueueBag) {
             throw new \Exception('Unsupported QueueBag');
         }
         $queue = new Queue($this->getConnectionManager(), $queueBag, $this->logger, $this->connectionName);

Transport/Subscriber/Publisher.php

Doc Comments +3 added lines

@@ -17,6 +17,9 @@
     private $exchange;
     private $exchangeType;
 
+    /**
+     * @param string $exchange
+     */
     public function __construct(
         $exchange,
         $exchangeType = ExchangeType::FANOUT,

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         $this->refreshChannel();
         $queueBag = new SubscriberQueueBag($this->getExchange(), $this->getExchangeType(), $this->getQueueName());
         $this->getChannel()->exchangeDeclare($queueBag->getExchangeDeclare());
-        $msg = new CmobiAMQPMessage((string) $data);
+        $msg = new CmobiAMQPMessage((string)$data);
         $this->getChannel()->basic_publish($msg, $queueBag->getExchange());
 
         $this->getChannel()->close();