Completed
Pull Request — 6.0 (#1921)
by nhzex
04:55
created
src/think/Log.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: liu21st <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think;
14 14
 
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
     public function getConfig(string $name = null, $default = null)
57 57
     {
58 58
         if (!is_null($name)) {
59
-            return $this->app->config->get('log.' . $name, $default);
59
+            return $this->app->config->get('log.'.$name, $default);
60 60
         }
61 61
 
62 62
         return $this->app->config->get('log');
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
      */
181 181
     public function record($msg, string $type = 'info', array $context = [], bool $lazy = true)
182 182
     {
183
-        $channel = $this->getConfig('type_channel.' . $type);
183
+        $channel = $this->getConfig('type_channel.'.$type);
184 184
 
185 185
         $this->channel($channel)->record($msg, $type, $context, $lazy);
186 186
 
Please login to merge, or discard this patch.
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;
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
         $type   = $this->resolveType($name);
116 116
         $config = $this->resolveConfig($name);
117 117
 
118
-        $method = 'create' . Str::studly($type) . 'Driver';
118
+        $method = 'create'.Str::studly($type).'Driver';
119 119
 
120 120
         if (method_exists($this, $method)) {
121 121
             return $this->$method($config);
Please login to merge, or discard this patch.
src/think/log/driver/File.php 1 patch
Spacing   +14 added lines, -15 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\log\driver;
14 14
 
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
         }
49 49
 
50 50
         if (empty($this->config['path'])) {
51
-            $this->config['path'] = $app->getRuntimePath() . 'log';
51
+            $this->config['path'] = $app->getRuntimePath().'log';
52 52
         }
53 53
 
54 54
         if (substr($this->config['path'], -1) != DIRECTORY_SEPARATOR) {
@@ -82,8 +82,7 @@  discard block
 block discarded – undo
82 82
                 }
83 83
 
84 84
                 $message[] = $this->config['json'] ?
85
-                    json_encode(['time' => $time, 'type' => $type, 'msg' => $msg], $this->config['json_options']) :
86
-                    sprintf($this->config['format'], $time, $type, $msg);
85
+                    json_encode(['time' => $time, 'type' => $type, 'msg' => $msg], $this->config['json_options']) : sprintf($this->config['format'], $time, $type, $msg);
87 86
             }
88 87
 
89 88
             if (true === $this->config['apart_level'] || in_array($type, $this->config['apart_level'])) {
@@ -121,7 +120,7 @@  discard block
 block discarded – undo
121 120
             $info[$type] = is_array($msg) ? implode(PHP_EOL, $msg) : $msg;
122 121
         }
123 122
 
124
-        $message = implode(PHP_EOL, $info) . PHP_EOL;
123
+        $message = implode(PHP_EOL, $info).PHP_EOL;
125 124
 
126 125
         return error_log($message, 3, $destination);
127 126
     }
@@ -135,7 +134,7 @@  discard block
 block discarded – undo
135 134
     {
136 135
 
137 136
         if ($this->config['max_files']) {
138
-            $files = glob($this->config['path'] . '*.log');
137
+            $files = glob($this->config['path'].'*.log');
139 138
 
140 139
             try {
141 140
                 if (count($files) > $this->config['max_files']) {
@@ -148,16 +147,16 @@  discard block
 block discarded – undo
148 147
 
149 148
         if ($this->config['single']) {
150 149
             $name        = is_string($this->config['single']) ? $this->config['single'] : 'single';
151
-            $destination = $this->config['path'] . $name . '.log';
150
+            $destination = $this->config['path'].$name.'.log';
152 151
         } else {
153 152
 
154 153
             if ($this->config['max_files']) {
155
-                $filename = date('Ymd') . '.log';
154
+                $filename = date('Ymd').'.log';
156 155
             } else {
157
-                $filename = date('Ym') . DIRECTORY_SEPARATOR . date('d') . '.log';
156
+                $filename = date('Ym').DIRECTORY_SEPARATOR.date('d').'.log';
158 157
             }
159 158
 
160
-            $destination = $this->config['path'] . $filename;
159
+            $destination = $this->config['path'].$filename;
161 160
         }
162 161
 
163 162
         return $destination;
@@ -176,14 +175,14 @@  discard block
 block discarded – undo
176 175
         if ($this->config['single']) {
177 176
             $name = is_string($this->config['single']) ? $this->config['single'] : 'single';
178 177
 
179
-            $name .= '_' . $type;
178
+            $name .= '_'.$type;
180 179
         } elseif ($this->config['max_files']) {
181
-            $name = date('Ymd') . '_' . $type;
180
+            $name = date('Ymd').'_'.$type;
182 181
         } else {
183
-            $name = date('d') . '_' . $type;
182
+            $name = date('d').'_'.$type;
184 183
         }
185 184
 
186
-        return $path . DIRECTORY_SEPARATOR . $name . '.log';
185
+        return $path.DIRECTORY_SEPARATOR.$name.'.log';
187 186
     }
188 187
 
189 188
     /**
@@ -196,7 +195,7 @@  discard block
 block discarded – undo
196 195
     {
197 196
         if (is_file($destination) && floor($this->config['file_size']) <= filesize($destination)) {
198 197
             try {
199
-                rename($destination, dirname($destination) . DIRECTORY_SEPARATOR . time() . '-' . basename($destination));
198
+                rename($destination, dirname($destination).DIRECTORY_SEPARATOR.time().'-'.basename($destination));
200 199
             } catch (\Exception $e) {
201 200
                 //
202 201
             }
Please login to merge, or discard this patch.
src/think/Cache.php 1 patch
Spacing   +2 added lines, -2 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
 
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
     public function getConfig(string $name = null, $default = null)
47 47
     {
48 48
         if (!is_null($name)) {
49
-            return $this->app->config->get('cache.' . $name, $default);
49
+            return $this->app->config->get('cache.'.$name, $default);
50 50
         }
51 51
 
52 52
         return $this->app->config->get('cache');
Please login to merge, or discard this patch.
src/think/cache/Driver.php 1 patch
Spacing   +8 added lines, -8 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\cache;
14 14
 
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
      */
85 85
     public function getCacheKey(string $name): string
86 86
     {
87
-        return $this->options['prefix'] . $name;
87
+        return $this->options['prefix'].$name;
88 88
     }
89 89
 
90 90
     /**
@@ -145,14 +145,14 @@  discard block
 block discarded – undo
145 145
 
146 146
         $time = time();
147 147
 
148
-        while ($time + 5 > time() && $this->has($name . '_lock')) {
148
+        while ($time + 5 > time() && $this->has($name.'_lock')) {
149 149
             // 存在锁定则等待
150 150
             usleep(200000);
151 151
         }
152 152
 
153 153
         try {
154 154
             // 锁定
155
-            $this->set($name . '_lock', true);
155
+            $this->set($name.'_lock', true);
156 156
 
157 157
             if ($value instanceof Closure) {
158 158
                 // 获取缓存数据
@@ -163,9 +163,9 @@  discard block
 block discarded – undo
163 163
             $this->set($name, $value, $expire);
164 164
 
165 165
             // 解锁
166
-            $this->delete($name . '_lock');
166
+            $this->delete($name.'_lock');
167 167
         } catch (Exception | throwable $e) {
168
-            $this->delete($name . '_lock');
168
+            $this->delete($name.'_lock');
169 169
             throw $e;
170 170
         }
171 171
 
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
         $key  = implode('-', $name);
185 185
 
186 186
         if (!isset($this->tag[$key])) {
187
-            $name            = array_map(function ($val) {
187
+            $name            = array_map(function($val) {
188 188
                 return $this->getTagKey($val);
189 189
             }, $name);
190 190
             $this->tag[$key] = new TagSet($name, $this);
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
      */
213 213
     public function getTagKey(string $tag): string
214 214
     {
215
-        return $this->options['tag_prefix'] . md5($tag);
215
+        return $this->options['tag_prefix'].md5($tag);
216 216
     }
217 217
 
218 218
     /**
Please login to merge, or discard this patch.
src/think/cache/driver/File.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\cache\driver;
14 14
 
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
         }
49 49
 
50 50
         if (empty($this->options['path'])) {
51
-            $this->options['path'] = $app->getRuntimePath() . 'cache';
51
+            $this->options['path'] = $app->getRuntimePath().'cache';
52 52
         }
53 53
 
54 54
         if (substr($this->options['path'], -1) != DIRECTORY_SEPARATOR) {
@@ -68,14 +68,14 @@  discard block
 block discarded – undo
68 68
 
69 69
         if ($this->options['cache_subdir']) {
70 70
             // 使用子目录
71
-            $name = substr($name, 0, 2) . DIRECTORY_SEPARATOR . substr($name, 2);
71
+            $name = substr($name, 0, 2).DIRECTORY_SEPARATOR.substr($name, 2);
72 72
         }
73 73
 
74 74
         if ($this->options['prefix']) {
75
-            $name = $this->options['prefix'] . DIRECTORY_SEPARATOR . $name;
75
+            $name = $this->options['prefix'].DIRECTORY_SEPARATOR.$name;
76 76
         }
77 77
 
78
-        return $this->options['path'] . $name . '.php';
78
+        return $this->options['path'].$name.'.php';
79 79
     }
80 80
 
81 81
     /**
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
             $data = gzcompress($data, 3);
182 182
         }
183 183
 
184
-        $data   = "<?php\n//" . sprintf('%012d', $expire) . "\n exit();?>\n" . $data;
184
+        $data   = "<?php\n//".sprintf('%012d', $expire)."\n exit();?>\n".$data;
185 185
         $result = file_put_contents($filename, $data);
186 186
 
187 187
         if ($result) {
@@ -250,7 +250,7 @@  discard block
 block discarded – undo
250 250
     {
251 251
         $this->writeTimes++;
252 252
 
253
-        $dirname = $this->options['path'] . $this->options['prefix'];
253
+        $dirname = $this->options['path'].$this->options['prefix'];
254 254
 
255 255
         $this->rmdir($dirname);
256 256
 
Please login to merge, or discard this patch.
src/think/cache/driver/Memcache.php 1 patch
Spacing   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -63,8 +63,7 @@  discard block
 block discarded – undo
63 63
         foreach ($hosts as $i => $host) {
64 64
             $port = $ports[$i] ?? $ports[0];
65 65
             $this->options['timeout'] > 0 ?
66
-                $this->handler->addServer($host, (int) $port, $this->options['persistent'], 1, $this->options['timeout']) :
67
-                $this->handler->addServer($host, (int) $port, $this->options['persistent'], 1);
66
+                $this->handler->addServer($host, (int) $port, $this->options['persistent'], 1, $this->options['timeout']) : $this->handler->addServer($host, (int) $port, $this->options['persistent'], 1);
68 67
         }
69 68
     }
70 69
 
@@ -176,8 +175,7 @@  discard block
 block discarded – undo
176 175
         $key = $this->getCacheKey($name);
177 176
 
178 177
         return false === $ttl ?
179
-            $this->handler->delete($key) :
180
-            $this->handler->delete($key, $ttl);
178
+            $this->handler->delete($key) : $this->handler->delete($key, $ttl);
181 179
     }
182 180
 
183 181
     /**
Please login to merge, or discard this patch.
tests/CacheTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -118,10 +118,10 @@
 block discarded – undo
118 118
         $redis->shouldReceive("del")->once()->with('baz')->andReturnTrue();
119 119
         $redis->shouldReceive("flushDB")->once()->andReturnTrue();
120 120
         $redis->shouldReceive("set")->once()->with('bar', \Opis\Closure\serialize('foobar'))->andReturnTrue();
121
-        $redis->shouldReceive("sAdd")->once()->with('tag:' . md5('foo'), 'bar')->andReturnTrue();
122
-        $redis->shouldReceive("sMembers")->once()->with('tag:' . md5('foo'))->andReturn(['bar']);
121
+        $redis->shouldReceive("sAdd")->once()->with('tag:'.md5('foo'), 'bar')->andReturnTrue();
122
+        $redis->shouldReceive("sMembers")->once()->with('tag:'.md5('foo'))->andReturn(['bar']);
123 123
         $redis->shouldReceive("del")->once()->with(['bar'])->andReturnTrue();
124
-        $redis->shouldReceive("del")->once()->with('tag:' . md5('foo'))->andReturnTrue();
124
+        $redis->shouldReceive("del")->once()->with('tag:'.md5('foo'))->andReturnTrue();
125 125
 
126 126
         $this->cache->set('foo', 5);
127 127
         $this->cache->inc('foo');
Please login to merge, or discard this patch.
src/think/App.php 1 patch
Spacing   +26 added lines, -26 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
 
@@ -155,13 +155,13 @@  discard block
 block discarded – undo
155 155
      */
156 156
     public function __construct(string $rootPath = '')
157 157
     {
158
-        $this->thinkPath   = dirname(__DIR__) . DIRECTORY_SEPARATOR;
159
-        $this->rootPath    = $rootPath ? rtrim($rootPath, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR : $this->getDefaultRootPath();
160
-        $this->appPath     = $this->rootPath . 'app' . DIRECTORY_SEPARATOR;
161
-        $this->runtimePath = $this->rootPath . 'runtime' . DIRECTORY_SEPARATOR;
158
+        $this->thinkPath   = dirname(__DIR__).DIRECTORY_SEPARATOR;
159
+        $this->rootPath    = $rootPath ? rtrim($rootPath, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR : $this->getDefaultRootPath();
160
+        $this->appPath     = $this->rootPath.'app'.DIRECTORY_SEPARATOR;
161
+        $this->runtimePath = $this->rootPath.'runtime'.DIRECTORY_SEPARATOR;
162 162
 
163
-        if (is_file($this->appPath . 'provider.php')) {
164
-            $this->bind(include $this->appPath . 'provider.php');
163
+        if (is_file($this->appPath.'provider.php')) {
164
+            $this->bind(include $this->appPath.'provider.php');
165 165
         }
166 166
 
167 167
         static::setInstance($this);
@@ -221,7 +221,7 @@  discard block
 block discarded – undo
221 221
     public function getService($service)
222 222
     {
223 223
         $name = is_string($service) ? $service : get_class($service);
224
-        return array_values(array_filter($this->services, function ($value) use ($name) {
224
+        return array_values(array_filter($this->services, function($value) use ($name) {
225 225
             return $value instanceof $name;
226 226
         }, ARRAY_FILTER_USE_BOTH))[0] ?? null;
227 227
     }
@@ -297,7 +297,7 @@  discard block
 block discarded – undo
297 297
      */
298 298
     public function getBasePath(): string
299 299
     {
300
-        return $this->rootPath . 'app' . DIRECTORY_SEPARATOR;
300
+        return $this->rootPath.'app'.DIRECTORY_SEPARATOR;
301 301
     }
302 302
 
303 303
     /**
@@ -355,7 +355,7 @@  discard block
 block discarded – undo
355 355
      */
356 356
     public function getConfigPath(): string
357 357
     {
358
-        return $this->rootPath . 'config' . DIRECTORY_SEPARATOR;
358
+        return $this->rootPath.'config'.DIRECTORY_SEPARATOR;
359 359
     }
360 360
 
361 361
     /**
@@ -401,8 +401,8 @@  discard block
 block discarded – undo
401 401
         $this->beginMem  = memory_get_usage();
402 402
 
403 403
         // 加载环境变量
404
-        if (is_file($this->rootPath . '.env')) {
405
-            $this->env->load($this->rootPath . '.env');
404
+        if (is_file($this->rootPath.'.env')) {
405
+            $this->env->load($this->rootPath.'.env');
406 406
         }
407 407
 
408 408
         $this->configExt = $this->env->get('config_ext', '.php');
@@ -415,7 +415,7 @@  discard block
 block discarded – undo
415 415
         // 加载框架默认语言包
416 416
         $langSet = $this->lang->defaultLangSet();
417 417
 
418
-        $this->lang->load($this->thinkPath . 'lang' . DIRECTORY_SEPARATOR . $langSet . '.php');
418
+        $this->lang->load($this->thinkPath.'lang'.DIRECTORY_SEPARATOR.$langSet.'.php');
419 419
 
420 420
         // 加载应用默认语言包
421 421
         $this->loadLangPack($langSet);
@@ -454,7 +454,7 @@  discard block
 block discarded – undo
454 454
         }
455 455
 
456 456
         // 加载系统语言包
457
-        $files = glob($this->appPath . 'lang' . DIRECTORY_SEPARATOR . $langset . '.*');
457
+        $files = glob($this->appPath.'lang'.DIRECTORY_SEPARATOR.$langset.'.*');
458 458
         $this->lang->load($files);
459 459
 
460 460
         // 加载扩展(自定义)语言包
@@ -472,7 +472,7 @@  discard block
 block discarded – undo
472 472
      */
473 473
     public function boot(): void
474 474
     {
475
-        array_walk($this->services, function ($service) {
475
+        array_walk($this->services, function($service) {
476 476
             $this->bootService($service);
477 477
         });
478 478
     }
@@ -486,30 +486,30 @@  discard block
 block discarded – undo
486 486
     {
487 487
         $appPath = $this->getAppPath();
488 488
 
489
-        if (is_file($appPath . 'common.php')) {
490
-            include_once $appPath . 'common.php';
489
+        if (is_file($appPath.'common.php')) {
490
+            include_once $appPath.'common.php';
491 491
         }
492 492
 
493
-        include_once $this->thinkPath . 'helper.php';
493
+        include_once $this->thinkPath.'helper.php';
494 494
 
495 495
         $configPath = $this->getConfigPath();
496 496
 
497 497
         $files = [];
498 498
 
499 499
         if (is_dir($configPath)) {
500
-            $files = glob($configPath . '*' . $this->configExt);
500
+            $files = glob($configPath.'*'.$this->configExt);
501 501
         }
502 502
 
503 503
         foreach ($files as $file) {
504 504
             $this->config->load($file, pathinfo($file, PATHINFO_FILENAME));
505 505
         }
506 506
 
507
-        if (is_file($appPath . 'event.php')) {
508
-            $this->loadEvent(include $appPath . 'event.php');
507
+        if (is_file($appPath.'event.php')) {
508
+            $this->loadEvent(include $appPath.'event.php');
509 509
         }
510 510
 
511
-        if (is_file($appPath . 'service.php')) {
512
-            $services = include $appPath . 'service.php';
511
+        if (is_file($appPath.'service.php')) {
512
+            $services = include $appPath.'service.php';
513 513
             foreach ($services as $service) {
514 514
                 $this->register($service);
515 515
             }
@@ -574,9 +574,9 @@  discard block
 block discarded – undo
574 574
         $name  = str_replace(['/', '.'], '\\', $name);
575 575
         $array = explode('\\', $name);
576 576
         $class = Str::studly(array_pop($array));
577
-        $path  = $array ? implode('\\', $array) . '\\' : '';
577
+        $path  = $array ? implode('\\', $array).'\\' : '';
578 578
 
579
-        return $this->namespace . '\\' . $layer . '\\' . $path . $class;
579
+        return $this->namespace.'\\'.$layer.'\\'.$path.$class;
580 580
     }
581 581
 
582 582
     /**
@@ -597,7 +597,7 @@  discard block
 block discarded – undo
597 597
     {
598 598
         $path = dirname(dirname(dirname(dirname($this->thinkPath))));
599 599
 
600
-        return $path . DIRECTORY_SEPARATOR;
600
+        return $path.DIRECTORY_SEPARATOR;
601 601
     }
602 602
 
603 603
 }
Please login to merge, or discard this patch.