[Zope3-checkins] CVS: Zope3/src/zope/app/workflow/stateful - definition.py:1.9 instance.py:1.15

Stephan Richter srichter at cosmos.phy.tufts.edu
Mon Dec 15 08:33:48 EST 2003


Update of /cvs-repository/Zope3/src/zope/app/workflow/stateful
In directory cvs.zope.org:/tmp/cvs-serv8615/src/zope/app/workflow/stateful

Modified Files:
	definition.py instance.py 
Log Message:
My apologies for breaking the tests. I checked in late at night, that says 
all. I should know better.

But this surfaced a real problem with the workflow API that I need to 
address later; see the XXX comments in the diff.


=== Zope3/src/zope/app/workflow/stateful/definition.py 1.8 => 1.9 ===
--- Zope3/src/zope/app/workflow/stateful/definition.py:1.8	Mon Nov  3 23:04:24 2003
+++ Zope3/src/zope/app/workflow/stateful/definition.py	Mon Dec 15 08:33:17 2003
@@ -205,6 +205,11 @@
         # XXX
         # Process instances need to have a place, so they can look things
         # up.  It's not clear to me (Jim) what place they should have.
+
+        # XXX: The parent of the process instance should be the object it is
+        # created for!!! This will cause all sorts of head-aches, but at this
+        # stage we do not have the object around; it would need some API
+        # changes to do that for which I do not have time right now. (SR)
         pi_obj.__parent__ = self
 
 


=== Zope3/src/zope/app/workflow/stateful/instance.py 1.14 => 1.15 ===
--- Zope3/src/zope/app/workflow/stateful/instance.py:1.14	Sun Dec 14 23:31:36 2003
+++ Zope3/src/zope/app/workflow/stateful/instance.py	Mon Dec 15 08:33:17 2003
@@ -216,7 +216,7 @@
 
     def getProcessDefinition(self):
         """Get the ProcessDefinition object from WorkflowService."""
-        svc =  getService(zapi.getParent(self).context, "Workflows")
+        svc =  getService(self, "Workflows")
         return svc.getProcessDefinition(self.processDefinitionName)
 
     # XXX this is not entirely tested




More information about the Zope3-Checkins mailing list