[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