Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 365999

Summary:

Initial participant set on a ChoreographyTask should be the initiating participant

Product:

[SOA] BPMN2Modeler

Reporter:

Gary Brown <gary>

Component:

Assignee:

Robert Brodt <bbrodt>

Status:

RESOLVED FIXED

QA Contact:

Severity:

enhancement

Priority:

Flags:

bbrodt: iplog+

Version:

unspecified

Target Milestone:

future

Hardware:

All

OS:

Linux

Whiteboard:

Attachments:

Description	Flags
Patch to provide a solution	none

Description Gary Brown

2011-12-08 05:06:20 EST

Build Identifier: 

Like the new 'add participant' mechanism for a ChoreographyTask - makes it a lot easier to setup the participants than going through the advanced properties.

One slight improvement would be that when adding the first participant to the choreography task, if that participant could also be set as the 'initiating participant', then it would be even better. Users could still change which participant is the initiator, but using the properties, but by default assuming the first participant is the initiator is a reasonable approach.


Reproducible: Always

Comment 1 Gary Brown

2011-12-08 05:08:17 EST

One other point - the initiating participant ref property uses the 'toString()' value of the Participant, rather than the name. It would be cleaner if this field either used the name, or some composite string made up of the name and id (for cases where possibly the name has not been defined)?

Comment 2 Gary Brown

2012-01-10 12:22:03 EST

Created attachment 209274 [details]
Patch to provide a solution

Provides the initial participant based on the first participant set. This does not mean the user cannot change the initiating participant to another participant, however in most cases the user would set the initiator first, and therefore this change reduces the work by the user in most situations.

Comment 3 Robert Brodt

2012-01-26 08:50:14 EST

Thanks Gary, I've applied the patch.