[Zope3-checkins] CVS: Zope3/src/zope/app/browser/services - configure.zcml:1.6

R. David Murray bitz@bitdance.com
Thu, 26 Dec 2002 18:22:17 -0500


Update of /cvs-repository/Zope3/src/zope/app/browser/services
In directory cvs.zope.org:/tmp/cvs-serv21705/src/zope/app/browser/services

Modified Files:
	configure.zcml 
Log Message:
Improve the efficiency of the basic persistent auth source.  Instead of
a BTreeContainer, use two BTrees, one mapping from ids to Users,
and one from login to Users.  This eliminates the need to look at
every User when looking up the user at login time.  Also standarize
the implementation of the helper method getPrincipalByLogin to raise
an error (NotFoundError) when the login isn't found, and provide
the corresponding query method.  Fix the tests to check for NotFoundError
as per the IAuthenticationService spec for getPrincipal.  Finally,
eliminate the ILocalAuthenticationService interface, since it doesn't
seem to add anything we need.  XXX the IContainer implementation is
not fully unit tested, since the BaseTestIContainer tests need to be
refactored first to facilitate it.


=== Zope3/src/zope/app/browser/services/configure.zcml 1.5 => 1.6 ===
--- Zope3/src/zope/app/browser/services/configure.zcml:1.5	Thu Dec 26 17:11:53 2002
+++ Zope3/src/zope/app/browser/services/configure.zcml	Thu Dec 26 18:21:46 2002
@@ -149,7 +149,7 @@
 
   <browser:view 
      permission="zope.ManageServices" 
-     for="zope.app.services.auth.ILocalAuthenticationService"
+     for="zope.app.services.auth.IAuthenticationService"
      factory="zope.app.browser.container.contents.Contents">
 
      <browser:page name="index.html" attribute="contents" />
@@ -158,7 +158,7 @@
 
   <browser:view
       name="+"  
-      for="zope.app.services.auth.ILocalAuthenticationService"
+      for="zope.app.services.auth.IAuthenticationService"
       permission="zope.ManageContent"
       factory="zope.app.browser.services.auth.AddUser" >