Completed
Pull Request — 6.0 (#2187)
by topphp
08:15
created
tests/HttpTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 
48 48
         $route = m::mock(Route::class);
49 49
 
50
-        $route->shouldReceive('dispatch')->withArgs(function ($req, $withRoute) use ($request) {
50
+        $route->shouldReceive('dispatch')->withArgs(function($req, $withRoute) use ($request) {
51 51
             if ($withRoute) {
52 52
                 $withRoute();
53 53
             }
@@ -77,8 +77,8 @@  discard block
 block discarded – undo
77 77
             ],
78 78
         ]);
79 79
 
80
-        $this->app->shouldReceive('getBasePath')->andReturn($root->getChild('app')->url() . DIRECTORY_SEPARATOR);
81
-        $this->app->shouldReceive('getRootPath')->andReturn($root->url() . DIRECTORY_SEPARATOR);
80
+        $this->app->shouldReceive('getBasePath')->andReturn($root->getChild('app')->url().DIRECTORY_SEPARATOR);
81
+        $this->app->shouldReceive('getRootPath')->andReturn($root->url().DIRECTORY_SEPARATOR);
82 82
 
83 83
         $request  = m::mock(Request::class)->makePartial();
84 84
         $response = m::mock(Response::class)->makePartial();
Please login to merge, or discard this patch.
src/think/Http.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;
14 14
 
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
     {
53 53
         $this->app = $app;
54 54
 
55
-        $this->routePath = $this->app->getRootPath() . 'route' . DIRECTORY_SEPARATOR;
55
+        $this->routePath = $this->app->getRootPath().'route'.DIRECTORY_SEPARATOR;
56 56
     }
57 57
 
58 58
     /**
@@ -199,14 +199,14 @@  discard block
 block discarded – undo
199 199
 
200 200
         return $this->app->middleware->pipeline()
201 201
             ->send($request)
202
-            ->then(function ($request) {
202
+            ->then(function($request) {
203 203
                 return $this->dispatchToRoute($request);
204 204
             });
205 205
     }
206 206
 
207 207
     protected function dispatchToRoute($request)
208 208
     {
209
-        $withRoute = $this->app->config->get('app.with_route', true) ? function () {
209
+        $withRoute = $this->app->config->get('app.with_route', true) ? function() {
210 210
             $this->loadRoutes();
211 211
         } : null;
212 212
 
@@ -218,8 +218,8 @@  discard block
 block discarded – undo
218 218
      */
219 219
     protected function loadMiddleware(): void
220 220
     {
221
-        if (is_file($this->app->getBasePath() . 'middleware.php')) {
222
-            $this->app->middleware->import(include $this->app->getBasePath() . 'middleware.php');
221
+        if (is_file($this->app->getBasePath().'middleware.php')) {
222
+            $this->app->middleware->import(include $this->app->getBasePath().'middleware.php');
223 223
         }
224 224
     }
225 225
 
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
         $routePath = $this->getRoutePath();
235 235
 
236 236
         if (is_dir($routePath)) {
237
-            $files = glob($routePath . '*.php');
237
+            $files = glob($routePath.'*.php');
238 238
             foreach ($files as $file) {
239 239
                 include $file;
240 240
             }
Please login to merge, or discard this patch.
src/think/console/Command.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: yunwuxin <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think\console;
14 14
 
@@ -404,7 +404,7 @@  discard block
 block discarded – undo
404 404
         ];
405 405
         $replacements = [
406 406
             $name,
407
-            $_SERVER['PHP_SELF'] . ' ' . $name,
407
+            $_SERVER['PHP_SELF'].' '.$name,
408 408
         ];
409 409
 
410 410
         return str_replace($placeholders, $replacements, $this->getHelp());
Please login to merge, or discard this patch.
src/think/console/command/optimize/Schema.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -48,21 +48,21 @@  discard block
 block discarded – undo
48 48
             $tables = $this->app->db->getConnection()->getTables($dbName);
49 49
         } else {
50 50
             if ($dir) {
51
-                $appPath   = $this->app->getBasePath() . $dir . DIRECTORY_SEPARATOR;
52
-                $namespace = 'app\\' . $dir;
51
+                $appPath   = $this->app->getBasePath().$dir.DIRECTORY_SEPARATOR;
52
+                $namespace = 'app\\'.$dir;
53 53
             } else {
54 54
                 $appPath   = $this->app->getBasePath();
55 55
                 $namespace = 'app';
56 56
             }
57 57
 
58
-            $path = $appPath . 'model';
58
+            $path = $appPath.'model';
59 59
             $list = is_dir($path) ? scandir($path) : [];
60 60
 
61 61
             foreach ($list as $file) {
62 62
                 if (0 === strpos($file, '.')) {
63 63
                     continue;
64 64
                 }
65
-                $class = '\\' . $namespace . '\\model\\' . pathinfo($file, PATHINFO_FILENAME);
65
+                $class = '\\'.$namespace.'\\model\\'.pathinfo($file, PATHINFO_FILENAME);
66 66
                 $this->buildModelSchema($class);
67 67
             }
68 68
 
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
             return;
71 71
         }
72 72
 
73
-        $db = isset($dbName) ? $dbName . '.' : '';
73
+        $db = isset($dbName) ? $dbName.'.' : '';
74 74
         $this->buildDataBaseSchema($schemaPath, $tables, $db);
75 75
 
76 76
         $output->writeln('<info>Succeed!</info>');
@@ -89,27 +89,27 @@  discard block
 block discarded – undo
89 89
             if (!is_dir($path)) {
90 90
                 mkdir($path, 0755, true);
91 91
             }
92
-            $content = '<?php ' . PHP_EOL . 'return ';
92
+            $content = '<?php '.PHP_EOL.'return ';
93 93
             $info    = $model->db()->getConnection()->getTableFieldsInfo($table);
94
-            $content .= var_export($info, true) . ';';
94
+            $content .= var_export($info, true).';';
95 95
 
96
-            file_put_contents($path . $dbName . '.' . $table . '.php', $content);
96
+            file_put_contents($path.$dbName.'.'.$table.'.php', $content);
97 97
         }
98 98
     }
