@@ -59,7 +59,7 @@ |
||
59 | 59 | |
60 | 60 | $this->driver->consume( |
61 | 61 | $this->queueName, |
62 | - function (Message $message) { |
|
62 | + function(Message $message) { |
|
63 | 63 | $this->monitor->monitor($this, $message); |
64 | 64 | return $this->handler->handle($message); |
65 | 65 | }, |
@@ -165,7 +165,7 @@ |
||
165 | 165 | $flags = $autoAck ? AMQP_AUTOACK : AMQP_NOPARAM; |
166 | 166 | |
167 | 167 | try { |
168 | - $queue->consume(function (\AMQPEnvelope $message) use ($callback, $queueName) { |
|
168 | + $queue->consume(function(\AMQPEnvelope $message) use ($callback, $queueName) { |
|
169 | 169 | |
170 | 170 | $burrowMessage = new Message( |
171 | 171 | $message->getBody(), |
@@ -57,7 +57,7 @@ discard block |
||
57 | 57 | $durable = ($type === self::QUEUE_DURABLE); |
58 | 58 | $exclusive = ($type === self::QUEUE_EXCLUSIVE); |
59 | 59 | |
60 | - list($name, , ) = $this->getChannel()->queue_declare($queueName, false, $durable, $exclusive, false); |
|
60 | + list($name,,) = $this->getChannel()->queue_declare($queueName, false, $durable, $exclusive, false); |
|
61 | 61 | |
62 | 62 | return $name; |
63 | 63 | } |
@@ -72,7 +72,7 @@ discard block |
||
72 | 72 | */ |
73 | 73 | public function declareExchange($exchangeName = '', $type = self::EXCHANGE_TYPE_FANOUT) |
74 | 74 | { |
75 | - list($name, , ) = $this->getChannel()->exchange_declare($exchangeName, $type, false, true, false); |
|
75 | + list($name,,) = $this->getChannel()->exchange_declare($exchangeName, $type, false, true, false); |
|
76 | 76 | |
77 | 77 | return $name; |
78 | 78 | } |
@@ -166,7 +166,7 @@ discard block |
||
166 | 166 | $autoAck, |
167 | 167 | false, |
168 | 168 | false, |
169 | - function (AMQPMessage $message) use ($callback, $queueName) { |
|
169 | + function(AMQPMessage $message) use ($callback, $queueName) { |
|
170 | 170 | $burrowMessage = new Message( |
171 | 171 | $message->getBody(), |
172 | 172 | '', // Impossible to retrieve here |
@@ -242,7 +242,7 @@ discard block |
||
242 | 242 | } |
243 | 243 | |
244 | 244 | /** |
245 | - * @param $timeout |
|
245 | + * @param integer $timeout |
|
246 | 246 | * |
247 | 247 | * @throws \Exception |
248 | 248 | */ |
@@ -291,7 +291,7 @@ discard block |
||
291 | 291 | /** |
292 | 292 | * @param AMQPMessage $message |
293 | 293 | * |
294 | - * @return array |
|
294 | + * @return string[] |
|
295 | 295 | * |
296 | 296 | * @throws \OutOfBoundsException |
297 | 297 | */ |
@@ -58,7 +58,7 @@ |
||
58 | 58 | |
59 | 59 | $this->driver->consume( |
60 | 60 | $replyTo, |
61 | - function (Message $message) use ($correlationId, &$response) { |
|
61 | + function(Message $message) use ($correlationId, &$response) { |
|
62 | 62 | if ($message->getCorrelationId() == $correlationId) { |
63 | 63 | $response = $message->getBody(); |
64 | 64 | return QueueHandler::STOP_CONSUMING; |
@@ -119,14 +119,12 @@ |
||
119 | 119 | Assertion::notNull($this->sync, 'You must specify if the handler must be sync or async'); |
120 | 120 | |
121 | 121 | $syncAsync = $this->sync ? |
122 | - new SyncConsumerHandler($consumer, $this->driver) : |
|
123 | - new AsyncConsumerHandler($consumer); |
|
122 | + new SyncConsumerHandler($consumer, $this->driver) : new AsyncConsumerHandler($consumer); |
|
124 | 123 | |
125 | 124 | $ackHandler = new AckHandler($syncAsync, $this->driver, $this->requeueOnFailure); |
126 | 125 | |
127 | 126 | $handler = $this->stopOnFailure ? |
128 | - new StopOnExceptionHandler($ackHandler) : |
|
129 | - new ContinueOnExceptionHandler($ackHandler); |
|
127 | + new StopOnExceptionHandler($ackHandler) : new ContinueOnExceptionHandler($ackHandler); |
|
130 | 128 | |
131 | 129 | $syncAsync->setLogger($this->logger); |
132 | 130 | $handler->setLogger($this->logger); |