Completed
Push — master ( f19f8c...ffee04 )
by Dan
02:51
created
Src/Router/Loaders/YamlLoader.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -68,7 +68,7 @@
 block discarded – undo
68 68
                         $collection->addRoute($routeMethod, $route['path'], $route['handler'], $route['names']);
69 69
                     }
70 70
                 }
71
-            } catch (ParseException $e) {
71
+            }catch (ParseException $e) {
72 72
                 throw new RouterException($e->getMessage());
73 73
             }
74 74
 
Please login to merge, or discard this patch.
Src/Router/Dispatcher/Dispatcher.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
 
117 117
         try {
118 118
             $resolvedHandler['controller'] = $this->_findClass($controllerMethod[0]);
119
-        } catch (\Exception $e) {
119
+        }catch (\Exception $e) {
120 120
             throw new DispatchException($e->getMessage());
121 121
         }
122 122
 
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
 
168 168
         try {
169 169
             $this->_getParamsFromVariableName($args);
170
-        } catch (\Exception $e) {
170
+        }catch (\Exception $e) {
171 171
             $this->_getParamsFromTypeHint($args);
172 172
         }
173 173
 
@@ -293,12 +293,12 @@  discard block
 block discarded – undo
293 293
         try {
294 294
             $this->_checkWhiteList($controller, $this->options);
295 295
             $this->_checkBlackList($controller, $this->options);
296
-        } catch (\Exception $e) {
296
+        }catch (\Exception $e) {
297 297
             unset($controllerObj);
298 298
             throw new DispatchException($e->getMessage());
299 299
         }
300 300
 
301
-        if (!method_exists($controllerObj,$method)){
301
+        if (!method_exists($controllerObj, $method)) {
302 302
             throw new DispatchException('Controller method does not exist');
303 303
         }
304 304
 
Please login to merge, or discard this patch.
Tests/Router/RouteCollectionTest.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -33,9 +33,9 @@  discard block
 block discarded – undo
33 33
     {
34 34
         $handler = 'my-handler';
35 35
         $routes = $this->collection;
36
-        $routes->group('/path', function () use ($routes, $handler) {
36
+        $routes->group('/path', function() use ($routes, $handler) {
37 37
             $routes->addRoute(['GET', 'POST'], '/foo', $handler, ['global']);
38
-            $routes->group('/sub-dir', function () use ($routes, $handler) {
38
+            $routes->group('/sub-dir', function() use ($routes, $handler) {
39 39
                 $routes->addRoute(['GET'], '/sub-page', $handler, ['sub-dir']);
40 40
             });
41 41
         });
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
     public function testMergeCollection()
176 176
     {
177 177
         $handler = 'handler::string';
178
-        $names = ['routes','names'];
178
+        $names = ['routes', 'names'];
179 179
         $collection = new RouteCollection();
180 180
         $collection->addRoute('GET', '/path', $handler, $names);
181 181
         $collection->addRoute('GET', '/another', $handler, $names);
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
     {
195 195
         $this->setExpectedException(RouteException::class);
196 196
         $handler = 'handler::string';
197
-        $names = ['routes','names'];
197
+        $names = ['routes', 'names'];
198 198
         $collection = new RouteCollection();
199 199
         $collection->addRoute('GET', '/path', $handler, $names);
200 200
         $collection->addRoute('GET', '/another', $handler, $names);
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
         $rawPattern = '/mypattern/';
241 241
         $expectedPattern = '/' . \ltrim($rawPattern, '/');
242 242
         $expected = \implode('', $groups) . $expectedPattern;
243
-        $expected = \rtrim($expected,'/');
243
+        $expected = \rtrim($expected, '/');
244 244
 
245 245
         $this->assertEquals($expected, $collection->formatRoutePattern($rawPattern));
246 246
     }
@@ -250,11 +250,11 @@  discard block
 block discarded – undo
250 250
      */
251 251
     public function testMergeGroupNames()
252 252
     {
253
-        $expected = ['route-name','group-name'];
253
+        $expected = ['route-name', 'group-name'];
254 254
         $handler = 'myhandler';
255 255
         $routes = $this->collection;
256 256
 
257
-        $routes->group('/path', function () use ($routes, $handler) {
257
+        $routes->group('/path', function() use ($routes, $handler) {
258 258
             $routes->addRoute(['GET', 'POST'], '/foo', $handler, ['route-name']);
259 259
         }, ['group-name']);
260 260
 
Please login to merge, or discard this patch.