Max M wrote:
yuppie wrote:
As of Zope 2.8.2 (http://svn.zope.org/?rev=38738&view=rev) the '@' character is allowed in object IDs.


Zope 3 does not allow object IDs starting with '@' because those names are reserved for views.

Five makes Zope 3 style views available in Zope 2, so IDs starting with '@' should be disallowed in Zope 2 as well.

If there are no objections I'll fix that in Zope 2.8, 2.9 and trunk.

[...]

If you just want to prohibit "id.startswith('@')" it is fine by me.

I didn't propose to revert revision 38738, just want to fix it by adding

    if id[:1] == '@':
        raise BadRequest('The id "%s" is invalid because it begins '
                         'with "@".' % id)

So I count your reply as a +1.

Cheers, Yuppie

_______________________________________________
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 )

Reply via email to