yuppie wrote at 2006-3-15 11:23 +0100:
> ...
>Zope 2's checkValidId makes sure this doesn't happen with Zope 2 folder
>methods, Zope 3's NameChooser makes sure this doesn't happen with Zope 3
>folder views. Even the bad_id-patch described above doesn't allow to
>override folder methods.
Maybe,
Dieter Maurer wrote:
> Chris McDonough wrote at 2006-3-13 10:21 -0500:
> > ... silly id restrictions ...
> >Here's my current monkeypatch to Zope to unrestrict a good number of
> >characters:
> >
> >def patch_objectmanager_badid():
> > """ Causes Zope to be less restrictive in the set of charac
On Mon, Mar 13, 2006 at 07:06:28PM +0100, yuppie wrote:
> I'm concerned about the people we encourage to use Five technology.
> Views are a major feature of Five. Should we warn people not to use
> views? Or instruct them how to patch Zope 2 to protect views against
> being masked by content IDs
Chris McDonough wrote at 2006-3-13 10:21 -0500:
> ... silly id restrictions ...
>Here's my current monkeypatch to Zope to unrestrict a good number of
>characters:
>
>def patch_objectmanager_badid():
> """ Causes Zope to be less restrictive in the set of characters it
> accepts as valid wi
yuppie wrote at 2006-3-13 10:09 +0100:
> ...
>I don't understand that differentiation. Five is part of Zope 2, so each
>Five issue is also a Zope 2 issue.
But you can fix this issue either in "Five" or in "ObjectManager".
You propose a change in "ObjectManager" which affects not Five
use of Zope
On Mar 13, 2006, at 1:22 PM, Chris McDonough wrote:
On Mar 13, 2006, at 1:06 PM, yuppie wrote:
I'm not concerned about my own app code. I know the problem and
how to fix it.
And I'm not concerned about people like you who monkeypatch that
code. You know that monkeypatching is always on yo
I doubt this will break a significant amount of code. The
restriction was removed 5 months ago and AFAICS it was removed to
allow email addresses as IDs. That use case will not be broken if
we disallow again IDs starting with '@'.
It seems that you can reasonably easily apply the "@" rest
On Mon, 2006-03-13 at 02:46 +0100, Philipp von Weitershausen wrote:
> Here's what we could do: We factor the name validation part in ObjectManager
> (which is _checkId) out to a namechooser adapter. Five already has one in
> Five.browser.adding. Then, we can also provide an optional namechooser ada
Chris McDonough wrote:
> >> and should be fixed in Zope 2.8, 2.9 and trunk.
> >
> > We'd be changing Zope 2's behaviour (not fixing a Five bug), I would
> > therefore vote for making this change on the Zope 2 trunk only.
>
> I'd be apt to not further restrict the set of identifiers an OFS content
>
> yuppie wrote:
>> Zope 2 doesn't allow '+' in content IDs (actually the error message says
>> the ID contains characters illegal in URLs), but you can use content IDs
>> like '@@edit.html'. If the lookup order is changed as proposed
>> (http://codespeak.net/pipermail/z3-five/2006q1/001186.html) th
10 matches
Mail list logo