TurboGears /.-ed, >new == True< or >new == "True"<

[Andy Leszczynski]

watch this:
http://www.turbogears.org.nyud.net:8...MinuteWiki.mov

or read this:
http://www.turbogears.org.nyud.net:8090/docs/wiki2 0/page4.html

should not it be:

2 def save(self, pagename, data, submit, new):
3 hub.begin()
4 if new == True:
5 page = Page(pagename=pagename, data=data)
6 else:
7 page = Page.byPagename(pagename)
8 page.data = data

instead of:

4 if new == "True":

[Fredrik Lundh]

Andy Leszczynski wrote:

watch this:
http://www.turbogears.org.nyud.net:8...MinuteWiki.mov

or read this:
http://www.turbogears.org.nyud.net:8090/docs/wiki2 0/page4.html

should not it be:

2 def save(self, pagename, data, submit, new):
3 hub.begin()
4 if new == True:
5 page = Page(pagename=pagename, data=data)
6 else:
7 page = Page.byPagename(pagename)
8 page.data = data

instead of:

4 if new == "True":

from page 5:

http://www.turbogears.org.nyud.net:8...i20/page5.html

Converting incoming arguments

Something that's a little ugly in that save method is the
'if new == "True"'. Wouldn't it be nicer to just use the
more pythonic 'if new'? Piece of cake... we just need to
use a validator. /.../

</F>

[Sybren Stuvel]

Andy Leszczynski enlightened us with:

should not it be:

2 def save(self, pagename, data, submit, new):
3 hub.begin()
4 if new == True:
5 page = Page(pagename=pagename, data=data)
6 else:
7 page = Page.byPagename(pagename)
8 page.data = data

instead of:

4 if new == "True":

No it should not. The values passed to the function are the strings
passed by the GET request, hence all strings. There are methods of
dealing with this - read the rest of the documentation.

Sybren
--
The problem with the world is stupidity. Not saying there should be a
capital punishment for stupidity, but why don't we just take the
safety labels off of everything and let the problem solve itself?
Frank Zappa

[Andy Leszczynski]

Sybren Stuvel wrote:

Andy Leszczynski enlightened us with:

should not it be:

2 def save(self, pagename, data, submit, new):
3 hub.begin()
4 if new == True:
5 page = Page(pagename=pagename, data=data)
6 else:
7 page = Page.byPagename(pagename)
8 page.data = data

instead of:

4 if new == "True":

No it should not. The values passed to the function are the strings
passed by the GET request, hence all strings. There are methods of
dealing with this - read the rest of the documentation.

Sybren

So how does it correspond to other piece of the code:

2 def notfound(self, pagename):
3 return dict(pagename=pagename, data="", new=True)

new is a boolean here?

A.

[Erik Max Francis]

Andy Leszczynski wrote:

So how does it correspond to other piece of the code:

2 def notfound(self, pagename):
3 return dict(pagename=pagename, data="", new=True)

new is a boolean here?

It comes through as a CGI query.

--
Erik Max Francis && ma*@alcyone.com && http://www.alcyone.com/max/
San Jose, CA, USA && 37 20 N 121 53 W && AIM erikmaxfrancis
Men and women, women and men. It will never work.
-- Erica Jong