codeburnerframework / routing

src/Collectors/ResourceCollectorTrait.php

Braces +3 added lines, -2 removed lines

@@ -76,8 +76,9 @@
     public function resources(array $controllers)
     {
         $resource = new Resource;
-        foreach ($controllers as $controller)
-            $resource->set($this->resource($controller));
+        foreach ($controllers as $controller) {
+                    $resource->set($this->resource($controller));
+        }
         return $resource;
     }
 

src/Resource.php

Braces +3 added lines, -1 removed lines

@@ -173,7 +173,9 @@
         foreach ($segments as $index => $segment) {
             if (strpos($segment, "{") === 0) {
                    $pattern .= "/{" . $segments[$index - 1] . "_" . ltrim($segment, "{");
-            } else $pattern .= $segment;
+            } else {
+            	$pattern .= $segment;
+            }
         }
 
         return $pattern;

src/Strategies/EnhancerAbstractStrategy.php

Braces +3 added lines, -1 removed lines

@@ -41,7 +41,9 @@
     {
         if ($route->hasMetadata($this->metadataStrategyKey)) {
                $route->setStrategy($route->getMetadata($this->metadataStrategyKey));
-        } else $route->setStrategy(null);
+        } else {
+        	$route->setStrategy(null);
+        }
 
         $this->enhance($route);
 

src/Collector.php

Braces +21 added lines, -11 removed lines

@@ -117,7 +117,9 @@ 
             if (strpos($pattern, "{") !== false) {
                    $index = $this->getDynamicIndex($method, $pattern);
                    $this->dynamics[$index][$pattern] = $route;
-            } else $this->statics[$method][$pattern] = $route;
+            } else {
+            	$this->statics[$method][$pattern] = $route;
+            }
         }
 
         return $group;
@@ -142,8 +144,9 @@ 
     public function match(array $methods, $pattern, $action)
     {
         $group = new Group;
-        foreach ($methods as $method)
-            $group->set($this->set($method, $pattern, $action));
+        foreach ($methods as $method) {
+                    $group->set($this->set($method, $pattern, $action));
+        }
         return $group;
     }
 
@@ -186,8 +189,9 @@ 
     public function group(array $routes)
     {
         $group = new Group;
-        foreach ($routes as $route)
-            $group->set($route);
+        foreach ($routes as $route) {
+                    $group->set($route);
+        }
         return $group;
     }
 
