In article <3EB78D0E.3030903@web.de> you write:
<http://mail.zope.org/pipermail/zope-dev/2002-July/016913.html>
I did that on request of Florent and thought it would be a no-brainer. <http://mail.zope.org/pipermail/zope-checkins/2003-April/022598.html>
But apparently I did it wrong. Would it be ok to change it to:
def __init__(id=None): if id is not None: self.id = str(id)
Or should I revert that change completely?
Why not add the title as well? It seems common for constructors that accept an id parameter to also accept a title.
ZopePageTemplate and PythonScript constructors don't accept title. Florent just asked for id. But I can live with both solutions.
I think that having the constructor set id is the thing that minimises surprises. But that's just a small cleanup I requested, if it poses problem just forget about it. Florent -- Florent Guillaume, Nuxeo (Paris, France) +33 1 40 33 79 87 http://nuxeo.com mailto:fg@nuxeo.com