[Zope3-checkins] SVN: Zope3/trunk/src/zope/app/publisher/browser/configure.zcml Allowed annotations on the BrowserRequest

[Gary Poster]

On Nov 3, 2005, at 10:44 AM, Stephan Richter wrote:
> Since "annotations" is already there in 3.1, it will be easier to  
> adjust the
> new code to this attribute instead of deprecating annotations and  
> introducing
> __annotations__, unless Dimitry is willing to do that. :-) I agree  
> with
> Dimitry that using __annotations__ makes more sense, since this is  
> what we
> use anywhere else.

The '__annotations__' name is an implementation detail of the  
attribute annotations package, has nothing to do with IAnnotatable or  
IAnnotations, and was designed for objects that do not have  
annotations as part of their core contract.  If something has  
annotations as part of their core contract, such as the request  
object, I see no reason to resort to "underwear".  Moreover, this  
part of the request contract already exists in 3.1, and we rely on  
it.  I'm -1 on converting to '__annotations__' for the request.

Gary