[Zope3-checkins] CVS: Zope3/lib/python/Zope/App/OFS/Services - Configuration.py:1.7

[Steve Alexander]

Update of /cvs-repository/Zope3/lib/python/Zope/App/OFS/Services
In directory cvs.zope.org:/tmp/cvs-serv8602/lib/python/Zope/App/OFS/Services

Modified Files:
	Configuration.py 
Log Message:
corrected docs typo, answered question posed by someone in an XXX comment
and added explanatory comment about an implied interface import.


=== Zope3/lib/python/Zope/App/OFS/Services/Configuration.py 1.6 => 1.7 ===
--- Zope3/lib/python/Zope/App/OFS/Services/Configuration.py:1.6	Wed Dec 18 15:23:02 2002
+++ Zope3/lib/python/Zope/App/OFS/Services/Configuration.py	Sat Dec 21 14:52:09 2002
@@ -290,6 +290,7 @@
     of the component.
     """
 
+    # NamedConfiguration.__implements__ includes IDeleteNotifiable
     __implements__ = (INamedComponentConfiguration,
                       NamedConfiguration.__implements__, IAddNotifiable)
 
@@ -308,7 +309,7 @@
         # We have to be clever here. We need to do an honest to
         # god unrestricted traveral, which means we have to
         # traverse from an unproxied object. But, it's not enough
-        # for the service manager to be unproxies, because the
+        # for the service manager to be unproxied, because the
         # path is an absolute path. When absolute paths are
         # traversed, the traverser finds the physical root and
         # traverses from there, so we need to make sure the
@@ -321,6 +322,9 @@
         if wrapped_self.permission:
             if type(component) is Proxy:
                 # XXX what is this?
+                # Answer: There should be at most one security Proxy around
+                # an object. So, if we're going to add a new security proxy,
+                # we need to remove any existing one.
                 component = removeSecurityProxy(component)
 
             interface = wrapped_self.getInterface()