[Zope3-checkins] CVS:
Zope3/src/zope/app/interfaces/services/pluggableauth - __init__.py:1.3
Tres Seaver
tseaver@zope.com
03 Jul 2003 12:43:29 -0400
On Thu, 2003-07-03 at 11:43, Stephan Richter wrote:
> Update of /cvs-repository/Zope3/src/zope/app/interfaces/services/pluggableauth
> In directory cvs.zope.org:/tmp/cvs-serv4253/src/zope/app/interfaces/services/pluggableauth
>
> Modified Files:
> __init__.py
> Log Message:
> Fixed simple interface confusion. I was required to implement
> IPrincipalSource to have ILoginPasswordPrincipalSource, which was no good,
> since I was required to implement a write interface. However, I think this
> was simply a typo, since the implementation of the interfaces did not make this assumption.
Shouldn't you derive from IReadPrincipalSource?
> === Zope3/src/zope/app/interfaces/services/pluggableauth/__init__.py 1.2 => 1.3 ===
> --- Zope3/src/zope/app/interfaces/services/pluggableauth/__init__.py:1.2 Mon Jun 23 18:46:16 2003
> +++ Zope3/src/zope/app/interfaces/services/pluggableauth/__init__.py Thu Jul 3 11:42:45 2003
> @@ -68,7 +68,7 @@
> class IPrincipalSource(IReadPrincipalSource, IWritePrincipalSource):
> """A read-write principal source."""
>
> -class ILoginPasswordPrincipalSource(IPrincipalSource):
> +class ILoginPasswordPrincipalSource(Interface):
> """ A principal source which can authenticate a user given a
> login and a password """
Tres.
--
===============================================================
Tres Seaver tseaver@zope.com
Zope Corporation "Zope Dealers" http://www.zope.com