Login | Register For Free | Help	Search this list this category for: (Advanced)

Mailing List Archive: Zope: CMF [dev] .zexp imports and notifyWorkflowCreated   Index | Next | Previous | View Flat

y.2009 at wcm-solutions Feb 18, 2009, 4:04 AM Views: 1465 Permalink [dev] .zexp imports and notifyWorkflowCreated Hi! Moving the notifyWorkflowCreated call from _finishConstruction to the IObjectAddedEvent subscriber changed the behavior of .zexp imports: The workflow state is now always reset to the initial state. AFAICT that's no useful behavior for imports. This is caused by the fact that the notifyCreated method of WorkflowTool always resets the workflow states. Is that a feature or a bug of notifyCreated? Is anybody using notifyCreated for resetting workflow states? Where is the best place to fix this? Should IObjectAddedEvent be triggered on imports? Should the subscriber call notifyWorkflowCreated if the object already has a workflow history? Should notifyWorkflowCreated call WorkflowTool.notifyCreated if the object already has a workflow history? Should WorkflowTool.notifyCreated call notifyCreated of workflows that already have a state? Should notifyCreated of workflows keep existing states untouched? AFAICS the easiest way to fix this is changing WorkflowTool.notifyCreated, making sure it only calls notifyCreated for workflows without a state in the workflow history. Any objections or better ideas? Cheers, Yuppie _______________________________________________ Zope-CMF maillist - Zope-CMF[at]lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests

Subject User Time [dev] .zexp imports and notifyWorkflowCreated y.2009 at wcm-solutions Feb 18, 2009, 4:04 AM     Re: [dev] .zexp imports and notifyWorkflowCreated tseaver at palladion Feb 18, 2009, 9:37 AM         Re: [dev] .zexp imports and notifyWorkflowCreated y.2009 at wcm-solutions Feb 18, 2009, 12:35 PM             Re: [dev] .zexp imports and notifyWorkflowCreated wichert at wiggy Feb 18, 2009, 12:51 PM                 Re: [dev] .zexp imports and notifyWorkflowCreated y.2009 at wcm-solutions Feb 18, 2009, 1:14 PM                 Re: [dev] .zexp imports and notifyWorkflowCreated dieter at handshake Feb 21, 2009, 4:24 AM                 Re: [dev] .zexp imports and notifyWorkflowCreated charlie at begeistert Feb 23, 2009, 11:25 AM             Re: [dev] .zexp imports and notifyWorkflowCreated andrew at sawdog Feb 18, 2009, 12:56 PM     Re: [dev] .zexp imports and notifyWorkflowCreated tseaver at palladion Feb 18, 2009, 9:37 AM     Re: [dev] .zexp imports and notifyWorkflowCreated r.ritz at biologie Feb 25, 2009, 12:59 AM         Re: [dev] .zexp imports and notifyWorkflowCreated wichert at wiggy Feb 25, 2009, 5:18 AM             Re: [dev] .zexp imports and notifyWorkflowCreated charlie at begeistert Feb 25, 2009, 5:43 AM             Re: [dev] .zexp imports and notifyWorkflowCreated r.ritz at biologie Feb 25, 2009, 6:52 AM     Re: [dev] .zexp imports and notifyWorkflowCreated wichert at wiggy Feb 25, 2009, 7:10 AM         Re: [dev] .zexp imports and notifyWorkflowCreated charlie at begeistert Feb 25, 2009, 7:26 AM     Re: [dev] .zexp imports and notifyWorkflowCreated wichert at wiggy Feb 25, 2009, 8:48 AM         Re: [dev] .zexp imports and notifyWorkflowCreated tseaver at palladion Feb 25, 2009, 9:15 AM             Re: [dev] .zexp imports and notifyWorkflowCreated wichert at wiggy Feb 25, 2009, 11:04 AM             Re: [dev] .zexp imports and notifyWorkflowCreated wichert at wiggy Feb 25, 2009, 11:04 AM         Re: [dev] .zexp imports and notifyWorkflowCreated tseaver at palladion Feb 25, 2009, 9:15 AM

Index | Next | Previous | View Flat   Zope     Announce     Users     Dev     Coders     CMF     DB     Web     XML     CVS

Interested in having your list archived? Contact lists@gossamer-threads.com
 

Web Applications & Managed Hosting Powered by Gossamer Threads Inc.