99 99
 
100 100
     protected function buildDataBaseSchema(string $path, array $tables, string $db): void
101 101
     {
102 102
         if ('' == $db) {
103
-            $dbName = $this->app->db->getConnection()->getConfig('database') . '.';
103
+            $dbName = $this->app->db->getConnection()->getConfig('database').'.';
104 104
         } else {
105 105
             $dbName = $db;
106 106
         }
107 107
 
108 108
         foreach ($tables as $table) {
109
-            $content = '<?php ' . PHP_EOL . 'return ';
110
-            $info    = $this->app->db->getConnection()->getTableFieldsInfo($db . $table);
111
-            $content .= var_export($info, true) . ';';
112
-            file_put_contents($path . $dbName . $table . '.php', $content);
109
+            $content = '<?php '.PHP_EOL.'return ';
110
+            $info    = $this->app->db->getConnection()->getTableFieldsInfo($db.$table);
111
+            $content .= var_export($info, true).';';
112
+            file_put_contents($path.$dbName.$table.'.php', $content);
113 113
         }
114 114
     }
115 115
 }
Please login to merge, or discard this patch.
src/think/console/command/RouteList.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
     {
43 43
         $dir = $input->getArgument('dir') ?: '';
44 44
 
45
-        $filename = $this->app->getRootPath() . 'runtime' . DIRECTORY_SEPARATOR . ($dir ? $dir . DIRECTORY_SEPARATOR : '') . 'route_list.php';
45
+        $filename = $this->app->getRootPath().'runtime'.DIRECTORY_SEPARATOR.($dir ? $dir.DIRECTORY_SEPARATOR : '').'route_list.php';
46 46
 
47 47
         if (is_file($filename)) {
48 48
             unlink($filename);
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
         }
52 52
 
53 53
         $content = $this->getRouteList($dir);
54
-        file_put_contents($filename, 'Route List' . PHP_EOL . $content);
54
+        file_put_contents($filename, 'Route List'.PHP_EOL.$content);
55 55
     }
56 56
 
57 57
     protected function getRouteList(string $dir = null): string
@@ -60,16 +60,16 @@  discard block
 block discarded – undo
60 60
         $this->app->route->clear();
61 61
 
62 62
         if ($dir) {
63
-            $path = $this->app->getRootPath() . 'route' . DIRECTORY_SEPARATOR . $dir . DIRECTORY_SEPARATOR;
63
+            $path = $this->app->getRootPath().'route'.DIRECTORY_SEPARATOR.$dir.DIRECTORY_SEPARATOR;
64 64
         } else {
65
-            $path = $this->app->getRootPath() . 'route' . DIRECTORY_SEPARATOR;
65
+            $path = $this->app->getRootPath().'route'.DIRECTORY_SEPARATOR;
66 66
         }
67 67
 
68 68
         $files = is_dir($path) ? scandir($path) : [];
69 69
 
70 70
         foreach ($files as $file) {
71 71
             if (strpos($file, '.php')) {
72
-                include $path . $file;
72
+                include $path.$file;
73 73
             }
74 74
         }
75 75
 
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
                 $sort = $this->sortBy[$sort];
109 109
             }
110 110
 
