Completed
Pull Request — 6.0 (#1921)
by nhzex
04:55
created
tests/HttpTest.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 
49 49
         $route = m::mock(Route::class);
50 50
 
51
-        $route->shouldReceive('dispatch')->withArgs(function ($req, $withRoute) use ($request) {
51
+        $route->shouldReceive('dispatch')->withArgs(function($req, $withRoute) use ($request) {
52 52
             if ($withRoute) {
53 53
                 $withRoute();
54 54
             }
@@ -80,8 +80,8 @@  discard block
 block discarded – undo
80 80
 
81 81
         $this->http->multi(false);
82 82
 
83
-        $this->app->shouldReceive('getBasePath')->andReturn($root->getChild('app')->url() . DIRECTORY_SEPARATOR);
84
-        $this->app->shouldReceive('getRootPath')->andReturn($root->url() . DIRECTORY_SEPARATOR);
83
+        $this->app->shouldReceive('getBasePath')->andReturn($root->getChild('app')->url().DIRECTORY_SEPARATOR);
84
+        $this->app->shouldReceive('getRootPath')->andReturn($root->url().DIRECTORY_SEPARATOR);
85 85
 
86 86
         $request  = m::mock(Request::class)->makePartial();
87 87
         $response = m::mock(Response::class)->makePartial();
@@ -140,9 +140,9 @@  discard block
 block discarded – undo
140 140
 
141 141
         $this->app->shouldReceive('get')->with('config')->andReturn($config);
142 142
 
143
-        $this->app->shouldReceive('getBasePath')->andReturn($root->getChild('app')->url() . DIRECTORY_SEPARATOR);
144
-        $this->app->shouldReceive('getRootPath')->andReturn($root->url() . DIRECTORY_SEPARATOR);
145
-        $this->app->shouldReceive('getConfigPath')->andReturn($root->getChild('config')->url() . DIRECTORY_SEPARATOR);
143
+        $this->app->shouldReceive('getBasePath')->andReturn($root->getChild('app')->url().DIRECTORY_SEPARATOR);
144
+        $this->app->shouldReceive('getRootPath')->andReturn($root->url().DIRECTORY_SEPARATOR);
145
+        $this->app->shouldReceive('getConfigPath')->andReturn($root->getChild('config')->url().DIRECTORY_SEPARATOR);
146 146
 
147 147
         $response = m::mock(Response::class)->makePartial();
148 148
 
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
             $this->assertTrue($this->http->isBindDomain());
175 175
         }
176 176
         if ($path) {
177
-            $this->assertEquals(rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR, $this->app->getAppPath());
177
+            $this->assertEquals(rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR, $this->app->getAppPath());
178 178
         }
179 179
     }
180 180
 
Please login to merge, or discard this patch.
src/think/middleware/FormTokenCheck.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\middleware;
14 14
 
Please login to merge, or discard this patch.
src/think/middleware/AllowCrossDomain.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\middleware;
14 14
 
Please login to merge, or discard this patch.
src/think/contract/SessionHandlerInterface.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\contract;
14 14
 
Please login to merge, or discard this patch.
src/think/contract/TemplateHandlerInterface.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\contract;
14 14
 
Please login to merge, or discard this patch.
src/think/Route.php 1 patch
Spacing   +12 added lines, -12 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
 
@@ -175,9 +175,9 @@  discard block
 block discarded – undo
175 175
             $this->cache = $this->app->cache->store(true === $this->config['route_check_cache'] ? '' : $this->config['route_check_cache']);
176 176
         }
177 177
 
178
-        if (is_file($this->app->getRuntimePath() . 'route.php')) {
178
+        if (is_file($this->app->getRuntimePath().'route.php')) {
179 179
             // 读取路由映射文件
180
-            $this->import(include $this->app->getRuntimePath() . 'route.php');
180
+            $this->import(include $this->app->getRuntimePath().'route.php');
181 181
         }
182 182
     }
183 183
 
@@ -390,13 +390,13 @@  discard block
 block discarded – undo
390 390
         if (is_null($domain)) {
391 391
             $domain = $this->host;
392 392
         } elseif (false === strpos($domain, '.')) {
393
-            $domain .= '.' . $this->request->rootDomain();
393
+            $domain .= '.'.$this->request->rootDomain();
394 394
         }
395 395
 
396 396
         $subDomain = $this->request->subDomain();
397 397
 
398 398
         if (strpos($subDomain, '.')) {
399
-            $name = '*' . strstr($subDomain, '.');
399
+            $name = '*'.strstr($subDomain, '.');
400 400
         }
401 401
 
402 402
         if (isset($this->bind[$domain])) {
@@ -727,7 +727,7 @@  discard block
 block discarded – undo
727 727
         $this->init();
728 728
 
729 729
         if ($withRoute) {
730
-            $checkCallback = function () use ($request, $withRoute) {
730
+            $checkCallback = function() use ($request, $withRoute) {
731 731
                 //加载路由
732 732
                 $withRoute();
733 733
                 return $this->check();
@@ -746,7 +746,7 @@  discard block
 block discarded – undo
746 746
 
747 747
         $dispatch->init($this->app);
748 748
 
749
-        $this->app->middleware->add(function () use ($dispatch) {
749
+        $this->app->middleware->add(function() use ($dispatch) {
750 750
             try {
751 751
                 $response = $dispatch->run();
752 752
             } catch (HttpResponseException $exception) {
@@ -770,7 +770,7 @@  discard block
 block discarded – undo
770 770
             $closure  = $this->config['route_check_cache_key'];
771 771
             $routeKey = $closure($request);
772 772
         } else {
773
-            $routeKey = md5($request->baseUrl(true) . ':' . $request->method());
773
+            $routeKey = md5($request->baseUrl(true).':'.$request->method());
774 774
         }
775 775
 
776 776
         return $routeKey;
@@ -820,10 +820,10 @@  discard block
 block discarded – undo
820 820
             $path = $pathinfo;
821 821
         } elseif ($suffix) {
822 822
             // 去除正常的URL后缀
823
-            $path = preg_replace('/\.(' . ltrim($suffix, '.') . ')$/i', '', $pathinfo);
823
+            $path = preg_replace('/\.('.ltrim($suffix, '.').')$/i', '', $pathinfo);
824 824
         } else {
825 825
             // 允许任何后缀访问
826
-            $path = preg_replace('/\.' . $this->request->ext() . '$/i', '', $pathinfo);
826
+            $path = preg_replace('/\.'.$this->request->ext().'$/i', '', $pathinfo);
827 827
         }
828 828
 
829 829
         return $path;
@@ -866,9 +866,9 @@  discard block
 block discarded – undo
866 866
                 $item = $this->domains[$this->host];
867 867
             } elseif (isset($this->domains[$subDomain])) {
868 868
                 $item = $this->domains[$subDomain];
869
-            } elseif (isset($this->domains['*.' . $domain2]) && !empty($domain3)) {
869
+            } elseif (isset($this->domains['*.'.$domain2]) && !empty($domain3)) {
870 870
                 // 泛三级域名
871
-                $item      = $this->domains['*.' . $domain2];
871
+                $item      = $this->domains['*.'.$domain2];
872 872
                 $panDomain = $domain3;
873 873
             } elseif (isset($this->domains['*']) && !empty($domain2)) {
874 874
                 // 泛二级域名
Please login to merge, or discard this patch.
src/think/route/RuleItem.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\route;
14 14
 
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
         $rule = '/' != $rule ? ltrim($rule, '/') : '';
135 135
 
136 136
         if ($this->parent && $prefix = $this->parent->getFullName()) {
137
-            $rule = $prefix . ($rule ? '/' . ltrim($rule, '/') : '');
137
+            $rule = $prefix.($rule ? '/'.ltrim($rule, '/') : '');
138 138
         }
139 139
 
140 140
         if (false !== strpos($rule, ':')) {
@@ -237,7 +237,7 @@  discard block
 block discarded – undo
237 237
 
238 238
         if (isset($option['ext'])) {
239 239
             // 路由ext参数 优先于系统配置的URL伪静态后缀参数
240
-            $url = preg_replace('/\.(' . $request->ext() . ')$/i', '', $url);
240
+            $url = preg_replace('/\.('.$request->ext().')$/i', '', $url);
241 241
         }
242 242
 
243 243
         return $url;
@@ -261,38 +261,38 @@  discard block
 block discarded – undo
261 261
         $pattern = array_merge($this->parent->getPattern(), $this->pattern);
262 262
 
263 263
         // 检查完整规则定义
264
-        if (isset($pattern['__url__']) && !preg_match(0 === strpos($pattern['__url__'], '/') ? $pattern['__url__'] : '/^' . $pattern['__url__'] . ($completeMatch ? '$' : '') . '/', str_replace('|', $depr, $url))) {
264
+        if (isset($pattern['__url__']) && !preg_match(0 === strpos($pattern['__url__'], '/') ? $pattern['__url__'] : '/^'.$pattern['__url__'].($completeMatch ? '$' : '').'/', str_replace('|', $depr, $url))) {
265 265
             return false;
266 266
         }
267 267
 
268 268
         $var  = [];
269
-        $url  = $depr . str_replace('|', $depr, $url);
270
-        $rule = $depr . str_replace('/', $depr, $this->rule);
269
+        $url  = $depr.str_replace('|', $depr, $url);
270
+        $rule = $depr.str_replace('/', $depr, $this->rule);
271 271
 
272 272
         if ($depr == $rule && $depr != $url) {
273 273
             return false;
274 274
         }
275 275
 
276 276
         if (false === strpos($rule, '<')) {
277
-            if (0 === strcasecmp($rule, $url) || (!$completeMatch && 0 === strncasecmp($rule . $depr, $url . $depr, strlen($rule . $depr)))) {
277
+            if (0 === strcasecmp($rule, $url) || (!$completeMatch && 0 === strncasecmp($rule.$depr, $url.$depr, strlen($rule.$depr)))) {
278 278
                 return $var;
279 279
             }
280 280
             return false;
281 281
         }
282 282
 
283
-        $slash = preg_quote('/-' . $depr, '/');
283
+        $slash = preg_quote('/-'.$depr, '/');
284 284
 
285
-        if ($matchRule = preg_split('/[' . $slash . ']?<\w+\??>/', $rule, 2)) {
285
+        if ($matchRule = preg_split('/['.$slash.']?<\w+\??>/', $rule, 2)) {
286 286
             if ($matchRule[0] && 0 !== strncasecmp($rule, $url, strlen($matchRule[0]))) {
287 287
                 return false;
288 288
             }
289 289
         }
290 290
 
291
-        if (preg_match_all('/[' . $slash . ']?<?\w+\??>?/', $rule, $matches)) {
291
+        if (preg_match_all('/['.$slash.']?<?\w+\??>?/', $rule, $matches)) {
292 292
             $regex = $this->buildRuleRegex($rule, $matches[0], $pattern, $option, $completeMatch);
293 293
 
294 294
             try {
295
-                if (!preg_match('/^' . $regex . ($completeMatch ? '$' : '') . '/u', $url, $match)) {
295
+                if (!preg_match('/^'.$regex.($completeMatch ? '$' : '').'/u', $url, $match)) {
296 296
                     return false;
297 297
                 }
298 298
             } catch (\Exception $e) {
Please login to merge, or discard this patch.
src/think/filesystem/driver/Local.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: yunwuxin <[email protected]>
10 10
 // +----------------------------------------------------------------------
11
-declare (strict_types = 1);
11
+declare(strict_types=1);
12 12
 
13 13
 namespace think\filesystem\driver;
14 14
 
Please login to merge, or discard this patch.
src/think/Http.php 1 patch
Spacing   +24 added lines, -24 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 __construct(App $app)
57 57
     {
58 58
         $this->app   = $app;
59
-        $this->multi = is_dir($this->app->getBasePath() . 'controller') ? false : true;
59
+        $this->multi = is_dir($this->app->getBasePath().'controller') ? false : true;
60 60
     }
61 61
 
62 62
     /**
@@ -172,8 +172,8 @@  discard block
 block discarded – undo
172 172
         $this->initialize();
173 173
 
174 174
         // 加载全局中间件
175
-        if (is_file($this->app->getBasePath() . 'middleware.php')) {
176
-            $this->app->middleware->import(include $this->app->getBasePath() . 'middleware.php');
175
+        if (is_file($this->app->getBasePath().'middleware.php')) {
176
+            $this->app->middleware->import(include $this->app->getBasePath().'middleware.php');
177 177
         }
178 178
 
179 179
         if ($this->multi) {
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
         // 监听HttpRun
187 187
         $this->app->event->trigger('HttpRun');
188 188
 
189
-        $withRoute = $this->app->config->get('app.with_route', true) ? function () {
189
+        $withRoute = $this->app->config->get('app.with_route', true) ? function() {
190 190
             $this->loadRoutes();
191 191
         } : null;
192 192
 
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
     {
203 203
         // 加载路由定义
204 204
         if (is_dir($this->getRoutePath())) {
205
-            $files = glob($this->getRoutePath() . '*.php');
205
+            $files = glob($this->getRoutePath().'*.php');
206 206
             foreach ($files as $file) {
207 207
                 include $file;
208 208
             }
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
      */
219 219
     protected function getRoutePath(): string
220 220
     {
221
-        return $this->app->getRootPath() . 'route' . DIRECTORY_SEPARATOR . ($this->isMulti() ? $this->getName() . DIRECTORY_SEPARATOR : '');
221
+        return $this->app->getRootPath().'route'.DIRECTORY_SEPARATOR.($this->isMulti() ? $this->getName().DIRECTORY_SEPARATOR : '');
222 222
     }
223 223
 
224 224
     /**
@@ -303,7 +303,7 @@  discard block
 block discarded – undo
303 303
                         $appName = $map[$name];
304 304
                     }
305 305
                 } elseif ($name && (false !== array_search($name, $map) || in_array($name, $deny))) {
306
-                    throw new HttpException(404, 'app not exists:' . $name);
306
+                    throw new HttpException(404, 'app not exists:'.$name);
307 307
                 } elseif ($name && isset($map['*'])) {
308 308
                     $appName = $map['*'];
309 309
                 } else {
@@ -311,7 +311,7 @@  discard block
 block discarded – undo
311 311
                 }
312 312
 
313 313
                 if ($name) {
314
-                    $this->app->request->setRoot('/' . $name);
314
+                    $this->app->request->setRoot('/'.$name);
315 315
                     $this->app->request->setPathinfo(strpos($path, '/') ? ltrim(strstr($path, '/'), '/') : '');
316 316
                 }
317 317
             }
@@ -331,41 +331,41 @@  discard block
 block discarded – undo
331 331
     {
332 332
         $this->name = $appName;
333 333
         $this->app->request->setApp($appName);
334
-        $this->app->setAppPath($this->path ?: $this->app->getBasePath() . $appName . DIRECTORY_SEPARATOR);
335
-        $this->app->setRuntimePath($this->app->getRootPath() . 'runtime' . DIRECTORY_SEPARATOR . $appName . DIRECTORY_SEPARATOR);
334
+        $this->app->setAppPath($this->path ?: $this->app->getBasePath().$appName.DIRECTORY_SEPARATOR);
335
+        $this->app->setRuntimePath($this->app->getRootPath().'runtime'.DIRECTORY_SEPARATOR.$appName.DIRECTORY_SEPARATOR);
336 336
 
337 337
         //加载app文件
338 338
         if (is_dir($this->app->getAppPath())) {
339 339
             $appPath = $this->app->getAppPath();
340 340
 
341
-            if (is_file($appPath . 'common.php')) {
342
-                include_once $appPath . 'common.php';
341
+            if (is_file($appPath.'common.php')) {
342
+                include_once $appPath.'common.php';
343 343
             }
344 344
 
345 345
             $configPath = $this->app->getConfigPath();
346 346
 
347 347
             $files = [];
348 348
 
349
-            if (is_dir($configPath . $appName)) {
350
-                $files = array_merge($files, glob($configPath . $appName . DIRECTORY_SEPARATOR . '*' . $this->app->getConfigExt()));
351
-            } elseif (is_dir($appPath . 'config')) {
352
-                $files = array_merge($files, glob($appPath . 'config' . DIRECTORY_SEPARATOR . '*' . $this->app->getConfigExt()));
349
+            if (is_dir($configPath.$appName)) {
350
+                $files = array_merge($files, glob($configPath.$appName.DIRECTORY_SEPARATOR.'*'.$this->app->getConfigExt()));
351
+            } elseif (is_dir($appPath.'config')) {
352
+                $files = array_merge($files, glob($appPath.'config'.DIRECTORY_SEPARATOR.'*'.$this->app->getConfigExt()));
353 353
             }
354 354
 
355 355
             foreach ($files as $file) {
356 356
                 $this->app->config->load($file, pathinfo($file, PATHINFO_FILENAME));
357 357
             }
358 358
 
359
-            if (is_file($appPath . 'event.php')) {
360
-                $this->app->loadEvent(include $appPath . 'event.php');
359
+            if (is_file($appPath.'event.php')) {
360
+                $this->app->loadEvent(include $appPath.'event.php');
361 361
             }
362 362
 
363
-            if (is_file($appPath . 'middleware.php')) {
364
-                $this->app->middleware->import(include $appPath . 'middleware.php');
363
+            if (is_file($appPath.'middleware.php')) {
364
+                $this->app->middleware->import(include $appPath.'middleware.php');
365 365
             }
366 366
 
367
-            if (is_file($appPath . 'provider.php')) {
368
-                $this->app->bind(include $appPath . 'provider.php');
367
+            if (is_file($appPath.'provider.php')) {
368
+                $this->app->bind(include $appPath.'provider.php');
369 369
             }
370 370
         }
371 371
 
@@ -373,7 +373,7 @@  discard block
 block discarded – undo
373 373
         $this->app->loadLangPack($this->app->lang->defaultLangSet());
374 374
 
375 375
         // 设置应用命名空间
376
-        $this->app->setNamespace($this->app->config->get('app.app_namespace') ?: 'app\\' . $appName);
376
+        $this->app->setNamespace($this->app->config->get('app.app_namespace') ?: 'app\\'.$appName);
377 377
     }
378 378
 
379 379
     /**
Please login to merge, or discard this patch.