Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 365999 - Initial participant set on a ChoreographyTask should be the initiating participant
Summary: Initial participant set on a ChoreographyTask should be the initiating partic...
Status: RESOLVED FIXED
Alias: None
Product: BPMN2Modeler
Classification: SOA
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: All Linux
: P3 enhancement (vote)
Target Milestone: future   Edit
Assignee: Robert Brodt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-08 05:06 EST by Gary Brown CLA
Modified: 2012-10-09 11:51 EDT (History)
0 users

See Also:
bbrodt: iplog+

Attachments
Patch to provide a solution (1.33 KB, patch)
2012-01-10 12:22 EST, Gary Brown CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gary Brown CLA 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 CLA 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 CLA 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 CLA 2012-01-26 08:50:14 EST 
Thanks Gary, I've applied the patch.