[Zope-Checkins] CVS: Zope3/lib/python/Zope/App/ZMI/Views/Browser - GenericCreatorView.py:1.1.4.2

[Gary Poster]

Update of /cvs-repository/Zope3/lib/python/Zope/App/ZMI/Views/Browser
In directory cvs.zope.org:/tmp/cvs-serv31509/App/ZMI/Views/Browser

Modified Files:
      Tag: Zope-3x-branch
	GenericCreatorView.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/ZMI/Views/Browser/GenericCreatorView.py 1.1.4.1 => 1.1.4.2 ===
     # action method
     def action(self, name, REQUEST=None):
-        "Create an item of the class identified by the Addable (held in _context) within the container that is the parent of the Addable"
+        """Create an item of the class identified by the Addable (held
+        in _context) within the container that is the parent of the
+        Addable"""
         addable=self.getContext()
         container=getcontext(addable)
         
         if name in container.objectIds():
             raise DuplicateIDError, "ID '%s' already in use." % name
 
-        container.setObject(name, createObject(container, addable.id()))
+        container.setObject(name, createObject(container, addable.id))
 
         if REQUEST is not None:
             # for unit tests
             REQUEST.getResponse().redirect(REQUEST.URL[-3])
 
-        return self.confirmed( type_name=addable.id(), id=name )
+        return self.confirmed( type_name=addable.id, id=name )
 
     confirmed = PageTemplateFile('add_confirmed.pt')