Login | Register For Free | Help
Search 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
 
 


Interested in having your list archived? Contact lists@gossamer-threads.com
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.