eMAGTechLabs / RabbitMqBundle

Tests/RabbitMq/MultipleConsumerTest.php

Spacing +3 added lines, -3 removed lines

@@ -310,14 +310,14 @@ 
     {
         $this->amqpChannel->expects($this->any())
             ->method('basic_reject')
-            ->will($this->returnCallback(function ($delivery_tag, $requeue) use ($expectedMethod, $expectedRequeue) {
+            ->will($this->returnCallback(function($delivery_tag, $requeue) use ($expectedMethod, $expectedRequeue) {
                 Assert::assertSame($expectedMethod, 'basic_reject'); // Check if this function should be called.
                 Assert::assertSame($requeue, $expectedRequeue); // Check if the message should be requeued.
             }));
 
         $this->amqpChannel->expects($this->any())
             ->method('basic_ack')
-            ->will($this->returnCallback(function ($delivery_tag) use ($expectedMethod) {
+            ->will($this->returnCallback(function($delivery_tag) use ($expectedMethod) {
                 Assert::assertSame($expectedMethod, 'basic_ack'); // Check if this function should be called.
             }));
     }
@@ -329,7 +329,7 @@ 
      */
     private function prepareCallback(?int $processFlag)
     {
-        return function ($msg) use ($processFlag) {
+        return function($msg) use ($processFlag) {
             return $processFlag;
         };
     }

Tests/RabbitMq/AMQPConnectionFactoryTest.php

Spacing +117 added lines, -117 removed lines

@@ -23,19 +23,19 @@ 
         $this->assertInstanceOf('OldSound\RabbitMqBundle\Tests\RabbitMq\Fixtures\AMQPConnection', $instance);
         $this->assertEquals(array(
             'localhost', // host
-            5672,        // port
-            'guest',     // user
-            'guest',     // password
-            '/',         // vhost
-            false,       // insist
-            "AMQPLAIN",  // login method
-            null,        // login response
-            "en_US",     // locale
-            3,           // connection timeout
-            3,           // read write timeout
-            null,        // context
-            false,       // keepalive
-            0,           // heartbeat
+            5672, // port
+            'guest', // user
+            'guest', // password
+            '/', // vhost
+            false, // insist
+            "AMQPLAIN", // login method
+            null, // login response
+            "en_US", // locale
+            3, // connection timeout
+            3, // read write timeout
+            null, // context
+            false, // keepalive
+            0, // heartbeat
         ), $instance->constructParams);
     }
 
@@ -51,18 +51,18 @@ 
         $this->assertInstanceOf('PhpAmqpLib\Connection\AMQPSocketConnection', $instance);
         $this->assertEquals(array(
             'localhost', // host
-            5672,        // port
-            'guest',     // user
-            'guest',     // password
-            '/',         // vhost
-            false,       // insist
-            "AMQPLAIN",  // login method
-            null,        // login response
-            "en_US",     // locale
-            3,           // read_timeout
-            false,       // keepalive
-            3,           // write_timeout
-            0,           // heartbeat
+            5672, // port
+            'guest', // user
+            'guest', // password
+            '/', // vhost
+            false, // insist
+            "AMQPLAIN", // login method
+            null, // login response
+            "en_US", // locale
+            3, // read_timeout
+            false, // keepalive
+            3, // write_timeout
+            0, // heartbeat
         ), $instance->constructParams);
     }
 
@@ -81,18 +81,18 @@ 
         $this->assertInstanceOf('PhpAmqpLib\Connection\AMQPSocketConnection', $instance);
         $this->assertEquals(array(
             'localhost', // host
-            5672,        // port
-            'guest',     // user
-            'guest',     // password
-            '/',         // vhost
-            false,       // insist
-            "AMQPLAIN",  // login method
-            null,        // login response
-            "en_US",     // locale
-            31,           // read_timeout
-            false,       // keepalive
-            32,           // write_timeout
-            0,           // heartbeat
+            5672, // port
+            'guest', // user
+            'guest', // password
+            '/', // vhost
+            false, // insist
+            "AMQPLAIN", // login method
+            null, // login response
+            "en_US", // locale
+            31, // read_timeout
+            false, // keepalive
+            32, // write_timeout
+            0, // heartbeat
         ), $instance->constructParams);
     }
 
@@ -113,20 +113,20 @@ 
         $instance = $factory->createConnection();
         $this->assertInstanceOf('OldSound\RabbitMqBundle\Tests\RabbitMq\Fixtures\AMQPConnection', $instance);
         $this->assertEquals(array(
-            'foo_host',  // host
-            123,         // port
-            'foo_user',  // user
+            'foo_host', // host
+            123, // port
+            'foo_user', // user
             'foo_password', // password
-            '/vhost',    // vhost
-            false,       // insist
-            "AMQPLAIN",  // login method
-            null,        // login response
-            "en_US",     // locale
-            3,           // connection timeout
-            3,           // read write timeout
-            null,        // context
-            false,       // keepalive
-            0,           // heartbeat
+            '/vhost', // vhost
+            false, // insist
+            "AMQPLAIN", // login method
+            null, // login response
+            "en_US", // locale
+            3, // connection timeout
+            3, // read write timeout
+            null, // context
+            false, // keepalive
+            0, // heartbeat
         ), $instance->constructParams);
     }
 
@@ -148,20 +148,20 @@ 
         $instance = $factory->createConnection();
         $this->assertInstanceOf('OldSound\RabbitMqBundle\Tests\RabbitMq\Fixtures\AMQPConnection', $instance);
         $this->assertEquals(array(
-            'bar_host',  // host
-            321,         // port
-            'bar_user',  // user
+            'bar_host', // host
+            321, // port
+            'bar_user', // user
             'bar_password', // password
-            'whost',     // vhost
-            false,       // insist
-            "AMQPLAIN",  // login method
-            null,        // login response
-            "en_US",     // locale
-            6,           // connection timeout
-            6,           // read write timeout
-            null,        // context
-            true,        // keepalive
-            0,           // heartbeat
+            'whost', // vhost
+            false, // insist
+            "AMQPLAIN", // login method
+            null, // login response
+            "en_US", // locale
+            6, // connection timeout
+            6, // read write timeout
+            null, // context
+            true, // keepalive
+            0, // heartbeat
         ), $instance->constructParams);
     }
 
@@ -178,20 +178,20 @@ 
         $instance = $factory->createConnection();
         $this->assertInstanceOf('OldSound\RabbitMqBundle\Tests\RabbitMq\Fixtures\AMQPConnection', $instance);
         $this->assertEquals(array(
-            'hoast',     // host
-            10000,       // port
-            'usera',     // user
-            'apass',     // password
-            'v/host',    // vhost
-            false,       // insist
-            "AMQPLAIN",  // login method
-            null,        // login response
-            "en_US",     // locale
-            6,           // connection timeout
-            6,           // read write timeout
-            null,        // context
-            true,        // keepalive
-            0,           // heartbeat
+            'hoast', // host
+            10000, // port
+            'usera', // user
+            'apass', // password
+            'v/host', // vhost
+            false, // insist
+            "AMQPLAIN", // login method
+            null, // login response
+            "en_US", // locale
+            6, // connection timeout
+            6, // read write timeout
+            null, // context
+            true, // keepalive
+            0, // heartbeat
         ), $instance->constructParams);
     }
 
@@ -208,20 +208,20 @@ 
         $instance = $factory->createConnection();
         $this->assertInstanceOf('OldSound\RabbitMqBundle\Tests\RabbitMq\Fixtures\AMQPConnection', $instance);
         $this->assertEquals(array(
-            'host',     // host
-            321,        // port
-            'user',     // user
-            'pass',     // password
-            '',         // vhost
-            false,      // insist
+            'host', // host
+            321, // port
+            'user', // user
+            'pass', // password
+            '', // vhost
+            false, // insist
             "AMQPLAIN", // login method
-            null,       // login response
-            "en_US",    // locale
-            6,          // connection timeout
-            6,          // read write timeout
-            null,       // context
-            true,       // keepalive
-            0,          // heartbeat
+            null, // login response
+            "en_US", // locale
+            6, // connection timeout
+            6, // read write timeout
+            null, // context
+            true, // keepalive
+            0, // heartbeat
         ), $instance->constructParams);
     }
 
