@@ -22,12 +22,20 @@ class CMQQueue extends Queue implements QueueContract
2222 * @var array
2323 */
2424 protected $ queueOptions ;
25+
26+ /**
27+ * @var array
28+ */
2529 protected $ topicOptions ;
2630
2731 /**
2832 * @var Account
2933 */
3034 private $ queueAccount ;
35+
36+ /**
37+ * @var Account
38+ */
3139 private $ topicAccount ;
3240
3341 /**
@@ -42,9 +50,11 @@ class CMQQueue extends Queue implements QueueContract
4250
4351 /**
4452 * CMQQueue constructor.
53+ *
4554 * @param Account $queueAccount
4655 * @param Account $topicAccount
4756 * @param array $config
57+ *
4858 * @throws \ReflectionException
4959 */
5060 public function __construct (Account $ queueAccount , Account $ topicAccount , array $ config )
@@ -65,7 +75,7 @@ public function __construct(Account $queueAccount, Account $topicAccount, array
6575 */
6676 public function isPlain ()
6777 {
68- return Arr::get ($ this ->plainOptions , 'enable ' , false );
78+ return ( bool ) Arr::get ($ this ->plainOptions , 'enable ' );
6979 }
7080
7181 /**
@@ -106,11 +116,9 @@ public function push($job, $data = '', $queue = null)
106116 return $ this ->pushRaw ($ job ->getPayload (), $ queue );
107117 }
108118
109- if (self ::$ createPayload ->getNumberOfParameters () === 3 ) { // version >= 5.7
110- $ payload = $ this ->createPayload ($ job , $ queue , $ data );
111- } else {
112- $ payload = $ this ->createPayload ($ job , $ data );
113- }
119+ $ payload = self ::$ createPayload ->getNumberOfParameters () === 3
120+ ? $ this ->createPayload ($ job , $ queue , $ data ) // version >= 5.7
121+ : $ this ->createPayload ($ job , $ data );
114122
115123 return $ this ->pushRaw ($ payload , $ queue );
116124 }
@@ -178,11 +186,9 @@ public function later($delay, $job, $data = '', $queue = null)
178186 return $ this ->pushRaw ($ job ->getPayload (), $ queue , ['delay ' => $ delay ]);
179187 }
180188
181- if (self ::$ createPayload ->getNumberOfParameters () === 3 ) { // version >= 5.7
182- $ payload = $ this ->createPayload ($ job , $ queue , $ data );
183- } else {
184- $ payload = $ this ->createPayload ($ job , $ data );
185- }
189+ $ payload = self ::$ createPayload ->getNumberOfParameters () === 3
190+ ? $ this ->createPayload ($ job , $ queue , $ data ) // version >= 5.7
191+ : $ this ->createPayload ($ job , $ data );
186192
187193 return $ this ->pushRaw ($ payload , $ queue , ['delay ' => $ delay ]);
188194 }
@@ -200,7 +206,7 @@ public function pop($queue = null)
200206 $ queue = $ this ->getQueue ($ queue );
201207 $ message = $ queue ->receive_message ($ this ->queueOptions ['polling_wait_seconds ' ]);
202208 } catch (CMQServerException $ e ) {
203- if (self ::CMQ_QUEUE_NO_MESSAGE_CODE === (int )$ e ->getCode ()) { //ignore no message
209+ if (self ::CMQ_QUEUE_NO_MESSAGE_CODE === (int )$ e ->getCode ()) { // ignore no message
204210 return null ;
205211 }
206212 throw $ e ;
0 commit comments