contamobi / CmobiRabbitmqBundle

Transport/PubSub/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();

Transport/Rpc/RpcClient.php

Doc Comments +7 added lines, -4 removed lines

@@ -18,6 +18,9 @@ 
     private $correlationId;
     private $callbackQueue;
 
+    /**
+     * @param string $queueName
+     */
     public function __construct($queueName, ConnectionManager $manager, $fromName = '')
     {
         $this->queueName = $queueName;
@@ -52,7 +55,7 @@ 
     }
 
     /**
-     * @param $data
+     * @param string $data
      * @param int $expire
      * @param int $priority
      */
@@ -118,7 +121,7 @@ 
     /**
      * @todo unecessary method set, its only exists to run tests whitout stay jailed in infinite while waiting response.
      *
-     * @param $content
+     * @param string $content
      */
     public function setResponse($content)
     {
@@ -148,7 +151,7 @@ 
     }
 
     /**
-     * @return string
+     * @return boolean
      */
     public function getExchange()
     {
@@ -156,7 +159,7 @@ 
     }
 
     /**
-     * @return string
+     * @return boolean
      */
     public function getExchangeType()
     {

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         $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/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 +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         $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/WorkerBuilder.php

Doc Comments +1 added lines, -1 removed lines

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

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;

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,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);

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
      */

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/Rpc/RpcServerBuilder.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +1 added lines, -1 removed lines

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

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'],