[Bug 5445839] route params as object instead of

query string. Patch converted string to object in
the wrong place (throws on every request). Mis-
leading code in mojito-router.js removed, fix was
applied to route-maker.common.js

source/lib/app/autoload/route-maker.common.js

@@ -135,7 +135,9 @@ YUI.add('mojito-route-maker', function(Y, NAME) {
                 }
             }
 
-            route.params = Y.QueryString.parse(route.params);
+            if (typeof route.params !== 'object') {
+                route.params = Y.QueryString.parse(String(route.params));
+            }
 
             build = [];
             for (i in route.requires) {

source/lib/app/middleware/mojito-router.js

@@ -8,8 +8,7 @@
 /*jslint anon:true, sloppy:true, nomen:true*/
 
 
-var qs = require('querystring'),
-    logger,
+var logger,
     liburl = require('url'),
     RX_END_SLASHES = /\/+$/,
     NAME = 'UriRouter';
@@ -99,9 +98,12 @@ Router.prototype = {
             // of instance
             // command.action = routeMatch.call[1];
             command.context = req.context;
+
+            //routeMatch.param is converted to object in route-maker.common.js
+            //and is never a string here. i.e. this assert always passes:
+            //require('assert').ok(typeof routeMatch.param !== 'string');
             command.params = {
-                route: simpleMerge(routeMatch.query,
-                    qs.parse(routeMatch.param)),
+                route: simpleMerge(routeMatch.query, routeMatch.param),
                 url: query || {},
                 body: req.body || {},
                 file: {} // FUTURE: add multi-part file data here