feat($urlRouter): force URLs to have valid params

Author: nateabele

src/urlMatcherFactory.js

@@ -218,6 +218,28 @@ UrlMatcher.prototype.parameters = function () {
   return keys(this.params);
 };
 
+/**
+ * @ngdoc function
+ * @name ui.router.util.type:UrlMatcher#validate
+ * @methodOf ui.router.util.type:UrlMatcher
+ *
+ * @description
+ * Checks an object hash of parameters to validate their correctness according to the parameter
+ * types of this `UrlMatcher`.
+ *
+ * @param {Object} params The object hash of parameters to validate.
+ * @returns {Boolean} Returns `true` if `params` validates, otherwise `false`.
+ */
+UrlMatcher.prototype.validates = function (params) {
+  var result = true, self = this;
+
+  forEach(params, function(val, key) {
+    if (!self.params[key]) return;
+    result = result && self.params[key].is(val);
+  });
+  return result;
+}
+
 /**
  * @ngdoc function
  * @name ui.router.util.type:UrlMatcher#format

src/urlRouter.js

@@ -292,8 +292,11 @@ function $UrlRouterProvider(   $locationProvider,   $urlMatcherFactory) {
       },
 
       href: function(urlMatcher, params, options) {
+        if (!urlMatcher.validates(params)) return null;
+
         var isHtml5 = $locationProvider.html5Mode();
         var url = urlMatcher.format(params);
+        options = options || {};
 
         if (!isHtml5 && url) {
           url = "#" + $locationProvider.hashPrefix() + url;

test/urlRouterSpec.js

@@ -146,6 +146,20 @@ describe("UrlRouter", function () {
         expect($location.url()).toBe('/old');
       }));
     });
+
+    describe("URL generation", function() {
+      it("should return null when UrlMatcher rejects parameters", inject(function($urlRouter, $urlMatcherFactory) {
+        $urlMatcherFactory.type("custom", {
+          is: function(val) {
+            return val === 1138;
+          }
+        });
+        var matcher = new UrlMatcher("/foo/{param:custom}");
+
+        expect($urlRouter.href(matcher, { param: 1138 })).toBe('#/foo/1138');
+        expect($urlRouter.href(matcher, { param: 5 })).toBeNull();
+      }));
+    });
   });
 
 });