[Zope-CMF] Event Notification

seb bacon seb@jamkit.com
Tue, 10 Jul 2001 17:57:08 +0100


* Chris Withers <chrisw@nipltd.com> [010710 17:06]:
> Idea off top of head:
> 
> How about, in _createDiscussionForContent doing something (evil?) like:
> 
> content._normal_edit = content.edit
> content.edit = discussion_edit
> 
> where discussion_edit is defined as:
> 
> def discussion_edit(content,*args,**kw):
>    result = apply(content._normal_edit,args,kw)
>    # do metadata stuff here
>    return result

Sounds OK to me.  If you're doing that, though, you might as well use
the notify* workflow methods and wrap every method you think is
relevant as a WorkflowMethod.  Apart from...there's a bug in the
DefaultWorkflow which needs fixing, so perhaps not the best idea in
Mad Panic situations.  Shane, do you have any idea when you folks are
going to have time to look at that Workflow bug?
 
> Would that work?

Don't see why not

> Is it wise?

Probably not but who cares ;-)

seb