Re: zope.app.authentication

Fred Drake <[email protected]>
Newsgroups gmane.comp.web.zope.devel
Message-ID <[email protected]>
Sorry for bring an old thread back up, but it's relevant.

On Tue, Jan 26, 2010 at 4:06 AM, Souheil CHELFOUH <[email protected]> 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
_______________________________________________
Zope-Dev maillist  -  [email protected]
https://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 https://mail.zope.org/mailman/listinfo/zope-announce
 https://mail.zope.org/mailman/listinfo/zope )
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.