[Zope-Checkins] SVN: Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/ Revert a change to a unit test that would assert that the results of hybrid

Philipp von Weitershausen philikon at philikon.de
Thu May 25 15:45:48 EDT 2006


Log message for revision 68285:
  Revert a change to a unit test that would assert that the results of hybrid
  path expressions where the alternate expression wasn't a PathExpr is not called.
  See http://www.zope.org/Collectors/Zope/538, http://svn.zope.org/?view=rev&rev=22855
  and http://www.zope.org/Collectors/Zope3-dev/638.
  

Changed:
  U   Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/Expressions.py
  U   Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/tests/testExpressions.py

-=-
Modified: Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/Expressions.py
===================================================================
--- Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/Expressions.py	2006-05-25 19:43:25 UTC (rev 68284)
+++ Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/Expressions.py	2006-05-25 19:45:47 UTC (rev 68285)
@@ -116,6 +116,8 @@
         else:
             # On the last subexpression allow exceptions through.
             ob = self._subexprs[-1](econtext)
+            if self._hybrid:
+                return ob
 
         if self._name == 'nocall':
             return ob

Modified: Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/tests/testExpressions.py
===================================================================
--- Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/tests/testExpressions.py	2006-05-25 19:43:25 UTC (rev 68284)
+++ Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/tests/testExpressions.py	2006-05-25 19:45:47 UTC (rev 68285)
@@ -76,7 +76,7 @@
         '''Test hybrid path expressions'''
         ec = self.ec
         assert ec.evaluate('x | python:1+1') == 2
-        assert ec.evaluate('x | python:int') == 0
+        assert ec.evaluate('x | python:int') == int
         assert ec.evaluate('x | string:x') == 'x'
         assert ec.evaluate('x | string:$one') == '1'
         assert ec.evaluate('x | not:exists:x')



More information about the Zope-Checkins mailing list