Completed
Pull Request — 6.0 (#2052)
by
unknown
05:39
created
src/think/filesystem/Driver.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\filesystem;
14 14
 
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
     public function putFileAs(string $path, File $file, string $name, array $options = [])
116 116
     {
117 117
         $stream = fopen($file->getRealPath(), 'r');
118
-        $path = trim($path . '/' . $name, '/');
118
+        $path = trim($path.'/'.$name, '/');
119 119
 
120 120
         $result = $this->putStream($path, $stream, $options);
121 121
 
Please login to merge, or discard this patch.
src/think/Middleware.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: Slince <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think;
14 14
 
@@ -128,8 +128,8 @@  discard block
 block discarded – undo
128 128
     public function pipeline(string $type = 'global')
129 129
     {
130 130
         return (new Pipeline())
131
-            ->through(array_map(function ($middleware) {
132
-                return function ($request, $next) use ($middleware) {
131
+            ->through(array_map(function($middleware) {
132
+                return function($request, $next) use ($middleware) {
133 133
                     list($call, $param) = $middleware;
134 134
                     if (is_array($call) && is_string($call[0])) {
135 135
                         $call = [$this->app->make($call[0]), $call[1]];
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
226 226
     protected function sortMiddleware(array $middlewares)
227 227
     {
228 228
         $priority = $this->app->config->get('middleware.priority', []);
229
-        uasort($middlewares, function ($a, $b) use ($priority) {
229
+        uasort($middlewares, function($a, $b) use ($priority) {
230 230
             $aPriority = $this->getMiddlewarePriority($priority, $a);
231 231
             $bPriority = $this->getMiddlewarePriority($priority, $b);
232 232
             return $bPriority - $aPriority;
Please login to merge, or discard this patch.
src/think/route/dispatch/Url.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\route\dispatch;
14 14
 
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
         if ($bind && preg_match('/^[a-z]/is', $bind)) {
48 48
             $bind = str_replace('/', $depr, $bind);
49 49
             // 如果有域名绑定
50
-            $url = $bind . ('.' != substr($bind, -1) ? $depr : '') . ltrim($url, $depr);
50
+            $url = $bind.('.' != substr($bind, -1) ? $depr : '').ltrim($url, $depr);
51 51
         }
52 52
 
53 53
         $path = $this->rule->parseUrlPath($url);
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
         $controller = !empty($path) ? array_shift($path) : null;
60 60
 
61 61
         if ($controller && !preg_match('/^[A-Za-z][\w|\.]*$/', $controller)) {
62
-            throw new HttpException(404, 'controller not exists:' . $controller);
62
+            throw new HttpException(404, 'controller not exists:'.$controller);
63 63
         }
64 64
 
65 65
         // 解析操作
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
 
69 69
         // 解析额外参数
70 70
         if ($path) {
71
-            preg_replace_callback('/(\w+)\|([^\|]+)/', function ($match) use (&$var) {
71
+            preg_replace_callback('/(\w+)\|([^\|]+)/', function($match) use (&$var) {
72 72
                 $var[$match[1]] = strip_tags($match[2]);
73 73
             }, implode('|', $path));
74 74
         }
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
         $route = [$controller, $action];
87 87
 
88 88
         if ($this->hasDefinedRoute($route)) {
89
-            throw new HttpException(404, 'invalid request:' . str_replace('|', $depr, $url));
89
+            throw new HttpException(404, 'invalid request:'.str_replace('|', $depr, $url));
90 90
         }
91 91
 
92 92
         return $route;
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
         list($controller, $action) = $route;
104 104
 
105 105
         // 检查地址是否被定义过路由
106
-        $name = strtolower(Str::studly($controller) . '/' . $action);
106
+        $name = strtolower(Str::studly($controller).'/'.$action);
107 107
 
108 108
         $host   = $this->request->host(true);
109 109
         $method = $this->request->method();
Please login to merge, or discard this patch.
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/view/driver/Php.php 1 patch
Spacing   +11 added lines, -11 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\view\driver;
14 14
 
@@ -78,14 +78,14 @@  discard block
 block discarded – undo
78 78
 
79 79
         // 模板不存在 抛出异常
80 80
         if (!is_file($template)) {
81
-            throw new RuntimeException('template not exists:' . $template);
81
+            throw new RuntimeException('template not exists:'.$template);
82 82
         }
83 83
 
84 84
         $this->template = $template;
85 85
 
86 86
         // 记录视图信息
87 87
         $this->app->log
88
-            ->record('[ VIEW ] ' . $template . ' [ ' . var_export(array_keys($data), true) . ' ]');
88
+            ->record('[ VIEW ] '.$template.' [ '.var_export(array_keys($data), true).' ]');
89 89
 
90 90
         extract($data, EXTR_OVERWRITE);
91 91
 
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
         $this->content = $content;
105 105
 
106 106
         extract($data, EXTR_OVERWRITE);
107
-        eval('?>' . $this->content);
107
+        eval('?>'.$this->content);
108 108
     }
109 109
 
110 110
     /**
@@ -129,10 +129,10 @@  discard block
 block discarded – undo
129 129
             $appName = isset($app) ? $app : $this->app->http->getName();
130 130
             $view    = $this->config['view_dir_name'];
131 131
 
132
-            if (is_dir($this->app->getAppPath() . $view)) {
133
-                $path = isset($app) ? $this->app->getBasePath() . ($appName ? $appName . DIRECTORY_SEPARATOR : '') . $view . DIRECTORY_SEPARATOR : $this->app->getAppPath() . $view . DIRECTORY_SEPARATOR;
132
+            if (is_dir($this->app->getAppPath().$view)) {
133
+                $path = isset($app) ? $this->app->getBasePath().($appName ? $appName.DIRECTORY_SEPARATOR : '').$view.DIRECTORY_SEPARATOR : $this->app->getAppPath().$view.DIRECTORY_SEPARATOR;
134 134
             } else {
135
-                $path = $this->app->getRootPath() . $view . DIRECTORY_SEPARATOR . ($appName ? $appName . DIRECTORY_SEPARATOR : '');
135
+                $path = $this->app->getRootPath().$view.DIRECTORY_SEPARATOR.($appName ? $appName.DIRECTORY_SEPARATOR : '');
136 136
             }
137 137
         }
138 138
 
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
             $controller = $request->controller();
144 144
             if (strpos($controller, '.')) {
145 145
                 $pos        = strrpos($controller, '.');
146
-                $controller = substr($controller, 0, $pos) . '.' . Str::snake(substr($controller, $pos + 1));
146
+                $controller = substr($controller, 0, $pos).'.'.Str::snake(substr($controller, $pos + 1));
147 147
             } else {
148 148
                 $controller = Str::snake($controller);
149 149
             }
@@ -159,16 +159,16 @@  discard block
 block discarded – undo
159 159
                         $template = Str::snake($request->action());
160 160
                     }
161 161
 
162
-                    $template = str_replace('.', DIRECTORY_SEPARATOR, $controller) . $depr . $template;
162
+                    $template = str_replace('.', DIRECTORY_SEPARATOR, $controller).$depr.$template;
163 163
                 } elseif (false === strpos($template, $depr)) {
164
-                    $template = str_replace('.', DIRECTORY_SEPARATOR, $controller) . $depr . $template;
164
+                    $template = str_replace('.', DIRECTORY_SEPARATOR, $controller).$depr.$template;
165 165
                 }
166 166
             }
167 167
         } else {
168 168
             $template = str_replace(['/', ':'], $depr, substr($template, 1));
169 169
         }
170 170
 
171
-        return $path . ltrim($template, '/') . '.' . ltrim($this->config['view_suffix'], '.');
171
+        return $path.ltrim($template, '/').'.'.ltrim($this->config['view_suffix'], '.');
172 172
     }
173 173
 
174 174
     /**
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/Clear.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -35,18 +35,18 @@  discard block
 block discarded – undo
35 35
         if ($input->getOption('route')) {
36 36
             $this->app->cache->clear('route_cache');
37 37
         } else {
38
-            $runtimePath = $this->app->getRootPath() . 'runtime' . DIRECTORY_SEPARATOR;
38
+            $runtimePath = $this->app->getRootPath().'runtime'.DIRECTORY_SEPARATOR;
39 39
 
40 40
             if ($input->getOption('cache')) {
41
-                $path = $runtimePath . 'cache';
41
+                $path = $runtimePath.'cache';
42 42
             } elseif ($input->getOption('log')) {
43
-                $path = $runtimePath . 'log';
43
+                $path = $runtimePath.'log';
44 44
             } else {
45 45
                 $path = $input->getOption('path') ?: $runtimePath;
46 46
             }
47 47
 
48 48
             $rmdir = $input->getOption('dir') ? true : false;
49
-            $this->clear(rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR, $rmdir);
49
+            $this->clear(rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR, $rmdir);
50 50
         }
51 51
 
52 52
         $output->writeln("<info>Clear Successed</info>");
@@ -57,13 +57,13 @@  discard block
 block discarded – undo
57 57
         $files = is_dir($path) ? scandir($path) : [];
58 58
 
59 59
         foreach ($files as $file) {
60
-            if ('.' != $file && '..' != $file && is_dir($path . $file)) {
61
-                array_map('unlink', glob($path . $file . DIRECTORY_SEPARATOR . '*.*'));
60
+            if ('.' != $file && '..' != $file && is_dir($path.$file)) {
61
+                array_map('unlink', glob($path.$file.DIRECTORY_SEPARATOR.'*.*'));
62 62
                 if ($rmdir) {
63
-                    rmdir($path . $file);
63
+                    rmdir($path.$file);
64 64
                 }
65
-            } elseif ('.gitignore' != $file && is_file($path . $file)) {
66
-                unlink($path . $file);
65
+            } elseif ('.gitignore' != $file && is_file($path.$file)) {
66
+                unlink($path.$file);
67 67
             }
68 68
         }
69 69
     }
Please login to merge, or discard this patch.