111
-            uasort($rows, function ($a, $b) use ($sort) {
111
+            uasort($rows, function($a, $b) use ($sort) {
112 112
                 $itemA = $a[$sort] ?? null;
113 113
                 $itemB = $b[$sort] ?? null;
114 114
 
Please login to merge, or discard this patch.
tests/MiddlewareTest.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -64,9 +64,9 @@  discard block
 block discarded – undo
64 64
 
65 65
         $this->middleware->add('foo');
66 66
         $this->assertEquals(3, count($this->middleware->all()));
67
-        $this->middleware->add(function () {
67
+        $this->middleware->add(function() {
68 68
         });
69
-        $this->middleware->add(function () {
69
+        $this->middleware->add(function() {
70 70
         });
71 71
         $this->assertEquals(5, count($this->middleware->all()));
72 72
     }
@@ -85,10 +85,10 @@  discard block
 block discarded – undo
85 85
         $handle->shouldReceive('report')->with($e)->andReturnNull();
86 86
         $handle->shouldReceive('render')->with($request, $e)->andReturn($response);
87 87
 
88
-        $foo->shouldReceive('handle')->once()->andReturnUsing(function ($request, $next) {
88
+        $foo->shouldReceive('handle')->once()->andReturnUsing(function($request, $next) {
89 89
             return $next($request);
90 90
         });
91
-        $bar->shouldReceive('handle')->once()->andReturnUsing(function ($request, $next) use ($e) {
91
+        $bar->shouldReceive('handle')->once()->andReturnUsing(function($request, $next) use ($e) {
92 92
             $next($request);
93 93
             throw  $e;
94 94
         });
@@ -99,13 +99,13 @@  discard block
 block discarded – undo
99 99
 
100 100
         $this->config->shouldReceive('get')->once()->with('middleware.priority', [])->andReturn(['FooMiddleware', 'BarMiddleware']);
101 101
 
102
-        $this->middleware->import([function ($request, $next) {
102
+        $this->middleware->import([function($request, $next) {
103 103
             return $next($request);
104 104
         }, 'BarMiddleware', 'FooMiddleware']);
105 105
 
106 106
         $this->assertInstanceOf(Pipeline::class, $pipeline = $this->middleware->pipeline());
107 107
 
108
-        $pipeline->send($request)->then(function ($request) use ($e, $response) {
108
+        $pipeline->send($request)->then(function($request) use ($e, $response) {
109 109
             throw $e;
110 110
         });
111 111
 
Please login to merge, or discard this patch.
src/think/session/driver/File.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\session\driver;
14 14
 
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
         $this->config = array_merge($this->config, $config);
40 40
 
41 41
         if (empty($this->config['path'])) {
42
-            $this->config['path'] = $app->getRootPath() . 'runtime' . DIRECTORY_SEPARATOR . 'session' . DIRECTORY_SEPARATOR;
42
+            $this->config['path'] = $app->getRootPath().'runtime'.DIRECTORY_SEPARATOR.'session'.DIRECTORY_SEPARATOR;
43 43
         } elseif (substr($this->config['path'], -1) != DIRECTORY_SEPARATOR) {
44 44
             $this->config['path'] .= DIRECTORY_SEPARATOR;
45 45
         }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
         $lifetime = $this->config['expire'];
77 77
         $now      = time();
78 78
 
79
-        $files = $this->findFiles($this->config['path'], function (SplFileInfo $item) use ($lifetime, $now) {
79
+        $files = $this->findFiles($this->config['path'], function(SplFileInfo $item) use ($lifetime, $now) {
80 80
             return $now - $lifetime > $item->getMTime();
81 81
         });
82 82
 
@@ -118,12 +118,12 @@  discard block
 block discarded – undo
118 118
     {
119 119
         if ($this->config['prefix']) {
120 120
             // 使用子目录
121
-            $name = $this->config['prefix'] . DIRECTORY_SEPARATOR . 'sess_' . $name;
121
+            $name = $this->config['prefix'].DIRECTORY_SEPARATOR.'sess_'.$name;
122 122
         } else {
123
-            $name = 'sess_' . $name;
123
+            $name = 'sess_'.$name;
124 124
         }
125 125
 
126
-        $filename = $this->config['path'] . $name;
126
+        $filename = $this->config['path'].$name;
127 127
         $dir      = dirname($filename);
128 128
 
129 129
         if ($auto && !is_dir($dir)) {
Please login to merge, or discard this patch.
src/think/Route.php 1 patch
Spacing   +10 added lines, -10 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
 
@@ -160,9 +160,9 @@  discard block
 block discarded – undo
160 160
         $this->ruleName = new RuleName();
161 161
         $this->setDefaultDomain();
162 162
 
163
-        if (is_file($this->app->getRuntimePath() . 'route.php')) {
163
+        if (is_file($this->app->getRuntimePath().'route.php')) {
164 164
             // 读取路由映射文件
165
-            $this->import(include $this->app->getRuntimePath() . 'route.php');
165
+            $this->import(include $this->app->getRuntimePath().'route.php');
166 166
         }
167 167
     }
168 168
 
@@ -392,14 +392,14 @@  discard block
 block discarded – undo
392 392
         if (is_null($domain)) {
393 393
             $domain = $this->host;
394 394
         } elseif (false === strpos($domain, '.') && $this->request) {
395
-            $domain .= '.' . $this->request->rootDomain();
395
+            $domain .= '.'.$this->request->rootDomain();
396 396
         }
397 397
 
398 398
         if ($this->request) {
399 399
             $subDomain = $this->request->subDomain();
400 400
 
401 401
             if (strpos($subDomain, '.')) {
402
-                $name = '*' . strstr($subDomain, '.');
402
+                $name = '*'.strstr($subDomain, '.');
403 403
             }
404 404
         }
405 405
 
@@ -743,7 +743,7 @@  discard block
 block discarded – undo
743 743
 
744 744
         return $this->app->middleware->pipeline('route')
745 745
             ->send($request)
746
-            ->then(function () use ($dispatch) {
746
+            ->then(function() use ($dispatch) {
747 747
                 return $dispatch->run();
748 748
             });
749 749
     }
@@ -792,10 +792,10 @@  discard block
 block discarded – undo
792 792
             $path = $pathinfo;
793 793
         } elseif ($suffix) {
794 794
             // 去除正常的URL后缀
795
-            $path = preg_replace('/\.(' . ltrim($suffix, '.') . ')$/i', '', $pathinfo);
795
+            $path = preg_replace('/\.('.ltrim($suffix, '.').')$/i', '', $pathinfo);
796 796
         } else {
797 797
             // 允许任何后缀访问
798
-            $path = preg_replace('/\.' . $this->request->ext() . '$/i', '', $pathinfo);
798
+            $path = preg_replace('/\.'.$this->request->ext().'$/i', '', $pathinfo);
799 799
         }
800 800
 
801 801
         return $path;
@@ -838,9 +838,9 @@  discard block
 block discarded – undo
838 838
                 $item = $this->domains[$this->host];
839 839
             } elseif (isset($this->domains[$subDomain])) {
840 840
                 $item = $this->domains[$subDomain];
841
-            } elseif (isset($this->domains['*.' . $domain2]) && !empty($domain3)) {
841
+            } elseif (isset($this->domains['*.'.$domain2]) && !empty($domain3)) {
842 842
                 // 泛三级域名
843
-                $item      = $this->domains['*.' . $domain2];
843
+                $item      = $this->domains['*.'.$domain2];
844 844
                 $panDomain = $domain3;
845 845
             } elseif (isset($this->domains['*']) && !empty($domain2)) {
846 846
                 // 泛二级域名
Please login to merge, or discard this patch.
src/think/console/command/Clear.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -30,18 +30,18 @@  discard block
 block discarded – undo
30 30
 
31 31
     protected function execute(Input $input, Output $output)
32 32
     {
33
-        $runtimePath = $this->app->getRootPath() . 'runtime' . DIRECTORY_SEPARATOR;
33
+        $runtimePath = $this->app->getRootPath().'runtime'.DIRECTORY_SEPARATOR;
34 34
 
35 35
         if ($input->getOption('cache')) {
36
-            $path = $runtimePath . 'cache';
36
+            $path = $runtimePath.'cache';
37 37
         } elseif ($input->getOption('log')) {
38
-            $path = $runtimePath . 'log';
38
+            $path = $runtimePath.'log';
39 39
         } else {
40 40
             $path = $input->getOption('path') ?: $runtimePath;
41 41
         }
42 42
 
43 43
         $rmdir = $input->getOption('dir') ? true : false;
44
-        $this->clear(rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR, $rmdir);
44
+        $this->clear(rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR, $rmdir);
45 45
 
46 46
         $output->writeln("<info>Clear Successed</info>");
47 47
     }
@@ -51,13 +51,13 @@  discard block
 block discarded – undo
51 51
         $files = is_dir($path) ? scandir($path) : [];
52 52
 
53 53
         foreach ($files as $file) {
54
-            if ('.' != $file && '..' != $file && is_dir($path . $file)) {
55
-                array_map('unlink', glob($path . $file . DIRECTORY_SEPARATOR . '*.*'));
54
+            if ('.' != $file && '..' != $file && is_dir($path.$file)) {
55
+                array_map('unlink', glob($path.$file.DIRECTORY_SEPARATOR.'*.*'));
56 56
                 if ($rmdir) {
57
-                    rmdir($path . $file);
57
+                    rmdir($path.$file);
58 58
                 }
59
-            } elseif ('.gitignore' != $file && is_file($path . $file)) {
60
-                unlink($path . $file);
59
+            } elseif ('.gitignore' != $file && is_file($path.$file)) {
60
+                unlink($path.$file);
61 61
             }
62 62
         }
63 63
     }
Please login to merge, or discard this patch.