Bug Fixes

---------

- When a route with the same name as a previously registered route was
  added, the old route was not removed from the mapper's routelist.
  Symptom: the old registered route would be used (and possibly
  matched) during route lookup when it should not have had a chance to
  ever be used.

1 files changed, 3 insertions, 0 deletions

diff --git a/repoze/bfg/urldispatch.py b/repoze/bfg/urldispatch.py
index c577dfa9e..becde3ea2 100644
--- a/repoze/bfg/urldispatch.py
+++ b/repoze/bfg/urldispatch.py
@@ -28,6 +28,9 @@ class RoutesMapper(object):
         return self.routelist
 
     def connect(self, path, name, factory=None, predicates=()):
+        if name in self.routes:
+            oldroute = self.routes[name]
+            self.routelist.remove(oldroute)
         route = Route(path, name, factory, predicates)
         self.routelist.append(route)
         self.routes[name] = route