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

Bug 476365

Summary: StateMachine diagram : Adding transition edge graphically, displays error icon
Product: [Modeling] Papyrus-rt Reporter: Stephanie Chafe <schafe>
Component: toolAssignee: Remi Schnekenburger <rschnekenburger>
Status: CLOSED FIXED QA Contact: Peter Cigehn <peter.cigehn>
Severity: normal    
Priority: P3 CC: charles, peter.cigehn, rschnekenburger
Version: .7   
Target Milestone: 0.9.0   
Hardware: PC   
OS: Windows 7   
See Also: https://bugs.eclipse.org/bugs/show_bug.cgi?id=512809
Whiteboard:
Attachments:
Description Flags
screenshot none

Description Stephanie Chafe CLA 2015-09-01 12:04:14 EDT
Created attachment 256300 [details]
screenshot

Getting Started with Papyrus for RealTime tutorial

-graphically added a transition and attempted to name it on add
-an error icon is drawn on link, and the name is not saved

Section 7.11
3. Using the Palette's "Edges" drawer, draw a "Transition" from "Initial" to "Playing" named "initial", and another from "Playing" to itself (which we call a "self-transition") named "onPong".

Please see attachment for screenshot.
Comment 1 Peter Cigehn CLA 2015-12-10 08:05:02 EST
I guess that this is exactly the same issue as discussed on the Papyrus Forum. See the following thread for more information: https://www.eclipse.org/forums/index.php/t/1072448/

Unfortunately I cannot see if anyone else has written a Bugzilla on base Papyrus regarding this behavior.
Comment 2 Charles Rivet CLA 2016-08-29 13:54:13 EDT
Could no reproduce with Papyrus-RT 0.7.2.201608261452
Comment 3 Peter Cigehn CLA 2016-08-30 04:47:30 EDT
I can still reproduce this issue in the latest build of Papyrus-RT. The core issue in Papyrus is still there as indicated in Comment 1.

More detailed steps to reproduce:

1) Create a UML-RT model with the state machines profile applied
2) Create a Capsule1 in this model
3) Create a StateMachine in this Capsule1
4) Open the state machine diagram for this StateMachine
5) Create an additional State2 (apart from the already existing State1) using the State tool in the palette
6) Create a Transition from State1 to State2 using the Transition tool on the palette
7) Now a blue bordered box appears in which the user can start typing.
8) If the user assumes that this box that appears is used for giving the transition its name, then simply type a name, e.g. "t1".
9) Click somewhere outside the blue box to get it closed.
10) Now you get the error marker as shown in the attached screen shot.
11) In the Validation view you also have a problem stating "A transition with invalid text string found."

This is the same confusion that is discussed in the forum thread provided by the link in Comment 1.

I would suggest to reopen this one again (I do not have the access right to do so myself).
Comment 4 Charles Rivet CLA 2016-08-30 09:27:46 EDT
As requested
Comment 5 Charles Rivet CLA 2016-08-30 09:28:26 EDT
Assigned to 0.9/MVP2
Comment 6 Remi Schnekenburger CLA 2016-11-10 09:47:51 EST
(In reply to Charles Rivet from comment #5)
> Assigned to 0.9/MVP2

Charles, do you want to stay as assignee of this bug? Otherwise, I can probably handle it while working on state machines.
Comment 7 Charles Rivet CLA 2016-11-10 13:02:16 EST
(In reply to Remi Schnekenburger from comment #6)
> (In reply to Charles Rivet from comment #5)
> > Assigned to 0.9/MVP2
> 
> Charles, do you want to stay as assignee of this bug? Otherwise, I can
> probably handle it while working on state machines.

I have no idea why I am the assignee. Please feel free to take it.
Comment 8 Remi Schnekenburger CLA 2016-11-10 13:04:17 EST
I was suite suspicious also ;) assigning to myself.
Comment 9 Remi Schnekenburger CLA 2017-01-13 08:23:17 EST
(In reply to Remi Schnekenburger from comment #8)
> I was suite suspicious also ;) assigning to myself.

This one has been worked as a side effect on work for internal transitions (Bug 494287)
There is still the issue that the direct editor coming on transition tries to set a guard and effect rather than its name.
Comment 10 Remi Schnekenburger CLA 2017-01-30 06:24:46 EST
Direct editor now opens on the name label and edits the name of the transition.

closing bug.
Comment 11 Peter Cigehn CLA 2017-01-31 02:26:04 EST
Verified to be fixed in the latest Papyrus-RT build. The direct editor for a transition now (only) renames the transition (as expected). You no longer get a warning marker.

I leave it this bug in verified fixed state, and I leave it to Charles to decide if it can be closed.
Comment 12 Charles Rivet CLA 2017-01-31 11:59:50 EST
Closing