[BUGFIX] Optimisations + bugfixes.

src/Pecee/SimpleRouter/RouterBase.php

@@ -80,11 +80,13 @@ class RouterBase {
             }
 
             $this->currentRoute = $route;
+
             if($route instanceof RouterGroup && is_callable($route->getCallback())) {
                 $route->renderRoute($this->request);
                 $activeGroup = $route;
-                $mergedSettings = array_merge($settings, $route->getMergeableSettings());
+                $mergedSettings = array_merge($route->getMergeableSettings(), $settings);
             }
+
             $this->currentRoute = null;
 
             if(count($this->backstack)) {

src/Pecee/SimpleRouter/RouterEntry.php

@@ -136,7 +136,7 @@ abstract class RouterEntry {
      * @return mixed
      */
     public function getParameters(){
-        return $this->parameters;
+        return ($this->parameters === null) ? array() : $this->parameters;
     }
 
     /**
@@ -254,7 +254,8 @@ abstract class RouterEntry {
             $parameterRegex = $this->parametersRegex[$parameter];
         }
 
-        for($i = 0; $i < strlen($route); $i++) {
+        $routeLength = strlen($route);
+        for($i = 0; $i < $routeLength; $i++) {
 
             $character = $route[$i];
 

src/Pecee/SimpleRouter/RouterRoute.php

@@ -64,7 +64,6 @@ class RouterRoute extends RouterEntry {
      * @return self
      */
     public function setUrl($url) {
-
         $parameters = array();
         $matches = array();
 
@@ -72,12 +71,12 @@ class RouterRoute extends RouterEntry {
             $parameters = $matches[1];
         }
 
-        $max = count($parameters);
+        if(count($parameters)) {
-        if($max) {
+            $tmp = array();
-            for($i = 0; $i < $max; $i++) {
+            foreach($parameters as $param) {
-                $param = $parameters[$i];
+                $tmp[$param] = '';
-                $this->parameters[$param] = '';
             }
+            $this->parameters = $tmp;
         }
 
         $this->url = $url;