Sidnei da Silva wrote:
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,)

D'oh. It's in IPublicationRequest. Never mind then :).

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().

Yep.

+1 to request.hold().


--
http://worldcookery.com -- Professional Zope documentation and training


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

Reply via email to