Merge pull request tildeio#17 from mhelabs/reset-router

Router#reset exits & clears current route handlers

dist/router.amd.js

@@ -50,6 +50,21 @@ define("router",
         return this.recognizer.hasRoute(route);
       },
 
+      /**
+        Clears the current and target route handlers and triggers exit
+        on each of them starting at the leaf and traversing up through
+        its ancestors.
+      */
+      reset: function() {
+        eachHandler(this.currentHandlerInfos, function(handler) {
+          if (handler.exit) {
+            handler.exit();
+          }
+        });
+        this.currentHandlerInfos = null;
+        this.targetHandlerInfos = null;
+      },
+
       /**
         The entry point for handling a change to the URL (usually
         via the back and forward button).

dist/router.cjs.js

@@ -48,6 +48,21 @@ Router.prototype = {
     return this.recognizer.hasRoute(route);
   },
 
+  /**
+    Clears the current and target route handlers and triggers exit
+    on each of them starting at the leaf and traversing up through
+    its ancestors.
+  */
+  reset: function() {
+    eachHandler(this.currentHandlerInfos, function(handler) {
+      if (handler.exit) {
+        handler.exit();
+      }
+    });
+    this.currentHandlerInfos = null;
+    this.targetHandlerInfos = null;
+  },
+
   /**
     The entry point for handling a change to the URL (usually
     via the back and forward button).

dist/router.js

@@ -48,6 +48,21 @@
       return this.recognizer.hasRoute(route);
     },
 
+    /**
+      Clears the current and target route handlers and triggers exit
+      on each of them starting at the leaf and traversing up through
+      its ancestors.
+    */
+    reset: function() {
+      eachHandler(this.currentHandlerInfos, function(handler) {
+        if (handler.exit) {
+          handler.exit();
+        }
+      });
+      this.currentHandlerInfos = null;
+      this.targetHandlerInfos = null;
+    },
+
     /**
       The entry point for handling a change to the URL (usually
       via the back and forward button).

lib/router.js

@@ -48,6 +48,21 @@ Router.prototype = {
     return this.recognizer.hasRoute(route);
   },
 
+  /**
+    Clears the current and target route handlers and triggers exit
+    on each of them starting at the leaf and traversing up through
+    its ancestors.
+  */
+  reset: function() {
+    eachHandler(this.currentHandlerInfos, function(handler) {
+      if (handler.exit) {
+        handler.exit();
+      }
+    });
+    this.currentHandlerInfos = null;
+    this.targetHandlerInfos = null;
+  },
+
   /**
     The entry point for handling a change to the URL (usually
     via the back and forward button).

tests/tests.js

@@ -1703,3 +1703,31 @@ test("A final handler can specify an additional non-routable handler", function(
   router.handleURL("/index");
   router.transitionTo('showPost', { id: 1 });
 });
+
+test("reset exits and clears the current and target route handlers", function() {
+  var postIndexExited = false;
+  var showAllPostsExited = false;
+
+  var postIndexHandler = {
+    exit: function() {
+      postIndexExited = true;
+    }
+  };
+  var showAllPostsHandler = {
+    exit: function() {
+      showAllPostsExited = true;
+    }
+  };
+  handlers = {
+    postIndex: postIndexHandler,
+    showAllPosts: showAllPostsHandler
+  };
+
+  router.handleURL("/posts/all");
+  router.reset();
+
+  ok(postIndexExited, "Post index handler did not exit");
+  ok(showAllPostsExited, "Show all posts handler did not exit");
+  equal(router.currentHandlerInfos, null, "currentHandlerInfos should be null");
+  equal(router.targetHandlerInfos, null, "targetHandlerInfos should be null");
+});