[Zope3-checkins] CVS: Zope3/src/zope/app/browser/services/pluggableauth - __init__.py:1.3 configure.zcml:1.5

Stephan Richter srichter@cosmos.phy.tufts.edu
Thu, 3 Jul 2003 17:45:59 -0400


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

Modified Files:
	__init__.py configure.zcml 
Log Message:
- Got rid of IWritePrincipalSource interface and IReadPrincipalSource to
  become IPrincipalSource (based on Jim's IRC request)

- Then I noticed that some browser directives assumed that IPrincipalSource
  objects are automatically are IContainers, which is of course not true.
  So I added a IContainerPrincipalSource marker interface for principal
  sources that are containers (this way we can reuse the container screens).


=== Zope3/src/zope/app/browser/services/pluggableauth/__init__.py 1.2 => 1.3 ===
--- Zope3/src/zope/app/browser/services/pluggableauth/__init__.py:1.2	Mon Jun 23 18:46:14 2003
+++ Zope3/src/zope/app/browser/services/pluggableauth/__init__.py	Thu Jul  3 17:45:24 2003
@@ -2,7 +2,7 @@
 
 from zope.app.browser.services.service import Adding
 from zope.context import ContextSuper
-from zope.app.interfaces.services.pluggableauth import IReadPrincipalSource
+from zope.app.interfaces.services.pluggableauth import IPrincipalSource
         
 class PrincipalSourceAdding(Adding):
     """Adding subclass used for principal sources."""
@@ -11,7 +11,7 @@
 
     def add(self, content):
 
-        if not IReadPrincipalSource.isImplementedBy(content):
+        if not IPrincipalSource.isImplementedBy(content):
             raise TypeError("%s is not a readable principal source" % content)
 
         return ContextSuper(PrincipalSourceAdding, self).add(content)


=== Zope3/src/zope/app/browser/services/pluggableauth/configure.zcml 1.4 => 1.5 ===
--- Zope3/src/zope/app/browser/services/pluggableauth/configure.zcml:1.4	Thu Jun 26 10:50:22 2003
+++ Zope3/src/zope/app/browser/services/pluggableauth/configure.zcml	Thu Jul  3 17:45:24 2003
@@ -49,7 +49,7 @@
 <page
      name="contents.html" 
      menu="zmi_views" title="Contents"
-     for="zope.app.services.pluggableauth.IPrincipalSource"
+     for="zope.app.services.pluggableauth.IContainerPrincipalSource"
      permission="zope.ManageServices" 
      class="zope.app.browser.container.contents.Contents"
      attribute="contents"
@@ -57,13 +57,13 @@
 
 <view
     name="+"  
-    for="zope.app.services.pluggableauth.IPrincipalSource"
+    for="zope.app.services.pluggableauth.IContainerPrincipalSource"
     permission="zope.ManageContent"
     class="zope.app.browser.container.adding.Adding"/>
 
 <menuItem
     menu="zmi_actions" title="Add Principal"
-    for="zope.app.services.pluggableauth.IPrincipalSource"
+    for="zope.app.services.pluggableauth.IContainerPrincipalSource"
     action="+/AddPrincipalForm"
     />