@@ -6,5 +6,8 @@ |
||
6 | 6 | |
7 | 7 | interface DriverInterface |
8 | 8 | { |
9 | + /** |
|
10 | + * @return string |
|
11 | + */ |
|
9 | 12 | public function commands(Request $request); |
10 | 13 | } |
11 | 14 | \ No newline at end of file |
@@ -5,4 +5,5 @@ |
||
5 | 5 | use Clue\Redis\Protocol\Model\ModelInterface as ClueModelInterface; |
6 | 6 | |
7 | 7 | interface RequestInterface extends ClueModelInterface |
8 | -{} |
|
8 | +{ |
|
9 | +} |
@@ -128,9 +128,11 @@ discard block |
||
128 | 128 | $stream = $this->createClient($this->endpoint); |
129 | 129 | } |
130 | 130 | catch (Error $ex) |
131 | - {} |
|
131 | + { |
|
132 | +} |
|
132 | 133 | catch (Exception $ex) |
133 | - {} |
|
134 | + { |
|
135 | +} |
|
134 | 136 | |
135 | 137 | if ($ex !== null) |
136 | 138 | { |
@@ -320,9 +322,11 @@ discard block |
||
320 | 322 | return new Socket($endpoint, $this->loop); |
321 | 323 | } |
322 | 324 | catch (Error $ex) |
323 | - {} |
|
325 | + { |
|
326 | +} |
|
324 | 327 | catch (Exception $ex) |
325 | - {} |
|
328 | + { |
|
329 | +} |
|
326 | 330 | |
327 | 331 | throw new ExecutionException('Redis connection socket could not be created!', 0, $ex); |
328 | 332 | } |
@@ -149,7 +149,7 @@ discard block |
||
149 | 149 | $this->isBeingDisconnected = false; |
150 | 150 | $this->stream = $stream; |
151 | 151 | $this->handleStart(); |
152 | - $this->emit('start', [ $this ]); |
|
152 | + $this->emit('start', [$this]); |
|
153 | 153 | |
154 | 154 | return Promise::doResolve($this); |
155 | 155 | } |
@@ -178,7 +178,7 @@ discard block |
||
178 | 178 | |
179 | 179 | $this->reqs = []; |
180 | 180 | $this->handleStop(); |
181 | - $this->emit('stop', [ $this ]); |
|
181 | + $this->emit('stop', [$this]); |
|
182 | 182 | |
183 | 183 | if ($this->endPromise !== null) |
184 | 184 | { |
@@ -243,8 +243,8 @@ discard block |
||
243 | 243 | { |
244 | 244 | if ($this->stream !== null) |
245 | 245 | { |
246 | - $this->stream->on('data', [ $this, 'handleData' ]); |
|
247 | - $this->stream->on('close', [ $this, 'stop' ]); |
|
246 | + $this->stream->on('data', [$this, 'handleData']); |
|
247 | + $this->stream->on('close', [$this, 'stop']); |
|
248 | 248 | } |
249 | 249 | } |
250 | 250 | |
@@ -255,8 +255,8 @@ discard block |
||
255 | 255 | { |
256 | 256 | if ($this->stream !== null) |
257 | 257 | { |
258 | - $this->stream->removeListener('data', [ $this, 'handleData' ]); |
|
259 | - $this->stream->removeListener('close', [ $this, 'stop' ]); |
|
258 | + $this->stream->removeListener('data', [$this, 'handleData']); |
|
259 | + $this->stream->removeListener('close', [$this, 'stop']); |
|
260 | 260 | } |
261 | 261 | } |
262 | 262 | |
@@ -273,7 +273,7 @@ discard block |
||
273 | 273 | } |
274 | 274 | catch (ParserException $error) |
275 | 275 | { |
276 | - $this->emit('error', [ $this, $error ]); |
|
276 | + $this->emit('error', [$this, $error]); |
|
277 | 277 | $this->stop(); |
278 | 278 | return; |
279 | 279 | } |
@@ -286,7 +286,7 @@ discard block |
||
286 | 286 | } |
287 | 287 | catch (UnderflowException $error) |
288 | 288 | { |
289 | - $this->emit('error', [ $this, $error ]); |
|
289 | + $this->emit('error', [$this, $error]); |
|
290 | 290 | $this->stop(); |
291 | 291 | return; |
292 | 292 | } |
@@ -18,7 +18,7 @@ |
||
18 | 18 | * @param string $message |
19 | 19 | * @return mixed |
20 | 20 | */ |
21 | - public function ping($message='pong'); |
|
21 | + public function ping($message = 'pong'); |
|
22 | 22 | |
23 | 23 | /** |
24 | 24 | * @doc https://redis.io/commands/quit |
@@ -48,13 +48,13 @@ |
||
48 | 48 | $command = Enum::HGETALL; |
49 | 49 | $args = [$key]; |
50 | 50 | |
51 | - return $this->dispatch(Builder::build($command, $args))->then(function ($value) { |
|
51 | + return $this->dispatch(Builder::build($command, $args))->then(function($value) { |
|
52 | 52 | if (!empty($value)) { |
53 | 53 | $tmp = []; |
54 | 54 | $size = count($value); |
55 | - for ($i=0; $i<$size; $i+=2) { |
|
55 | + for ($i = 0; $i < $size; $i += 2) { |
|
56 | 56 | $field = $value[$i]; |
57 | - $val = $value[$i+1]; |
|
57 | + $val = $value[$i + 1]; |
|
58 | 58 | $tmp[$field] = $val; |
59 | 59 | } |
60 | 60 | $value = $tmp; |
@@ -49,10 +49,12 @@ discard block |
||
49 | 49 | $args = [$key]; |
50 | 50 | |
51 | 51 | return $this->dispatch(Builder::build($command, $args))->then(function ($value) { |
52 | - if (!empty($value)) { |
|
52 | + if (!empty($value)) |
|
53 | + { |
|
53 | 54 | $tmp = []; |
54 | 55 | $size = count($value); |
55 | - for ($i=0; $i<$size; $i+=2) { |
|
56 | + for ($i=0; $i<$size; $i+=2) |
|
57 | + { |
|
56 | 58 | $field = $value[$i]; |
57 | 59 | $val = $value[$i+1]; |
58 | 60 | $tmp[$field] = $val; |
@@ -134,8 +136,10 @@ discard block |
||
134 | 136 | //TODO: replace param $fvMap to ...$fvs,cauz hash map not allow duplicate key |
135 | 137 | $command = Enum::HMSET; |
136 | 138 | $args = [$key]; |
137 | - if (!empty($fvMap)) { |
|
138 | - foreach ($fvMap as $field => $value) { |
|
139 | + if (!empty($fvMap)) |
|
140 | + { |
|
141 | + foreach ($fvMap as $field => $value) |
|
142 | + { |
|
139 | 143 | $tmp[] = $field; |
140 | 144 | $tmp[] = $value; |
141 | 145 | } |
@@ -60,7 +60,7 @@ |
||
60 | 60 | { |
61 | 61 | // TODO: Implement geoDist() method. |
62 | 62 | $command = Enum::GEODIST; |
63 | - $args = [$key, $memberA, $memberB ,$unit]; |
|
63 | + $args = [$key, $memberA, $memberB, $unit]; |
|
64 | 64 | |
65 | 65 | return $this->dispatch(Builder::build($command, $args)); |
66 | 66 | } |
@@ -347,7 +347,7 @@ |
||
347 | 347 | * @override |
348 | 348 | * @inheritDoc |
349 | 349 | */ |
350 | - public function del($key,...$keys) |
|
350 | + public function del($key, ...$keys) |
|
351 | 351 | { |
352 | 352 | $command = Enum::DEL; |
353 | 353 | $keys[] = $key; |
@@ -47,7 +47,8 @@ discard block |
||
47 | 47 | $command = Enum::BITFIELD; |
48 | 48 | $subCommand = strtoupper($subCommand); |
49 | 49 | //TODO: control flow improvement |
50 | - switch ($subCommand) { |
|
50 | + switch ($subCommand) |
|
51 | + { |
|
51 | 52 | case 'GET' : { |
52 | 53 | @list ($type, $offset) = $param; |
53 | 54 | $args = [$key, $subCommand, $type, $offset]; |
@@ -305,8 +306,10 @@ discard block |
||
305 | 306 | //TODO: change the param $kvMap to ...$kv,cauz map not allow duplicate key |
306 | 307 | $command = Enum::MSET; |
307 | 308 | $args = []; |
308 | - if (!empty($kvMap)) { |
|
309 | - foreach ($kvMap as $key => $val) { |
|
309 | + if (!empty($kvMap)) |
|
310 | + { |
|
311 | + foreach ($kvMap as $key => $val) |
|
312 | + { |
|
310 | 313 | $args[] = $key; |
311 | 314 | $args[] = $val; |
312 | 315 | } |
@@ -323,8 +326,10 @@ discard block |
||
323 | 326 | { |
324 | 327 | $command = Enum::MSETNX; |
325 | 328 | $args = []; |
326 | - if (!empty($kvMap)) { |
|
327 | - foreach ($kvMap as $key => $val) { |
|
329 | + if (!empty($kvMap)) |
|
330 | + { |
|
331 | + foreach ($kvMap as $key => $val) |
|
332 | + { |
|
328 | 333 | $args[] = $key; |
329 | 334 | $args[] = $val; |
330 | 335 | } |
@@ -25,9 +25,9 @@ discard block |
||
25 | 25 | $keys[] = $timeout; |
26 | 26 | $args = $keys; |
27 | 27 | $promise = $this->dispatch(Builder::build($command, $args)); |
28 | - $promise = $promise->then(function ($value) { |
|
28 | + $promise = $promise->then(function($value) { |
|
29 | 29 | if (is_array($value)) { |
30 | - list($k,$v) = $value; |
|
30 | + list($k, $v) = $value; |
|
31 | 31 | |
32 | 32 | return [ |
33 | 33 | 'key'=>$k, |
@@ -52,9 +52,9 @@ discard block |
||
52 | 52 | $keys[] = $timeout; |
53 | 53 | $args = $keys; |
54 | 54 | $promise = $this->dispatch(Builder::build($command, $args)); |
55 | - $promise = $promise->then(function ($value) { |
|
55 | + $promise = $promise->then(function($value) { |
|
56 | 56 | if (is_array($value)) { |
57 | - list($k,$v) = $value; |
|
57 | + list($k, $v) = $value; |
|
58 | 58 | |
59 | 59 | return [ |
60 | 60 | 'key'=>$k, |
@@ -137,7 +137,7 @@ discard block |
||
137 | 137 | * @override |
138 | 138 | * @inheritDoc |
139 | 139 | */ |
140 | - public function lPush($key,...$values) |
|
140 | + public function lPush($key, ...$values) |
|
141 | 141 | { |
142 | 142 | $command = Enum::LPUSH; |
143 | 143 | array_unshift($values, $key); |
@@ -26,7 +26,8 @@ discard block |
||
26 | 26 | $args = $keys; |
27 | 27 | $promise = $this->dispatch(Builder::build($command, $args)); |
28 | 28 | $promise = $promise->then(function ($value) { |
29 | - if (is_array($value)) { |
|
29 | + if (is_array($value)) |
|
30 | + { |
|
30 | 31 | list($k,$v) = $value; |
31 | 32 | |
32 | 33 | return [ |
@@ -53,7 +54,8 @@ discard block |
||
53 | 54 | $args = $keys; |
54 | 55 | $promise = $this->dispatch(Builder::build($command, $args)); |
55 | 56 | $promise = $promise->then(function ($value) { |
56 | - if (is_array($value)) { |
|
57 | + if (is_array($value)) |
|
58 | + { |
|
57 | 59 | list($k,$v) = $value; |
58 | 60 | |
59 | 61 | return [ |
@@ -66,7 +66,6 @@ |
||
66 | 66 | * @param $key |
67 | 67 | * @param $star |
68 | 68 | * @param $stop |
69 | - * @param array $options |
|
70 | 69 | * @return mixed |
71 | 70 | */ |
72 | 71 | public function zRange($key, $star = 0, $stop = -1, $withScores = false); |