Log message for revision 76775:
  Added explanation about the last test failure that happens with 
providerexpressions / viewlets.
  

Changed:
  U   
Products.Five/branches/philikon-aq-and-__parent__/browser/providerexpression.py

-=-
Modified: 
Products.Five/branches/philikon-aq-and-__parent__/browser/providerexpression.py
===================================================================
--- 
Products.Five/branches/philikon-aq-and-__parent__/browser/providerexpression.py 
    2007-06-18 23:12:16 UTC (rev 76774)
+++ 
Products.Five/branches/philikon-aq-and-__parent__/browser/providerexpression.py 
    2007-06-18 23:16:56 UTC (rev 76775)
@@ -24,6 +24,11 @@
         if provider is None:
             raise cp_interfaces.ContentProviderLookupError(name)
 
+        # XXX We can either wrap this in the context and have three test
+        # failures in directives.txt or wrap it in the view (aka our
+        # __parent__) and have one test failure in provider.txt which also
+        # happens when we don't wrap this at all anymore :(
+        # Removing all the AQ-wrapping is probably the way to go here.
         if getattr(provider, '__of__', None) is not None:
             provider = provider.__of__(context)
 

_______________________________________________
Zope-Checkins maillist  -  Zope-Checkins@zope.org
http://mail.zope.org/mailman/listinfo/zope-checkins

Reply via email to