Re: [Zope-dev] Duplicate directive registration allowed

2007-11-12 Thread Malthe Borch
> I agree with your analysis. Could you file a bug report in launchpad?

Bug now filed: https://bugs.launchpad.net/zope3/+bug/162166.

\malthe
___
Zope-Dev maillist  -  Zope-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope-announce
 http://mail.zope.org/mailman/listinfo/zope )


Re: [Zope-dev] Duplicate directive registration allowed

2007-11-11 Thread Stephan Richter
On Wednesday 07 November 2007, Malthe Borch wrote:
> So for instance, if Five registers the  then there's
> nothing to prevent another package from including zope.viewlet's
> meta.zcml which will then override that directive, breaking viewlet
> support on Zope 2.10.

I agree with your analysis. Could you file a bug report in launchpad?

Thanks,
Stephan
-- 
Stephan Richter
CBU Physics & Chemistry (B.S.) / Tufts Physics (Ph.D. student)
Web2k - Web Software Design, Development and Training
___
Zope-Dev maillist  -  Zope-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope-announce
 http://mail.zope.org/mailman/listinfo/zope )


[Zope-dev] Duplicate directive registration allowed

2007-11-07 Thread Malthe Borch
In zope/configuration/config.py:register it seems that it isn't being
asserted that no previous registration of a directive with the same
name has happened.

This might be desirable (not sure why) and currently happens for these
directives:

('http://namespaces.zope.org/meta', u'directive')
('http://namespaces.zope.org/meta', u'groupingDirective')
('http://namespaces.zope.org/meta', u'complexDirective')
('http://namespaces.zope.org/zope', u'securityPolicy')

This is on Zope 2.10.

So for instance, if Five registers the  then there's
nothing to prevent another package from including zope.viewlet's
meta.zcml which will then override that directive, breaking viewlet
support on Zope 2.10.

\malthe
___
Zope-Dev maillist  -  Zope-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope-announce
 http://mail.zope.org/mailman/listinfo/zope )