[Zope-Checkins] SVN: Products.Five/branches/philikon-aq-and-__parent__/browser/providerexpression.py Added explanation about the last test failure that happens with providerexpressions / viewlets.

Hanno Schlichting plone at hannosch.info
Mon Jun 18 19:16:56 EDT 2007


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)
 



More information about the Zope-Checkins mailing list