Skip to content

Commit 31bba0e

Browse files
author
freyhsiao
committed
Apply fixes from StyleCI
1 parent 8f4a2a2 commit 31bba0e

File tree

3 files changed

+25
-20
lines changed

3 files changed

+25
-20
lines changed

src/Queue/CMQQueue.php

Lines changed: 19 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ class CMQQueue extends Queue implements QueueContract
5353
*
5454
* @param \Freyo\LaravelQueueCMQ\Queue\Driver\Account $queueAccount
5555
* @param \Freyo\LaravelQueueCMQ\Queue\Driver\Account $topicAccount
56-
* @param array $config
56+
* @param array $config
5757
*
5858
* @throws \ReflectionException
5959
*/
@@ -75,7 +75,7 @@ public function __construct(Account $queueAccount, Account $topicAccount, array
7575
*/
7676
public function isPlain()
7777
{
78-
return (bool)Arr::get($this->plainOptions, 'enable');
78+
return (bool) Arr::get($this->plainOptions, 'enable');
7979
}
8080

8181
/**
@@ -97,18 +97,19 @@ public function size($queue = null)
9797
{
9898
$attributes = $this->getQueue($queue)->get_attributes();
9999

100-
return (int)$attributes->activeMsgNum;
100+
return (int) $attributes->activeMsgNum;
101101
}
102102

103103
/**
104104
* Push a new job onto the queue.
105105
*
106106
* @param string|object $job
107-
* @param mixed $data
108-
* @param string $queue
107+
* @param mixed $data
108+
* @param string $queue
109109
*
110-
* @return mixed
111110
* @throws \Exception
111+
*
112+
* @return mixed
112113
*/
113114
public function push($job, $data = '', $queue = null)
114115
{
@@ -128,12 +129,13 @@ public function push($job, $data = '', $queue = null)
128129
*
129130
* @param string $payload
130131
* @param string $queue
131-
* @param array $options
132+
* @param array $options
132133
*
133-
* @return \Freyo\LaravelQueueCMQ\Queue\Driver\Message|array
134134
* @throws \Freyo\LaravelQueueCMQ\Queue\Driver\CMQServerNetworkException
135135
* @throws \Freyo\LaravelQueueCMQ\Queue\Driver\CMQServerException
136136
* @throws \Exception
137+
*
138+
* @return \Freyo\LaravelQueueCMQ\Queue\Driver\Message|array
137139
*/
138140
public function pushRaw($payload, $queue = null, array $options = [])
139141
{
@@ -155,7 +157,7 @@ function () use ($driver, $message, $queue) {
155157
});
156158
default:
157159
throw new \InvalidArgumentException(
158-
'Invalid CMQ topic filter: ' . $this->topicOptions['filter']
160+
'Invalid CMQ topic filter: '.$this->topicOptions['filter']
159161
);
160162
}
161163
}
@@ -169,12 +171,13 @@ function () use ($driver, $message, $queue) {
169171
* Push a new job onto the queue after a delay.
170172
*
171173
* @param \DateTimeInterface|\DateInterval|int $delay
172-
* @param string|object $job
173-
* @param mixed $data
174-
* @param string $queue
174+
* @param string|object $job
175+
* @param mixed $data
176+
* @param string $queue
175177
*
176-
* @return mixed
177178
* @throws \Exception
179+
*
180+
* @return mixed
178181
*/
179182
public function later($delay, $job, $data = '', $queue = null)
180183
{
@@ -206,9 +209,10 @@ public function pop($queue = null)
206209
$queue = $this->getQueue($queue);
207210
$message = $queue->receive_message($this->queueOptions['polling_wait_seconds']);
208211
} catch (CMQServerException $e) {
209-
if (self::CMQ_QUEUE_NO_MESSAGE_CODE === (int)$e->getCode()) { // ignore no message
210-
return null;
212+
if (self::CMQ_QUEUE_NO_MESSAGE_CODE === (int) $e->getCode()) { // ignore no message
213+
return;
211214
}
215+
212216
throw $e;
213217
}
214218

src/Queue/Connectors/CMQConnector.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,9 @@ class CMQConnector implements ConnectorInterface
1414
*
1515
* @param array $config
1616
*
17-
* @return \Illuminate\Contracts\Queue\Queue
1817
* @throws \ReflectionException
18+
*
19+
* @return \Illuminate\Contracts\Queue\Queue
1920
*/
2021
public function connect(array $config)
2122
{

src/Queue/Jobs/CMQJob.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,11 @@ class CMQJob extends Job implements JobContract
2424
/**
2525
* Create a new job instance.
2626
*
27-
* @param \Illuminate\Container\Container $container
28-
* @param \Freyo\LaravelQueueCMQ\Queue\CMQQueue $connection
27+
* @param \Illuminate\Container\Container $container
28+
* @param \Freyo\LaravelQueueCMQ\Queue\CMQQueue $connection
2929
* @param \Freyo\LaravelQueueCMQ\Queue\Driver\Message $message
30-
* @param \Freyo\LaravelQueueCMQ\Queue\Driver\Queue $queue
31-
* @param string $connectionName
30+
* @param \Freyo\LaravelQueueCMQ\Queue\Driver\Queue $queue
31+
* @param string $connectionName
3232
*/
3333
public function __construct(Container $container, CMQQueue $connection, Message $message, Queue $queue, $connectionName)
3434
{

0 commit comments

Comments
 (0)