Tres Seaver wrote:
My first patch (btw, anyone up for merging it?) was to DCWorkflow.
I think I'll be able to review and merge it this week.
Thanks Tres!
I could make another one to WorkflowTool, I guess, if you can suggest
where and what payload it needs (it's hardly difficult to fire an
event). We could fire from _invokeWithNotification() at just before (or
after) notifyBefore(), notifyException() and notifySuccess(), passing
the same parameters + 'w' (the workflow definition) as the first parameter.
I'd still keep them as separate patches that could be reviewed/merged
separately.
That's fine, although they might end up getting merged at the "same
time" if you get the second patch in before I get to reviewing the first.
All right, I'll see what I can do. I have about five things going on at
the same time at the moment, so I can't make strong promises, but if I
find an hour or two it should be fine.
Martin
_______________________________________________
Zope-CMF maillist - Zope-CMF@lists.zope.org
http://mail.zope.org/mailman/listinfo/zope-cmf
See http://collector.zope.org/CMF for bug reports and feature requests