[jboss-user] [jBPM Development] - JBPM-2820 and JBPM-2702

[Maciej Swiderski]

Maciej Swiderski [http://community.jboss.org/people/swiderski.maciej] replied to the discussion

"JBPM-2820 and JBPM-2702"

To view the discussion, visit: http://community.jboss.org/message/546001#546001

--------------------------------------------------------------
Hi HuiSheng,

please take a look at createExecution method in ExecutionImpl to verify if your fix is appropriate:

    childExecution.save();
    // make sure that child execution are saved before added to a persistent collection
    // cause of the 'assigned' id strategy, adding the childExecution to the persistent collection 
    // before the dbid is assigned will result in identifier of an instance of ExecutionImpl altered from 0 to x
 
    addExecution(childExecution);
    // composeIds uses the parent so the childExecution has to be added before the ids are composed
    childExecution.composeIds();

After your fix composeIds will be called twice, my concern is if that will not cause us any troubles. It is better to double check it.

Cheers
Maciej

--------------------------------------------------------------

Reply to this message by going to Community
[http://community.jboss.org/message/546001#546001]

Start a new discussion in jBPM Development at Community
[http://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2035]

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jboss-user/attachments/20100603/92e2c621/attachment-0001.html