[Zope-dev] zope.app.authentication

[Fred Drake]

Sorry for bring an old thread back up, but it's relevant.

On Tue, Jan 26, 2010 at 4:06 AM, Souheil CHELFOUH <trollfot at gmail.com> wrote:
> The tests pass and nothing has been changed but the imports.

Actually, the base classes for SessionCredentialsPlugin changed, and
no longer include these:

    persistent.Persistent
    zope.app.container.contained.Contained

These are probably good to retain.

The later should probably be replaced with
zope.container.contained.Contained, but the value is the same.  Though
simple, dropping this can affect existing applications in obscure
ways, since adapter lookups are likely to produce different results.

Are there any objections to adding these base classes back in?


  -Fred

-- 
Fred L. Drake, Jr.    <fdrake at gmail.com>
"A storm broke loose in my mind."  --Albert Einstein