Bug 537889 - "Link Note" must be available only on diagram or on container (not on list container)
Summary: "Link Note" must be available only on diagram or on container (not on list co...
Status: CLOSED FIXED
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 5.1.2   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: 6.1.0   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on: 535651
Blocks:
  Show dependency tree
 
Reported: 2018-08-13 04:38 EDT by Laurent Redor CLA
Modified: 2018-10-19 12:48 EDT (History)
4 users (show)

See Also:


Attachments
Use case to reproduce (3.82 KB, application/x-zip-compressed)
2018-10-05 06:03 EDT, Julien Dupont CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Laurent Redor CLA 2018-08-13 04:38:19 EDT
+++ This bug was initially created as a clone of Bug #535651 +++

Currently, when the tool "Link Note" is activated, if the mouse is over a Node or an Edge, there is a "forbidden" cursor (see forbiddenCursorOnNode.png). But the user can still click on the node. The representation selection dialog is opened but the OK button has no effect.
The click must have no effect or be really forbidden. For a simple Note, it has no effect.
Comment 1 Eclipse Genie CLA 2018-08-13 05:52:24 EDT
New Gerrit change created: https://git.eclipse.org/r/127336
Comment 3 Laurent Redor CLA 2018-09-25 06:32:17 EDT
Steps to validate:
* Open a diagram containing nodes, containers and edges.
* Activate the tool "Representation Link"
* Ensure that it is allowed only on diagram (forbidden on containers, nodes and edges)
Comment 4 Julien Dupont CLA 2018-10-05 06:02:38 EDT
It's possible to create a Representation Link on a simple container.
To reproduce:
- Import the use case joined
- Try to use the "Representation Link" tool on the EClass, the edge or on the attribute. It's impossible. Ok
- Try to use the "Representation Link" tool on the EPackage. It works and it should not! Ko
Comment 5 Julien Dupont CLA 2018-10-05 06:03:30 EDT
Created attachment 276130 [details]
Use case to reproduce
Comment 6 Felix Dorner CLA 2018-10-05 06:20:36 EDT
(In reply to Julien Dupont from comment #4)
> It's possible to create a Representation Link on a simple container.
Sorry, what is a 'simple' container? Are there also 'complex' containers? 

I am not sure if the "Steps To Validate" in Comment 3 are correct. Compare this with the bug title, which sais "... on diagram or on container". Which is right now? I think Diagram + Containers is correct, because that is where 'normal' notes work can be added.
Comment 7 Maxime Porhel CLA 2018-10-08 06:53:01 EDT
Title should be updated from
> "Link Note" must be available only on diagram or on container  
to
> "Link Note" must be available only on diagram
Comment 8 Felix Dorner CLA 2018-10-08 07:01:48 EDT
(In reply to Maxime Porhel from comment #7)
> Title should be updated from
> > "Link Note" must be available only on diagram or on container  
> to
> > "Link Note" must be available only on diagram

You're sure? Normal notes can be in a container. Why not link notes?
Comment 9 Laurent Redor CLA 2018-10-08 09:58:56 EDT
(In reply to Felix Dorner from comment #8)
> (In reply to Maxime Porhel from comment #7)
> > Title should be updated from
> > > "Link Note" must be available only on diagram or on container  
> > to
> > > "Link Note" must be available only on diagram
> 
> You're sure? Normal notes can be in a container. Why not link notes?

OK I think there is a confusion between "Container" and "Container List". Felix, in your comment bug 535651 comment c3 you said the inverse than just above.

So in conclusion "Representation Link" must have the same behavior than "Note". I think that the behavior is OK but the steps to validate are wrong.
Comment 10 Laurent Redor CLA 2018-10-08 11:24:16 EDT
Steps to validate:
* Open a diagram containing nodes, containers and edges.
* Activate the tool "Representation Link"
* Ensure that it is allowed only on diagram and on container (forbidden on list containers, nodes and edges)
Comment 11 Felix Dorner CLA 2018-10-08 11:35:28 EDT
(In reply to Laurent Redor from comment #9)
> Felix, in your comment bug 535651 comment c3 you said the inverse than just
> above.
That was about 'classes' which are list containers, here it's 'packages' which are normal containers. So yes, there was really some confusion about these two.
Comment 12 Pierre-Charles David CLA 2018-10-19 12:48:46 EDT
Available in Sirius 6.1.0, see https://wiki.eclipse.org/Sirius/6.1.0