contamobi / CmobiRabbitmqBundle

Transport/Rpc/RpcClient.php

Doc Comments +9 added lines, -5 removed lines

@@ -24,6 +24,10 @@ 
     private $correlationId;
     private $callbackQueue;
 
+    /**
+     * @param string $queueName
+     * @param string $fromName
+     */
     public function __construct($queueName, ConnectionManager $manager, $fromName, $connectionName = 'default')
     {
         $this->connectionName = $connectionName;
@@ -74,7 +78,7 @@ 
     }
 
     /**
-     * @param $data
+     * @param string $data
      * @param int $expire
      * @param int $priority
      * @throws QueueNotFoundException
@@ -151,7 +155,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)
     {
@@ -181,7 +185,7 @@ 
     }
 
     /**
-     * @return string
+     * @return boolean
      */
     public function getExchange()
     {
@@ -189,7 +193,7 @@ 
     }
 
     /**
-     * @return string
+     * @return boolean
      */
     public function getExchangeType()
     {
@@ -206,7 +210,7 @@ 
 
     /**
      * @param CmobiAMQPConnectionInterface $connection
-     * @param $expire
+     * @param integer $expire
      * @param $corralationId
      * @return CmobiAMQPConnectionInterface
      */

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Cmobi\RabbitmqBundle\Transport\Rpc;
 
 use Cmobi\RabbitmqBundle\Connection\CmobiAMQPChannel;
-use Cmobi\RabbitmqBundle\Connection\CmobiAMQPConnection;
 use Cmobi\RabbitmqBundle\Connection\CmobiAMQPConnectionInterface;
 use Cmobi\RabbitmqBundle\Connection\ConnectionManager;
 use Cmobi\RabbitmqBundle\Queue\CmobiAMQPMessage;

Spacing +7 added lines, -7 removed lines

@@ -86,11 +86,11 @@ 
         $connection = $this->connectionManager->getConnection($this->connectionName);
         $channel = $connection->channel();
 
-        if (! $this->queueHasExists($channel)) {
+        if (!$this->queueHasExists($channel)) {
             throw new QueueNotFoundException("Queue $this->queueName not declared.");
         }
         $msg = new CmobiAMQPMessage(
-            (string) $data,
+            (string)$data,
             [
                 'correlation_id' => $this->correlationId,
                 'reply_to' => $this->callbackQueue,
@@ -99,7 +99,7 @@ 
         );
         $channel->basic_publish($msg, '', $this->getQueueName());
 
-        while (! $this->response) {
+        while (!$this->response) {
             try {
                 $channel->wait(null, 0, ($expire / 1000));
             } catch (\Exception $e) {
@@ -169,7 +169,7 @@ 
     /** @return string */
     public function generateCorrelationId()
     {
-        return uniqid($this->getQueueName()) . Uuid::uuid4()->toString() . microtime();
+        return uniqid($this->getQueueName()).Uuid::uuid4()->toString().microtime();
     }
 
     /**
@@ -216,17 +216,17 @@ 
             try {
                 $connection->close();
                 $failed = false;
-                fwrite($this->logOutput, 'start RpcClient::forceReconnect() - trying connect...' . PHP_EOL);
+                fwrite($this->logOutput, 'start RpcClient::forceReconnect() - trying connect...'.PHP_EOL);
                 $connection = $this->getConnectionManager()->getConnection($this->connectionName);
                 $channel = $connection->channel();
                 $this->createCallbackQueue($channel, $expire, $corralationId);
             } catch (\Exception $e) {
                 $failed = true;
                 sleep(3);
-                fwrite($this->errOutput, 'failed RpcClient::forceReconnect() - ' . $e->getMessage() . PHP_EOL);
+                fwrite($this->errOutput, 'failed RpcClient::forceReconnect() - '.$e->getMessage().PHP_EOL);
             }
         } while ($failed);
-        fwrite($this->logOutput, 'RpcClient::forceReconnect() - connected!' . PHP_EOL);
+        fwrite($this->logOutput, 'RpcClient::forceReconnect() - connected!'.PHP_EOL);
 
         return $connection;
     }

Transport/Subscriber/Publisher.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Cmobi\RabbitmqBundle\Transport\Subscriber;
 
-use Cmobi\RabbitmqBundle\Connection\CmobiAMQPChannel;
 use Cmobi\RabbitmqBundle\Connection\CmobiAMQPConnectionInterface;
 use Cmobi\RabbitmqBundle\Connection\ConnectionManager;
 use Cmobi\RabbitmqBundle\Queue\CmobiAMQPMessage;

Spacing +1 added lines, -1 removed lines

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

Transport/Worker/Task.php

Spacing +2 added lines, -2 removed lines

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