On 11/15/06, Philipp von Weitershausen <[EMAIL PROTECTED]> wrote:
Cool. I have two problems with this, though:

* the underscore clearly indicates the "privateness" of the _hold
method. In fact, _hold isn't part of the official Request API, I guess
(if there even is such a thing...)

* this doesn't seem to exist in Zope 3 (at least not in IRequest ;)).

BEEEEP! Wrong answer! This is turned out to be public API but wasn't
officialy named so for backward compatibility I guess. Someone should
really have deprecated that in Zope 2 by adding an alias.

In my local checkout of Zope 2.9, in lib/python/zope/publisher/base.py:

   def hold(self, object):
       'See IPublicationRequest'
       self._held = self._held + (object,)


Essentially, _hold() just adds properties to the request object. Why not
use attribute annotations? I think that makes for a bit cleaner approach
while still being flexible enough to make it work in both Zope 2 and 3...

I don't see why since Zope 3 seems to have inherited hold().

--
Sidnei da Silva
Enfold Systems                http://enfoldsystems.com
Fax +1 832 201 8856     Office +1 713 942 2377 Ext 214

_______________________________________________
Framework-Team mailing list
Framework-Team@lists.plone.org
http://lists.plone.org/mailman/listinfo/framework-team

Reply via email to