Passed
Push — master ( 48facb...9d5835 )
by 世昌
02:25
created
suda/src/application/database/DebugObserver.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -26,9 +26,9 @@
 block discarded – undo
26 26
         $status = $result ? 'OK' : 'Err';
27 27
         if ($result) {
28 28
             $effect = $statement->getStatement()->rowCount();
29
-            $this->debug->info('query ['.$status.'] '.$query.' '. number_format($timeSpend, 5).'s and effect '. $effect . ' rows');
29
+            $this->debug->info('query ['.$status.'] '.$query.' '.number_format($timeSpend, 5).'s and effect '.$effect.' rows');
30 30
         } else {
31
-            $this->debug->error('query ['.$status.'] '.$query.' '. number_format($timeSpend, 5).'s');
31
+            $this->debug->error('query ['.$status.'] '.$query.' '.number_format($timeSpend, 5).'s');
32 32
             $this->debug->error('query ['.$status.'] '.\implode(':', $statement->getStatement()->errorInfo()));
33 33
         }
34 34
     }
Please login to merge, or discard this patch.
suda/src/application/AppicationBase.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
      * @param string $name
83 83
      * @return \suda\application\Module|null
84 84
      */
85
-    public function find(string $name):?Module
85
+    public function find(string $name): ?Module
86 86
     {
87 87
         return $this->module->get($name);
88 88
     }
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
                 $this->modulePaths[] = Resource::getPathByRelativedPath($path, $this->path);
117 117
             }
118 118
         } else {
119
-            $this->modulePaths = [ Resource::getPathByRelativedPath('modules', $this->path) ];
119
+            $this->modulePaths = [Resource::getPathByRelativedPath('modules', $this->path)];
120 120
         }
121 121
     }
122 122
 
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
      */
140 140
     protected function className(string $name)
141 141
     {
142
-        return str_replace(['.','/'], '\\', $name);
142
+        return str_replace(['.', '/'], '\\', $name);
143 143
     }
144 144
 
145 145
     /**
Please login to merge, or discard this patch.
suda/src/framework/route/uri/MatcherHelper.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -30,9 +30,9 @@  discard block
 block discarded – undo
30 30
         // 添加忽略
31 31
         $url = preg_replace('/(\[)([^\[\]]+)(?(1)\])/', '(?:$2)?', $url);
32 32
         // 添加 * ? 匹配
33
-        $url = str_replace(['\*','\?'], ['[^/]*?','[^/]'], $url);
33
+        $url = str_replace(['\*', '\?'], ['[^/]*?', '[^/]'], $url);
34 34
         // 编译页面参数
35
-        $url = preg_replace_callback('/\{(\w+)(?:\:([^}]+?))?\}/', function ($match) use (&$parameters) {
35
+        $url = preg_replace_callback('/\{(\w+)(?:\:([^}]+?))?\}/', function($match) use (&$parameters) {
36 36
             $name = $match[1];
37 37
             $type = 'string';
38 38
             $extra = '';
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
         // 拆分参数
62 62
         list($mapper, $query, $parameter) = static::analyseParameter($matcher, $parameter);
63 63
         // for * ?
64
-        $url = \str_replace(['*','?'], ['','-'], $uri);
64
+        $url = \str_replace(['*', '?'], ['', '-'], $uri);
65 65
         // for ignore value
66 66
         $url = static::parseIgnoreableParameter($url, $matcher, $parameter, $mapper);
67 67
         $url = static::replaceParameter($url, $matcher, $parameter, $mapper);
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 
74 74
     protected static function parseIgnoreableParameter(string $url, UriMatcher  $matcher, array $parameter, array $mapper):string
75 75
     {
76
-        return preg_replace_callback('/\[(.+?)\]/', function ($match) use ($matcher, $parameter, $mapper) {
76
+        return preg_replace_callback('/\[(.+?)\]/', function($match) use ($matcher, $parameter, $mapper) {
77 77
             if (preg_match('/\{(\w+).+?\}/', $match[1])) {
78 78
                 $count = 0;
79 79
                 $subUrl = static::replaceParameter($match[1], $matcher, $parameter, $mapper, true, $count);
@@ -109,14 +109,14 @@  discard block
 block discarded – undo
109 109
 
110 110
     protected static function replaceParameter(string $input, UriMatcher $matcher, array $parameter, array $mapper, bool $ignore = false, ?int &$count = null)
111 111
     {
112
-        return preg_replace_callback('/\{(\w+).+?\}/', function ($match) use ($matcher, $parameter, $mapper, $ignore, &$count) {
112
+        return preg_replace_callback('/\{(\w+).+?\}/', function($match) use ($matcher, $parameter, $mapper, $ignore, &$count) {
113 113
             if (\array_key_exists($match[1], $mapper)) {
114
-                $count ++;
114
+                $count++;
115 115
                 return $mapper[$match[1]]->packValue($parameter[$match[1]]);
116 116
             }
117 117
             if ($default = $matcher->getParameter($match[1])) {
118 118
                 if ($default->hasDefault()) {
119
-                    $count ++;
119
+                    $count++;
120 120
                     return $default->getDefaultValue();
121 121
                 }
122 122
             }
Please login to merge, or discard this patch.
suda/src/framework/route/uri/parameter/StringParameter.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
 
25 25
     public function packValue(?string $matched)
26 26
     {
27
-        return $matched ? urlencode($matched): '';
27
+        return $matched ? urlencode($matched) : '';
28 28
     }
29 29
     
30 30
     /**
Please login to merge, or discard this patch.
suda/src/application/loader/ModuleLoader.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
      */