@@ -254,19 +254,19 @@ 
         $this->assertEquals(array('ssl' => array('verify_peer' => false)), $options);
         $this->assertEquals(array(
             'ssl_host', // host
-            123,        // port
+            123, // port
             'ssl_user', // user
             'ssl_password', // password
-            '/ssl',      // vhost
-            false,       // insist
-            "AMQPLAIN",  // login method
-            null,        // login response
-            "en_US",     // locale
-            3,           // connection timeout
-            3,           // read write timeout
-            null,        // context checked earlier
-            false,       // keepalive
-            0,           // heartbeat
+            '/ssl', // vhost
+            false, // insist
+            "AMQPLAIN", // login method
+            null, // login response
+            "en_US", // locale
+            3, // connection timeout
+            3, // read write timeout
+            null, // context checked earlier
+            false, // keepalive
+            0, // heartbeat
         ), $instance->constructParams);
     }
 
@@ -277,7 +277,7 @@ 
             ->method('getConnectionParameters')
             ->will($this->returnValue(
                 array(
-                    'constructor_args' => array(1,2,3,4)
+                    'constructor_args' => array(1, 2, 3, 4)
                 )
             ));
         $factory = new AMQPConnectionFactory(
@@ -289,7 +289,7 @@ 
         /** @var AMQPConnection $instance */
         $instance = $factory->createConnection();
         $this->assertInstanceOf('OldSound\RabbitMqBundle\Tests\RabbitMq\Fixtures\AMQPConnection', $instance);
-        $this->assertEquals(array(1,2,3,4), $instance->constructParams);
+        $this->assertEquals(array(1, 2, 3, 4), $instance->constructParams);
     }
 
     public function testConnectionsParametersProvider(): void
