Completed
Pull Request — 6.0 (#1919)
by yun
09:27 queued 04:43
created
tests/DbTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@
 block discarded – undo
22 22
      */
23 23
     public function testConnect($config)
24 24
     {
25
-        $mysql = m::mock('overload:' . Mysql::class);
25
+        $mysql = m::mock('overload:'.Mysql::class);
26 26
 
27 27
         $db = new Db($config);
28 28
     }
Please login to merge, or discard this patch.
tests/AppTest.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -107,27 +107,27 @@  discard block
 block discarded – undo
107 107
 
108 108
     public function testPath()
109 109
     {
110
-        $rootPath = __DIR__ . DIRECTORY_SEPARATOR;
110
+        $rootPath = __DIR__.DIRECTORY_SEPARATOR;
111 111
 
112 112
         $app = new App($rootPath);
113 113
 
114 114
         $this->assertEquals($rootPath, $app->getRootPath());
115 115
 
116
-        $this->assertEquals(dirname(__DIR__) . DIRECTORY_SEPARATOR . 'src' . DIRECTORY_SEPARATOR, $app->getThinkPath());
116
+        $this->assertEquals(dirname(__DIR__).DIRECTORY_SEPARATOR.'src'.DIRECTORY_SEPARATOR, $app->getThinkPath());
117 117
 
118
-        $this->assertEquals($rootPath . 'app' . DIRECTORY_SEPARATOR, $app->getAppPath());
118
+        $this->assertEquals($rootPath.'app'.DIRECTORY_SEPARATOR, $app->getAppPath());
119 119
 
120
-        $appPath = $rootPath . 'app' . DIRECTORY_SEPARATOR . 'admin' . DIRECTORY_SEPARATOR;
120
+        $appPath = $rootPath.'app'.DIRECTORY_SEPARATOR.'admin'.DIRECTORY_SEPARATOR;
121 121
         $app->setAppPath($appPath);
122 122
         $this->assertEquals($appPath, $app->getAppPath());
123 123
 
124
-        $this->assertEquals($rootPath . 'app' . DIRECTORY_SEPARATOR, $app->getBasePath());
124
+        $this->assertEquals($rootPath.'app'.DIRECTORY_SEPARATOR, $app->getBasePath());
125 125
 
126
-        $this->assertEquals($rootPath . 'config' . DIRECTORY_SEPARATOR, $app->getConfigPath());
126
+        $this->assertEquals($rootPath.'config'.DIRECTORY_SEPARATOR, $app->getConfigPath());
127 127
 
128
-        $this->assertEquals($rootPath . 'runtime' . DIRECTORY_SEPARATOR, $app->getRuntimePath());
128
+        $this->assertEquals($rootPath.'runtime'.DIRECTORY_SEPARATOR, $app->getRuntimePath());
129 129
 
130
-        $runtimePath = $rootPath . 'runtime' . DIRECTORY_SEPARATOR . 'admin' . DIRECTORY_SEPARATOR;
130
+        $runtimePath = $rootPath.'runtime'.DIRECTORY_SEPARATOR.'admin'.DIRECTORY_SEPARATOR;
131 131
         $app->setRuntimePath($runtimePath);
132 132
         $this->assertEquals($runtimePath, $app->getRuntimePath());
133 133
     }
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
      */
140 140
     protected function prepareAppForInitialize(vfsStreamDirectory $root, $debug = true)
141 141
     {
142
-        $rootPath = $root->url() . DIRECTORY_SEPARATOR;
142
+        $rootPath = $root->url().DIRECTORY_SEPARATOR;
143 143
 
144 144
         $app = new App($rootPath);
145 145
 
@@ -148,12 +148,12 @@  discard block
 block discarded – undo
148 148
 
149 149
         $app->instance($initializer->mockery_getName(), $initializer);
150 150
 
151
-        (function () use ($initializer) {
151
+        (function() use ($initializer) {
152 152
             $this->initializers = [$initializer->mockery_getName()];
153 153
         })->call($app);
154 154
 
155 155
         $env = m::mock(Env::class);
156
-        $env->shouldReceive('load')->once()->with($rootPath . '.env');
156
+        $env->shouldReceive('load')->once()->with($rootPath.'.env');
157 157
         $env->shouldReceive('get')->once()->with('config_ext', '.php')->andReturn('.php');
158 158
         $env->shouldReceive('get')->once()->with('app_debug')->andReturn($debug);
159 159
 
Please login to merge, or discard this patch.
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.