[Zope-Checkins] CVS: Zope3/lib/python/Zope/App/OFS/Container - IContainer.py:1.1.2.4 SampleContainer.py:1.1.2.5

[Gary Poster]

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

Modified Files:
      Tag: Zope-3x-branch
	IContainer.py SampleContainer.py 
Log Message:
1. As per ZopeTop discussion, moved Addable out of ZMI into the Services folder, and turned it into a true service (this involved a lot of small changes, particularly in the folders and a number of tests) and gave it some hooks
2. used SteveA's ContextWrapper.ContextMethod to make the ServiceManager and Services placefully look up to parent placeful equivalents
3. Made Event into a more standard service, and gave it some hooks
4. Built the beginning of a placeful EventService (will need tests, and views, and EventChannels, and more meat; just wanted to check this in for now)
5. made it so you can't add an item to a folder with a blank string id, and updated the folder interface
6. some typos fixed here and there
7. a few more tests here and there

I'm checking this in then checking it out again to check my work; also tagged previous version as gary-service_update.



=== Zope3/lib/python/Zope/App/OFS/Container/IContainer.py 1.1.2.3 => 1.1.2.4 ===
 
     def setObject(name, object):
-        """Add the given object to the container under the given name."""
+        """Add the given object to the container under the given name.
+        
+        Raises a ValueError if name is an empty string."""
 
     def delObject(name):
         """Delete the named object from the container. Raises a KeyError


=== Zope3/lib/python/Zope/App/OFS/Container/SampleContainer.py 1.1.2.4 => 1.1.2.5 ===
 
 from IContainer import IContainer
+from types import StringTypes
 
 _marker = object()
 
@@ -83,6 +84,8 @@
 
     def setObject(self, name, object):
         '''See interface IWriteContainer'''
+        if type(name) in StringTypes and len(name)==0:
+            raise ValueError
         self.__data[name] = object
 
     def delObject(self, name):