181 181
     protected function loadRouteGroup(string $groupName)
182 182
     {
183
-        $group = $groupName === 'default' ? '': '-'. $groupName;
183
+        $group = $groupName === 'default' ? '' : '-'.$groupName;
184 184
         if ($path = $this->module->getResource()->getConfigResourcePath('config/route'.$group)) {
185 185
             $routeConfig = Config::loadConfig($path, [
186 186
                 'module' => $this->module->getName(),
@@ -204,7 +204,7 @@  discard block
 block discarded – undo
204 204
      */
205 205
     protected function loadRouteConfig(string $prefix, string $groupName, array $routeConfig)
206 206
     {
207
-        $module =  $this->module->getFullName();
207
+        $module = $this->module->getFullName();
208 208
         foreach ($routeConfig as $name => $config) {
209 209
             $exname = $this->application->getRouteName($name, $module, $groupName);
210 210
             $method = $config['method'] ?? [];
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
             if ($anti) {
219 219
                 $uri = '/'.trim($uri, '/');
220 220
             } else {
221
-                $uri = '/'.trim($prefix . $uri, '/');
221
+                $uri = '/'.trim($prefix.$uri, '/');
222 222
             }
223 223
             $this->application->request($method, $exname, $uri, $attriute);
224 224
         }
Please login to merge, or discard this patch.
suda/src/framework/debug/attach/DumpTrait.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -64,11 +64,11 @@  discard block
 block discarded – undo
64 64
     protected static function valueToString(string $key, $value, int $deep):string
65 65
     {
66 66
         if (is_string($value) && strlen($value) > static::$dumpStringLength) {
67
-            return  $key.'='.json_encode(substr($value, 0, static::$dumpStringLength), JSON_UNESCAPED_UNICODE) .'... ,';
67
+            return  $key.'='.json_encode(substr($value, 0, static::$dumpStringLength), JSON_UNESCAPED_UNICODE).'... ,';
68 68
         } elseif (is_bool($value)) {
69 69
             return $key.'='.($value ? 'true' : 'false').' ,';
70 70
         } else {
71
-            return $key.'='.self::parameterToString($value, $deep - 1) .' ,';
71
+            return $key.'='.self::parameterToString($value, $deep - 1).' ,';
72 72
         }
73 73
     }
74 74
 
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
         $argsDump = '';
117 117
         if (!empty($trace['args'])) {
118 118
             foreach ($trace['args'] as $arg) {
119
-                $argsDump .= self::parameterToString($arg) .',';
119
+                $argsDump .= self::parameterToString($arg).',';
120 120
             }
121 121
             $argsDump = rtrim($argsDump, ',');
122 122
         }
@@ -126,8 +126,8 @@  discard block
 block discarded – undo
126 126
 
127 127
     public static function dumpThrowable(\Throwable $e)
128 128
     {
129
-        $dump = get_class($e).':'. $e->getMessage() .PHP_EOL;
130
-        $dump .= 'At: ' . $e->getFile().':'.$e->getLine().PHP_EOL;
129
+        $dump = get_class($e).':'.$e->getMessage().PHP_EOL;
130
+        $dump .= 'At: '.$e->getFile().':'.$e->getLine().PHP_EOL;
131 131
         $dump .= static::dumpTrace($e->getTrace());
132 132
         return $dump;
133 133
     }
Please login to merge, or discard this patch.
suda/src/application/exception/MissingTemplateException.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
     public function __construct(string $name, int $type = 0)
14 14
     {
15 15
         $this->name = $name;
16
-        parent::__construct(sprintf('missing template %s', $name, $type == 0 ? 'source':'dest'));
16
+        parent::__construct(sprintf('missing template %s', $name, $type == 0 ? 'source' : 'dest'));
17 17
     }
18 18
 
19 19
 
Please login to merge, or discard this patch.
suda/src/application/template/CompilableTemplate.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 
107 107
     protected function getStaticOutpath()
108 108
     {
109
-        $path = $this->config['assets-public'] ?? \constant('SUDA_PUBLIC').'/assets/'. $this->getStaticName();
109
+        $path = $this->config['assets-public'] ?? \constant('SUDA_PUBLIC').'/assets/'.$this->getStaticName();
110 110
         FileSystem::make($path);
111 111
         return $path;
112 112
     }
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
     {
121 121
         $output = $this->config['output'] ?? \constant('SUDA_DATA').'/template';
122 122
         FileSystem::make($output);
123
-        return $output .'/'. $this->name.'-'.substr(md5_file($this->getSourcePath()), 10, 8).'.php';
123
+        return $output.'/'.$this->name.'-'.substr(md5_file($this->getSourcePath()), 10, 8).'.php';
124 124
     }
125 125
 
126 126
     /**
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
      *
129 129
      * @return string|null
130 130
      */
131
-    public function getSourcePath():?string
131
+    public function getSourcePath(): ?string
132 132
     {
133 133
         return $this->source ?? null;
134 134
     }
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
      *
150 150
      * @return void
151 151
      */
152
-    protected function compile(){
152
+    protected function compile() {
153 153
         $sourcePath = $this->getSourcePath();
154 154
         $destPath = $this->getPath();
155 155
         if ($sourcePath === null) {
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
     public function include(string $path)
203 203
     {
204 204
         $subfix = $this->config['subfix'] ?? '';
205
-        $included = new self(Resource::getPathByRelativedPath($path. $subfix, dirname($this->source)), $this->config);
205
+        $included = new self(Resource::getPathByRelativedPath($path.$subfix, dirname($this->source)), $this->config);
206 206
         $included->parent = $this;
207 207
         echo $included->getRenderedString();
208 208
     }
@@ -219,13 +219,13 @@  discard block
 block discarded – undo
219 219
     {
220 220
         $this->prepareStaticSource();
221 221
         if (\array_key_exists('assets-prefix', $this->config)) {
222
-            $prefix = $this->config['assets-prefix'] ;
222
+            $prefix = $this->config['assets-prefix'];
223 223
         } elseif (defined('SUDA_ASSETS')) {
224 224
             $prefix = \constant('SUDA_ASSETS');
225 225
         } else {
226 226
             $prefix = '/assets';
227 227
         }
228
-        return $prefix .'/'.$this->getStaticName();
228
+        return $prefix.'/'.$this->getStaticName();
229 229
     }
230 230
 
231 231
     protected function prepareStaticSource()
Please login to merge, or discard this patch.
suda/src/application/template/ModuleTemplate.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
      *
27 27
      * @return string|null
28 28
      */
29
-    public function getSourcePath():?string
29
+    public function getSourcePath(): ?string
30 30
     {
31 31
         $subfix = $this->config['subfix'] ?? '.tpl.html';
32 32
         return $this->getResource($this->module)->getResourcePath($this->getTemplatePath().'/'.$this->name.$subfix);
@@ -39,9 +39,9 @@  discard block
 block discarded – undo
39 39
      */
40 40
     public function getPath()
41 41
     {
42
-        $output = $this->config['output'] ?? \constant('SUDA_DATA').'/template/'. $this->uriName;
42
+        $output = $this->config['output'] ?? \constant('SUDA_DATA').'/template/'.$this->uriName;
43 43
         FileSystem::make($output);
44
-        return $output .'/'. $this->name.'.php';
44
+        return $output.'/'.$this->name.'.php';
45 45
     }
46 46
 
47 47
     /**
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
             }
70 70
             return $this->application->getUrl($this->request, $name, $values ?? [], true, $this->module, $this->group);
71 71
         } elseif (is_array($name) && \is_string($defaultName)) {
72
-            return $this->application->getUrl($this->request, $defaultName, array_merge($this->request->get() ?? [], $name) , true, $this->module, $this->group);
72
+            return $this->application->getUrl($this->request, $defaultName, array_merge($this->request->get() ?? [], $name), true, $this->module, $this->group);
73 73
         } elseif (is_string($defaultName)) {
74 74
             return $this->application->getUrl($this->request, $defaultName, $this->request->get() ?? [], true, $this->module, $this->group);
75 75
         }
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
         if ($this->request->getAttribute('route') === $full) {
89 89
             if (\is_array($parameter)) {
90 90
                 foreach ($parameter as $key => $value) {
91
-                    if ($this->request->getQuery($key) != $value){
91
+                    if ($this->request->getQuery($key) != $value) {
92 92
                         return false;
93 93
                     }
94 94
                 }
Please login to merge, or discard this patch.