Changes between Version 7 and Version 8 of GSoC2008

Timestamp:

Jun 30, 2008, 4:21:54 PM (16 years ago)

Author:

mkurczych

Comment:

--

GSoC2008

@@ -22 +22 @@
 
 {{{
-<a href="/edit/1" py:if="user.autorized()">[edit]</a>
+<a href="/edit/1" py:if="user.authorized()">[edit]</a>
 <p>1111 11 11</p>
-<a href="/edit/2" py:if="user.autorized()">[edit]</a>
+<a href="/edit/2" py:if="user.authorized()">[edit]</a>
 <p>222 22 22 2</p>
-<a href="/edit/3" py:if="user.autorized()">[edit]</a>
+<a href="/edit/3" py:if="user.authorized()">[edit]</a>
 <p>333 33333 33 333</p>
 }}}
@@ -41 +41 @@
 }}}
 Now fragment depends only on variable "a", which can be only True or False, which gives optimization possibilities.
+
+Or it should be even vars = "a = bool(user.authorized())" to avoid things like that:
+
+{{{
+>>> class Foo(object):
+...     def __len__(self):
+...             print "Sideeffect"
+...             return 1
+...
+>>> f = Foo()
+>>> if f:
+...     print "Something"
+...
+Sideeffect
+Something
+}}}
+
+Foo can also change its internal state, and return different values in different ifs, which is not good. Only when variable is pure bool (a.__class__ is bool) we are safe. :-)