@@ -202,7 +206,9 @@ 
     {
         if (strpos($pattern, "{") === false) {
                unset($this->statics[$method][$pattern]);
-        } else unset($this->dynamics[$this->getDynamicIndex($method, $pattern)][$pattern]);
+        } else {
+        	unset($this->dynamics[$this->getDynamicIndex($method, $pattern)][$pattern]);
+        }
     }
 
     /**
@@ -257,7 +263,9 @@ 
         if ($closingNumber !== count($segments) - 1) {
             if (preg_match("~" . self::DYNAMIC_REGEX . "(*SKIP)(*F)|\]~x", $withoutClosing)) {
                    throw new BadRouteException(BadRouteException::OPTIONAL_SEGMENTS_ON_MIDDLE);
-            } else throw new BadRouteException(BadRouteException::UNCLOSED_OPTIONAL_SEGMENTS);
+            } else {
+            	throw new BadRouteException(BadRouteException::UNCLOSED_OPTIONAL_SEGMENTS);
+            }
         }
     }
 
@@ -296,8 +304,9 @@ 
     public function findStaticRoute($method, $pattern)
     {
         $method = strtolower($method);
-        if (isset($this->statics[$method]) && isset($this->statics[$method][$pattern]))
-            return $this->statics[$method][$pattern];
+        if (isset($this->statics[$method]) && isset($this->statics[$method][$pattern])) {
+                    return $this->statics[$method][$pattern];
+        }
         return false;
     }
 
@@ -333,8 +342,9 @@ 
     public function getWildcards()
     {
         $wildcards = [];
-        foreach ($this->wildcards as $token => $regex)
-            $wildcards[substr($token, 1)] = substr($regex, 1);
+        foreach ($this->wildcards as $token => $regex) {
+                    $wildcards[substr($token, 1)] = substr($regex, 1);
+        }
         return $wildcards;
     }
 

src/Collectors/ControllerCollectorTrait.php

Braces +9 added lines, -5 removed lines

@@ -69,8 +69,9 @@ 
     public function controllers(array $controllers)
     {
         $group = new Group;
-        foreach ($controllers as $controller)
-            $group->set($this->controller($controller));
+        foreach ($controllers as $controller) {
+                    $group->set($this->controller($controller));
+        }
         return $group;
     }
 
@@ -100,8 +101,9 @@ 
     public function controllersWithoutPrefix(array $controllers)
     {
         $group = new Group;
-        foreach ($controllers as $controller)
-            $group->set($this->controllerWithoutPrefix($controller));
+        foreach ($controllers as $controller) {
+                    $group->set($this->controllerWithoutPrefix($controller));
+        }
         return $group;
     }
 
@@ -134,7 +136,9 @@ 
 
                 if ($strategy !== null) {
                        $route->setStrategy($strategy);
-                } else $route->setStrategy($controllerDefaultStrategy);
+                } else {
+                	$route->setStrategy($controllerDefaultStrategy);
+                }
 
                 $group->set($route);
             }

src/Group.php

Braces +39 added lines, -25 removed lines

@@ -37,9 +37,12 @@ 
     public function set($route)
     {
         if ($route instanceof Group) {
-            foreach ($route->all() as $r)
-                $this->routes[] = $r;
-        } else  $this->routes[] = $route;
+            foreach ($route->all() as $r) {
+                            $this->routes[] = $r;
+            }
+        } else {
+        	$this->routes[] = $route;
+        }
         return  $this;
     }
 
@@ -88,8 +91,9 @@ 
 
     public function forget()
     {
-        foreach ($this->routes as $route)
-            $route->forget();
+        foreach ($this->routes as $route) {
+                    $route->forget();
+        }
         return $this;
     }
 
@@ -102,8 +106,9 @@ 
 
     public function setMethod($method)
     {
-        foreach ($this->routes as $route)
-            $route->setMethod($method);
+        foreach ($this->routes as $route) {
+                    $route->setMethod($method);
+        }
         return $this;
     }
 
@@ -116,8 +121,9 @@ 
 
     public function setAction($action)
     {
-        foreach ($this->routes as $route)
-            $route->setAction($action);
+        foreach ($this->routes as $route) {
+                    $route->setAction($action);
+        }
         return $this;
     }
 
@@ -130,8 +136,9 @@ 
 
     public function setNamespace($namespace)
     {
-        foreach ($this->routes as $route)
-            $route->setNamespace($namespace);
+        foreach ($this->routes as $route) {
+                    $route->setNamespace($namespace);
+        }
         return $this;
     }
 
@@ -145,8 +152,9 @@ 
     public function setPrefix($prefix)
     {
         $prefix = "/" . ltrim($prefix, "/");
-        foreach ($this->routes as $route)
-            $route->setPattern(rtrim($prefix . $route->getPattern(), "/"));
+        foreach ($this->routes as $route) {
+                    $route->setPattern(rtrim($prefix . $route->getPattern(), "/"));
+        }
         return $this;
     }
 
@@ -161,8 +169,9 @@ 
 
     public function setMetadata($key, $value)
     {
-        foreach ($this->routes as $route)
-            $route->setMetadata($key, $value);
+        foreach ($this->routes as $route) {
+                    $route->setMetadata($key, $value);
+        }
         return $this;
     }
 
@@ -175,8 +184,9 @@ 
 
     public function setMetadataArray(array $metadata)
     {
-        foreach ($this->routes as $route)
-            $route->setMetadataArray($metadata);
+        foreach ($this->routes as $route) {
+                    $route->setMetadataArray($metadata);
+        }
         return $this;
     }
 
@@ -189,8 +199,9 @@ 
 
     public function setDefaults(array $defaults)
     {
-        foreach ($this->routes as $route)
-            $route->setDefaults($defaults);
+        foreach ($this->routes as $route) {
+                    $route->setDefaults($defaults);
+        }
         return $this;
     }
 
@@ -205,8 +216,9 @@ 
 
     public function setDefault($key, $value)
     {
-        foreach ($this->routes as $route)
-            $route->setDefault($key, $value);
+        foreach ($this->routes as $route) {
+                    $route->setDefault($key, $value);
+        }
         return $this;
     }
 
@@ -219,8 +231,9 @@ 
 
     public function setStrategy($strategy)
     {
-        foreach ($this->routes as $route)
-            $route->setStrategy($strategy);
+        foreach ($this->routes as $route) {
+                    $route->setStrategy($strategy);
+        }
         return $this;
     }
 
@@ -235,8 +248,9 @@ 
 
     public function setConstraint($name, $regex)
     {
-        foreach ($this->routes as $route)
-            $route->setConstraint($name, $regex);
+        foreach ($this->routes as $route) {
+                    $route->setConstraint($name, $regex);
+        }
         return $this;
     }
 

src/Route.php

Braces +3 added lines, -1 removed lines

@@ -208,7 +208,9 @@
 
         if ($this->controllerCreationFunction === null) {
                return new $controller;
-        } else return call_user_func($this->controllerCreationFunction, $controller);
+        } else {
+        	return call_user_func($this->controllerCreationFunction, $controller);
+        }
     }
 
     /**