[Zope3-checkins] CVS: Zope3/lib/python/Zope/App/OFS/Services/ServiceManager - IConfigurationManager.py:1.3 configure.zcml:1.8

[Jim Fulton]

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

Modified Files:
	IConfigurationManager.py configure.zcml 
Log Message:
Fixed bug in configuration management view. The view was using
obsolete methods.  Added security assertions for the methods for
moving configurations around.


=== Zope3/lib/python/Zope/App/OFS/Services/ServiceManager/IConfigurationManager.py 1.2 => 1.3 ===
--- Zope3/lib/python/Zope/App/OFS/Services/ServiceManager/IConfigurationManager.py:1.2	Sat Nov 30 13:39:16 2002
+++ Zope3/lib/python/Zope/App/OFS/Services/ServiceManager/IConfigurationManager.py	Sun Dec  1 05:28:22 2002
@@ -16,10 +16,15 @@
 """
 
 from Zope.App.OFS.Container.IContainer  import IContainerNamesContainer
+from Interface import Interface
 
-class IConfigurationManager(IContainerNamesContainer):
-    """Manage Configuration Directives
+class IOrderedContainer(Interface):
+    """Container with items that can be rearranged.
     """
+    
+    # Yes, maybe this should be in the container package, but, we are
+    # likely to come up with a better general ordering interface, so
+    # we'll leave this one here for now.
 
     def moveTop(names):
         """Move the objects corresponding to the given names to the top
@@ -36,6 +41,10 @@
     def moveDown(names):
         """Move the objects corresponding to the given names down
         """
+
+class IConfigurationManager(IContainerNamesContainer, IOrderedContainer):
+    """Manage Configurations
+    """
 
 __doc__ = IConfigurationManager.__doc__ + __doc__
 


=== Zope3/lib/python/Zope/App/OFS/Services/ServiceManager/configure.zcml 1.7 => 1.8 ===
--- Zope3/lib/python/Zope/App/OFS/Services/ServiceManager/configure.zcml:1.7	Sat Nov 30 13:39:17 2002
+++ Zope3/lib/python/Zope/App/OFS/Services/ServiceManager/configure.zcml	Sun Dec  1 05:28:22 2002
@@ -45,7 +45,7 @@
         interface="Zope.App.OFS.Container.IContainer.IWriteContainer" />
     <require
         permission="Zope.ManageServices"
-    	attributes="arrange_object remove_objects" />
+    	interface=".IConfigurationManager.IOrderedContainer" />
     <factory 
         id = "Zope.App.OFS.Services.ServiceManager.ConfigurationManager" 
 	permission = "Zope.ManageServices"