Passed
Pull Request — 6.0 (#2079)
by nhzex
06:11
created
src/think/Manager.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 // +----------------------------------------------------------------------
9 9
 // | Author: yunwuxin <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think;
14 14
 
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
     protected function resolveClass(string $type): string
94 94
     {
95 95
         if ($this->namespace || false !== strpos($type, '\\')) {
96
-            $class = false !== strpos($type, '\\') ? $type : $this->namespace . Str::studly($type);
96
+            $class = false !== strpos($type, '\\') ? $type : $this->namespace.Str::studly($type);
97 97
 
98 98
             if (class_exists($class)) {
99 99
                 return $class;
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
     {
126 126
         $type = $this->resolveType($name);
127 127
 
128
-        $method = 'create' . Str::studly($type) . 'Driver';
128
+        $method = 'create'.Str::studly($type).'Driver';
129 129
 
130 130
         $params = $this->resolveParams($name);
131 131
 
Please login to merge, or discard this patch.
src/think/Container.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 // +----------------------------------------------------------------------
9 9
 // | Author: liu21st <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think;
14 14
 
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
             return $this->make($abstract);
131 131
         }
132 132
 
133
-        throw new ClassNotFoundException('class not exists: ' . $abstract, $abstract);
133
+        throw new ClassNotFoundException('class not exists: '.$abstract, $abstract);
134 134
     }
135 135
 
136 136
     /**
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
304 304
     {
305 305
         if (is_array($method)) {
306 306
             [$class, $method] = $method;
307
-            $class   = is_object($class) ? $class : $this->invokeClass($class);
307
+            $class = is_object($class) ? $class : $this->invokeClass($class);
308 308
         } else {
309 309
             // 静态方法
310 310
             [$class, $method] = explode('::', $method);
@@ -373,7 +373,7 @@  discard block
 block discarded – undo
373 373
         try {
374 374
             $reflect = new ReflectionClass($class);
375 375
         } catch (ReflectionException $e) {
376
-            throw new ClassNotFoundException('class not exists: ' . $class, $class, $e);
376
+            throw new ClassNotFoundException('class not exists: '.$class, $class, $e);
377 377
         }
378 378
 
379 379
         if ($reflect->hasMethod('__make')) {
@@ -452,7 +452,7 @@  discard block
 block discarded – undo
452 452
             } elseif ($param->isDefaultValueAvailable()) {
453 453
                 $args[] = $param->getDefaultValue();
454 454
             } else {
455
-                throw new InvalidArgumentException('method param miss:' . $name);
455
+                throw new InvalidArgumentException('method param miss:'.$name);
456 456
             }
457 457
         }
458 458
 
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
      */
471 471
     public static function factory(string $name, string $namespace = '', ...$args)
472 472
     {
473
-        $class = false !== strpos($name, '\\') ? $name : $namespace . ucwords($name);
473
+        $class = false !== strpos($name, '\\') ? $name : $namespace.ucwords($name);
474 474
 
475 475
         return Container::getInstance()->invokeClass($class, $args);
476 476
     }
Please login to merge, or discard this patch.
src/think/exception/FuncNotFoundException.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
     public function __construct(string $message, string $func = '', Throwable $previous = null)
14 14
     {
15 15
         $this->message = $message;
16
-        $this->func   = $func;
16
+        $this->func = $func;
17 17
 
18 18
         parent::__construct($message, 0, $previous);
19 19
     }
Please login to merge, or discard this patch.
tests/ContainerTest.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 
64 64
         Container::setInstance($container);
65 65
 
66
-        $container->bind('name', function () {
66
+        $container->bind('name', function() {
67 67
             return 'Taylor';
68 68
         });
69 69
 
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
         $this->expectExceptionMessage('class not exists: name');
81 81
         $container->get('name');
82 82
 
83
-        $container->bind('name', function () {
83
+        $container->bind('name', function() {
84 84
             return 'Taylor';
85 85
         });
86 86
 
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
     {
92 92
         $container = new Container;
93 93
 
94
-        $container->bind('name', function () {
94
+        $container->bind('name', function() {
95 95
             return 'Taylor';
96 96
         });
97 97
 
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
     {
107 107
         $container = new Container;
108 108
 
109
-        $container->bind('name', function () {
109
+        $container->bind('name', function() {
110 110
             return 'Taylor';
111 111
         });
112 112
 
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
 
199 199
         $this->assertSame($object, Container::getInstance());
200 200
 
201
-        Container::setInstance(function () {
201
+        Container::setInstance(function() {
202 202
             return $this;
203 203
         });
204 204
 
@@ -210,10 +210,10 @@  discard block
 block discarded – undo
210 210
         $container = new Container();
211 211
         $container->bind(Container::class, $container);
212 212
 
213
-        $container->resolving(function (SomeClass $taylor, Container $container) {
213
+        $container->resolving(function(SomeClass $taylor, Container $container) {
214 214
             $taylor->count++;
215 215
         });
216
-        $container->resolving(SomeClass::class, function (SomeClass $taylor, Container $container) {
216
+        $container->resolving(SomeClass::class, function(SomeClass $taylor, Container $container) {
217 217
             $taylor->count++;
218 218
         });
219 219
 
@@ -252,19 +252,19 @@  discard block
 block discarded – undo
252 252
 
253 253
         $this->assertEquals('48', $container->invoke('ord', ['0']));
254 254
 
255
-        $this->assertSame($container, $container->invoke(Taylor::class . '::test', []));
255
+        $this->assertSame($container, $container->invoke(Taylor::class.'::test', []));
256 256
 
257
-        $this->assertSame($container, $container->invokeMethod(Taylor::class . '::test'));
257
+        $this->assertSame($container, $container->invokeMethod(Taylor::class.'::test'));
258 258
 
259 259
         $reflect = new ReflectionMethod($container, 'exists');
260 260
 
261 261
         $this->assertTrue($container->invokeReflectMethod($container, $reflect, [Container::class]));
262 262
 
263
-        $this->assertSame($container, $container->invoke(function (Container $container) {
263
+        $this->assertSame($container, $container->invoke(function(Container $container) {
264 264
             return $container;
265 265
         }));
266 266
 
267
-        $this->assertSame($container, $container->invoke(Taylor::class . '::test'));
267
+        $this->assertSame($container, $container->invoke(Taylor::class.'::test'));
268 268
 
269 269
         $object = $container->invokeClass(SomeClass::class);
270 270
         $this->assertInstanceOf(SomeClass::class, $object);
@@ -272,7 +272,7 @@  discard block
 block discarded – undo
272 272
 
273 273
         $stdClass = new stdClass();
274 274
 
275
-        $container->invoke(function (Container $container, stdClass $stdObject, $key1, $lowKey, $key2 = 'default') use ($stdClass) {
275
+        $container->invoke(function(Container $container, stdClass $stdObject, $key1, $lowKey, $key2 = 'default') use ($stdClass) {
276 276
             $this->assertEquals('value1', $key1);
277 277
             $this->assertEquals('default', $key2);
278 278
             $this->assertEquals('value2', $lowKey);
Please login to merge, or discard this patch.
src/think/middleware/CheckRequestCache.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 // +----------------------------------------------------------------------
9 9
 // | Author: liu21st <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think\middleware;
14 14
 
@@ -88,9 +88,9 @@  discard block
 block discarded – undo
88 88
 
89 89
         if (isset($key) && 200 == $response->getCode() && $response->isAllowCache()) {
90 90
             $header                  = $response->getHeader();
91
-            $header['Cache-Control'] = 'max-age=' . $expire . ',must-revalidate';
92
-            $header['Last-Modified'] = gmdate('D, d M Y H:i:s') . ' GMT';
93
-            $header['Expires']       = gmdate('D, d M Y H:i:s', time() + $expire) . ' GMT';
91
+            $header['Cache-Control'] = 'max-age='.$expire.',must-revalidate';
92
+            $header['Last-Modified'] = gmdate('D, d M Y H:i:s').' GMT';
93
+            $header['Expires']       = gmdate('D, d M Y H:i:s', time() + $expire).' GMT';
94 94
 
95 95
             $this->cache->tag($tag)->set($key, [$response->getContent(), $header, time()], $expire);
96 96
         }
@@ -141,12 +141,12 @@  discard block
 block discarded – undo
141 141
         if (false !== strpos($key, ':')) {
142 142
             $param = $request->param();
143 143
             foreach ($param as $item => $val) {
144
-                if (is_string($val) && false !== strpos($key, ':' . $item)) {
145
-                    $key = str_replace(':' . $item, $val, $key);
144
+                if (is_string($val) && false !== strpos($key, ':'.$item)) {
145
+                    $key = str_replace(':'.$item, $val, $key);
146 146
                 }
147 147
             }
148 148
         } elseif (strpos($key, ']')) {
149
-            if ('[' . $request->ext() . ']' == $key) {
149
+            if ('['.$request->ext().']' == $key) {
150 150
                 // 缓存某个后缀的请求
151 151
                 $key = md5($request->url());
152 152
             } else {
Please login to merge, or discard this patch.
src/think/route/Url.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 // +----------------------------------------------------------------------
9 9
 // | Author: liu21st <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think\route;
14 14
 
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
                 }
191 191
             }
192 192
         } elseif (false === strpos($domain, '.') && 0 !== strpos($domain, $rootDomain)) {
193
-            $domain .= '.' . $rootDomain;
193
+            $domain .= '.'.$rootDomain;
194 194
         }
195 195
 
196 196
         if (false !== strpos($domain, '://')) {
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
             $scheme = $this->https || $request->isSsl() ? 'https://' : 'http://';
200 200
         }
201 201
 
202
-        return $scheme . $domain;
202
+        return $scheme.$domain;
203 203
     }
204 204
 
205 205
     /**
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
             }
219 219
         }
220 220
 
221
-        return (empty($suffix) || 0 === strpos($suffix, '.')) ? (string) $suffix : '.' . $suffix;
221
+        return (empty($suffix) || 0 === strpos($suffix, '.')) ? (string) $suffix : '.'.$suffix;
222 222
     }
223 223
 
224 224
     /**
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
             // 解析到控制器
243 243
             $url = substr($url, 1);
244 244
         } elseif ('' === $url) {
245
-            $url = $request->controller() . '/' . $request->action();
245
+            $url = $request->controller().'/'.$request->action();
246 246
         } else {
247 247
             $controller = $request->controller();
248 248
 
@@ -250,7 +250,7 @@  discard block
 block discarded – undo
250 250
             $action     = array_pop($path);
251 251
             $controller = empty($path) ? $controller : array_pop($path);
252 252
 
253
-            $url = $controller . '/' . $action;
253
+            $url = $controller.'/'.$action;
254 254
         }
255 255
 
256 256
         return $url;
@@ -297,7 +297,7 @@  discard block
 block discarded – undo
297 297
     {
298 298
         $request = $this->app->request;
299 299
         if (is_string($allowDomain) && false === strpos($allowDomain, '.')) {
300
-            $allowDomain .= '.' . $request->rootDomain();
300
+            $allowDomain .= '.'.$request->rootDomain();
301 301
         }
302 302
         $port = $request->port();
303 303
 
@@ -316,7 +316,7 @@  discard block
 block discarded – undo
316 316
             }
317 317
 
318 318
             if ($port && !in_array($port, [80, 443])) {
319
-                $domain .= ':' . $port;
319
+                $domain .= ':'.$port;
320 320
             }
321 321
 
322 322
             if (empty($pattern)) {
@@ -327,12 +327,12 @@  discard block
 block discarded – undo
327 327
 
328 328
             foreach ($pattern as $key => $val) {
329 329
                 if (isset($vars[$key])) {
330
-                    $url = str_replace(['[:' . $key . ']', '<' . $key . '?>', ':' . $key, '<' . $key . '>'], $type ? $vars[$key] : urlencode((string) $vars[$key]), $url);
330
+                    $url = str_replace(['[:'.$key.']', '<'.$key.'?>', ':'.$key, '<'.$key.'>'], $type ? $vars[$key] : urlencode((string) $vars[$key]), $url);
331 331
                     unset($vars[$key]);
332 332
                     $url    = str_replace(['/?', '-?'], ['/', '-'], $url);
333 333
                     $result = [rtrim($url, '?/-'), $domain, $suffix];
334 334
                 } elseif (2 == $val) {
335
-                    $url    = str_replace(['/[:' . $key . ']', '[:' . $key . ']', '<' . $key . '?>'], '', $url);
335
+                    $url    = str_replace(['/[:'.$key.']', '[:'.$key.']', '<'.$key.'?>'], '', $url);
336 336
                     $url    = str_replace(['/?', '-?'], ['/', '-'], $url);
337 337
                     $result = [rtrim($url, '?/-'), $domain, $suffix];
338 338
                 } else {
@@ -360,7 +360,7 @@  discard block
 block discarded – undo
360 360
         if (0 === strpos($url, '[') && $pos = strpos($url, ']')) {
361 361
             // [name] 表示使用路由命名标识生成URL
362 362
             $name = substr($url, 1, $pos - 1);
363
-            $url  = 'name' . substr($url, $pos + 1);
363
+            $url  = 'name'.substr($url, $pos + 1);
364 364
         }
365 365
 
366 366
         if (false === strpos($url, '://') && 0 !== strpos($url, '/')) {
@@ -387,7 +387,7 @@  discard block
 block discarded – undo
387 387
         }
388 388
 
389 389
         if ($url) {
390
-            $checkName   = isset($name) ? $name : $url . (isset($info['query']) ? '?' . $info['query'] : '');
390
+            $checkName   = isset($name) ? $name : $url.(isset($info['query']) ? '?'.$info['query'] : '');
391 391
             $checkDomain = $domain && is_string($domain) ? $domain : null;
392 392
 
393 393
             $rule = $this->route->getName($checkName, $checkDomain);
@@ -413,7 +413,7 @@  discard block
 block discarded – undo
413 413
                 $suffix = $match[2];
414 414
             }
415 415
         } elseif (!empty($rule) && isset($name)) {
416
-            throw new \InvalidArgumentException('route name not exists:' . $name);
416
+            throw new \InvalidArgumentException('route name not exists:'.$name);
417 417
         } else {
418 418
             // 检测URL绑定
419 419
             $bind = $this->route->getDomainBind($domain && is_string($domain) ? $domain : null);
@@ -451,7 +451,7 @@  discard block
 block discarded – undo
451 451
             $file = str_replace('\\', '/', dirname($file));
452 452
         }
453 453
 
454
-        $url = rtrim($file, '/') . '/' . $url;
454
+        $url = rtrim($file, '/').'/'.$url;
455 455
 
456 456
         // URL后缀
457 457
         if ('/' == substr($url, -1) || '' == $url) {
@@ -461,33 +461,33 @@  discard block
 block discarded – undo
461 461
         }
462 462
 
463 463
         // 锚点
464
-        $anchor = !empty($anchor) ? '#' . $anchor : '';
464
+        $anchor = !empty($anchor) ? '#'.$anchor : '';
465 465
 
466 466
         // 参数组装
467 467
         if (!empty($vars)) {
468 468
             // 添加参数
469 469
             if ($this->route->config('url_common_param')) {
470 470
                 $vars = http_build_query($vars);
471
-                $url .= $suffix . '?' . $vars . $anchor;
471
+                $url .= $suffix.'?'.$vars.$anchor;
472 472
             } else {
473 473
                 foreach ($vars as $var => $val) {
474 474
                     $val = (string) $val;
475 475
                     if ('' !== $val) {
476
-                        $url .= $depr . $var . $depr . urlencode($val);
476
+                        $url .= $depr.$var.$depr.urlencode($val);
477 477
                     }
478 478
                 }
479 479
 
480
-                $url .= $suffix . $anchor;
480
+                $url .= $suffix.$anchor;
481 481
             }
482 482
         } else {
483
-            $url .= $suffix . $anchor;
483
+            $url .= $suffix.$anchor;
484 484
         }
485 485
 
486 486
         // 检测域名
487 487
         $domain = $this->parseDomain($url, $domain);
488 488
 
489 489
         // URL组装
490
-        return $domain . rtrim($this->root, '/') . '/' . ltrim($url, '/');
490
+        return $domain.rtrim($this->root, '/').'/'.ltrim($url, '/');
491 491
     }
492 492
 
493 493
     public function __toString()
Please login to merge, or discard this patch.
src/think/Response.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 // +----------------------------------------------------------------------
9 9
 // | Author: liu21st <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think;
14 14
 
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
      */
105 105
     public static function create($data = '', string $type = 'html', int $code = 200): Response
106 106
     {
107
-        $class = false !== strpos($type, '\\') ? $type : '\\think\\response\\' . ucfirst(strtolower($type));
107
+        $class = false !== strpos($type, '\\') ? $type : '\\think\\response\\'.ucfirst(strtolower($type));
108 108
 
109 109
         return Container::getInstance()->invokeClass($class, [$data, $code]);
110 110
     }
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
             http_response_code($this->code);
150 150
             // 发送头部信息
151 151
             foreach ($this->header as $name => $val) {
152
-                header($name . (!is_null($val) ? ':' . $val : ''));
152
+                header($name.(!is_null($val) ? ':'.$val : ''));
153 153
             }
154 154
         }
155 155
 
@@ -357,7 +357,7 @@  discard block
 block discarded – undo
357 357
      */
358 358
     public function contentType(string $contentType, string $charset = 'utf-8')
359 359
     {
360
-        $this->header['Content-Type'] = $contentType . '; charset=' . $charset;
360
+        $this->header['Content-Type'] = $contentType.'; charset='.$charset;
361 361
 
362 362
         return $this;
363 363
     }
Please login to merge, or discard this patch.
src/think/response/Html.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@
 block discarded – undo
8 8
 // +----------------------------------------------------------------------
9 9
 // | Author: liu21st <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think\response;
14 14
 
Please login to merge, or discard this patch.
src/think/session/Store.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -118,7 +118,7 @@
 block discarded – undo
118 118
      */
119 119
     public function setId($id = null): void
120 120
     {
121
-        $this->id = is_string($id) && strlen($id) === 32 ? $id : md5(microtime(true) . session_create_id());
121
+        $this->id = is_string($id) && strlen($id) === 32 ? $id : md5(microtime(true).session_create_id());
122 122
     }
123 123
 
124 124
     /**
Please login to merge, or discard this patch.