fix(UrlMatcher): query param function defaults

angular-ui · Apr 20, 2014 · f9c2053 · f9c2053
1 parent ed844c6
commit f9c2053
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/urlMatcherFactory.js b/src/urlMatcherFactory.js
@@ -230,7 +230,7 @@ UrlMatcher.prototype.exec = function (path, searchParams) {
   for (/**/; i < nTotal; i++) {
     param = params[i];
     cfg = this.params[param];
-    values[param] = isDefined(searchParams[param]) ? cfg.type.decode(searchParams[param]) : cfg.value;
+    values[param] = cfg.$value(searchParams[param]);
   }
 
   return values;

diff --git a/test/urlMatcherFactorySpec.js b/test/urlMatcherFactorySpec.js
@@ -356,6 +356,15 @@ describe("urlMatcherFactory", function () {
           }
         });
         expect(m.exec('/foo').bar).toBe("Value from bar()");
+
+        var m = new UrlMatcher('/foo?bar', {
+          params: {
+            bar: function() {
+              return "Value from bar()";
+            }
+          }
+        });
+        expect(m.exec('/foo').bar).toBe("Value from bar()");
       });
 
       it("should allow injectable functions", inject(function($stateParams) {