Split functionality, since not all request types use the concept of

views.

CHANGES.txt

@@ -2,14 +2,22 @@
 CHANGES
 =======
 
+0.2.2 (2007-12-??)
+------------------
+
+- Restructuring: Separated pluggable traverser functionality into two classes
+  for better code reuse.
+
+
 0.2.1 (2007-11-92)
 ------------------
 
-- bugfix: if viewlet and managers get nested a viewlet was not found if
+- Bugfix: if viewlet and managers get nested a viewlet was not found if
   the depth reaches 3 because the context was set to the page and not
-  to the context object
-- bugfix: replaced call to _getContextName because it has been removed from
-  absoluteURL
+  to the context object.
+
+- Bugfix: replaced call to ``_getContextName`` because it has been removed
+  from ``absoluteURL``.
 
 
 0.2.0 (2007-10-31)

src/z3c/traverser/traverser.py

@@ -24,26 +24,38 @@
 
 _marker = object()
 
-class PluggableTraverser(object):
-    """Generic Pluggable Traverser."""
 
+class BasePluggableTraverser(object):
     implements(interfaces.IPluggableTraverser)
 
     def __init__(self, context, request):
         self.context = context
         self.request = request
 
     def publishTraverse(self, request, name):
-        # 1. Look at all the traverser plugins, whether they have an answer.
+        # Look at all the traverser plugins, whether they have an answer.
         for traverser in subscribers((self.context, request),
                                      interfaces.ITraverserPlugin):
             try:
                 return traverser.publishTraverse(request, name)
             except NotFound:
                 pass
 
-        # 2. The traversers did not have an answer, so let's see whether it is
-        #    a view.
+        raise NotFound(self.context, name, request)
+
+
+class PluggableTraverser(BasePluggableTraverser):
+    """Generic Pluggable Traverser."""
+
+    def publishTraverse(self, request, name):
+        try:
+            return super(PluggableTraverser, self).publishTraverse(
+                request, name)
+        except NotFound:
+            pass
+
+        # The traversers did not have an answer, so let's see whether it is a
+        # view.
         view = queryMultiAdapter((self.context, request), name=name)
         if view is not None:
             return view