@@ -316,20 +316,20 @@ 
         $instance = $factory->createConnection();
         $this->assertInstanceOf('OldSound\RabbitMqBundle\Tests\RabbitMq\Fixtures\AMQPConnection', $instance);
         $this->assertEquals(array(
-            '1.2.3.4',   // host
-            5678,        // port
-            'admin',     // user
-            'admin',     // password
-            'foo',       // vhost
-            false,       // insist
-            "AMQPLAIN",  // login method
-            null,        // login response
-            "en_US",     // locale
-            3,           // connection timeout
-            3,           // read write timeout
-            null,        // context
-            false,       // keepalive
-            0,           // heartbeat
+            '1.2.3.4', // host
+            5678, // port
+            'admin', // user
+            'admin', // password
+            'foo', // vhost
+            false, // insist
+            "AMQPLAIN", // login method
+            null, // login response
+            "en_US", // locale
+            3, // connection timeout
+            3, // read write timeout
+            null, // context
+            false, // keepalive
+            0, // heartbeat
         ), $instance->constructParams);
     }
 

RabbitMq/MultipleConsumer.php

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
             //PHP 5.3 Compliant
             $currentObject = $this;
 
-            $this->getChannel()->basic_consume($name, $this->getQueueConsumerTag($name), false, false, false, false, function (AMQPMessage $msg) use($currentObject, $name) {
+            $this->getChannel()->basic_consume($name, $this->getQueueConsumerTag($name), false, false, false, false, function(AMQPMessage $msg) use($currentObject, $name) {
                 $currentObject->processQueueMessage($name, $msg);
             });
         }
@@ -70,7 +70,7 @@ 
     protected function queueDeclare(): void
     {
         foreach ($this->queues as $name => $options) {
-            list($queueName, ,) = $this->getChannel()->queue_declare(
+            list($queueName,,) = $this->getChannel()->queue_declare(
                 $name, $options['passive'],
                 $options['durable'], $options['exclusive'],
                 $options['auto_delete'], $options['nowait'],

RabbitMq/BatchConsumer.php

Spacing +1 added lines, -1 removed lines

@@ -263,7 +263,7 @@
     private function addMessage(AMQPMessage $message): void
     {
         $this->batchCounter++;
-        $this->messages[(int)$message->delivery_info['delivery_tag']] = $message;
+        $this->messages[(int) $message->delivery_info['delivery_tag']] = $message;
     }
 
     private function getMessage(int $deliveryTag): ?AMQPMessage

RabbitMq/RpcServer.php

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
     public function initServer(string $name): void
     {
         $this->setExchangeOptions(array('name' => $name, 'type' => 'direct'));
-        $this->setQueueOptions(array('name' => $name . '-queue'));
+        $this->setQueueOptions(array('name' => $name.'-queue'));
     }
 
     public function processMessage(AMQPMessage $msg): void
@@ -25,7 +25,7 @@ 
             $this->consumed++;
             $this->maybeStopConsumer();
         } catch (\Exception $e) {
-            $this->sendReply('error: ' . $e->getMessage(), $msg->get('reply_to'), $msg->get('correlation_id'));
+            $this->sendReply('error: '.$e->getMessage(), $msg->get('reply_to'), $msg->get('correlation_id'));
         }
     }
 

RabbitMq/BaseConsumer.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
     /**
      * @throws \ErrorException
      */
-    public function start (int $msgAmount = 0): void
+    public function start(int $msgAmount = 0): void
     {
         $this->target = $msgAmount;
 

RabbitMq/RpcClient.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
                                                    ? 'amq.rabbitmq.reply-to' // On direct reply-to mode, use predefined queue name
                                                    : $this->getQueueName(),
                                                'delivery_mode' => 1, // non durable
-                                               'expiration' => $expiration*1000,
+                                               'expiration' => $expiration * 1000,
                                                'correlation_id' => $requestId));
 
         $this->getChannel()->basic_publish($msg, $server, $routingKey);
@@ -102,7 +102,7 @@ 
     protected function getQueueName(): ?string
     {
         if (null === $this->queueName) {
-            list($this->queueName, ,) = $this->getChannel()->queue_declare("", false, false, true, false);
+            list($this->queueName,,) = $this->getChannel()->queue_declare("", false, false, true, false);
         }
 
         return $this->queueName;

RabbitMq/BaseAmqp.php

Spacing +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
     protected function queueDeclare(): void
     {
         if ($this->queueOptions['declare']) {
-            list($queueName, ,) = $this->getChannel()->queue_declare($this->queueOptions['name'], $this->queueOptions['passive'],
+            list($queueName,,) = $this->getChannel()->queue_declare($this->queueOptions['name'], $this->queueOptions['passive'],
                 $this->queueOptions['durable'], $this->queueOptions['exclusive'],
                 $this->queueOptions['auto_delete'], $this->queueOptions['nowait'],
                 $this->queueOptions['arguments'], $this->queueOptions['ticket']);

Command/BaseConsumerCommand.php

Spacing +2 added lines, -2 removed lines

@@ -100,7 +100,7 @@ 
         }
         $this->initConsumer($input);
 
-        return $this->consumer->consume((int)$this->amount);
+        return $this->consumer->consume((int) $this->amount);
     }
 
     protected function initConsumer(InputInterface $input): void
@@ -108,7 +108,7 @@ 
         $this->consumer = $this->getContainer()
                 ->get(sprintf($this->getConsumerService(), $input->getArgument('name')));
 
-        if (!is_null($input->getOption('memory-limit')) && ctype_digit((string) $input->getOption('memory-limit')) && (int)$input->getOption('memory-limit') > 0) {
+        if (!is_null($input->getOption('memory-limit')) && ctype_digit((string) $input->getOption('memory-limit')) && (int) $input->getOption('memory-limit') > 0) {
             $this->consumer->setMemoryLimit($input->getOption('memory-limit'));
         }
         $this->consumer->setRoutingKey($input->getOption('route'));