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

Bug 509300

Summary:	The go to marker creates a new Sirius session
Product:	[Modeling] Sirius	Reporter:	Florian Barbin <florian.barbin>
Component:	Diagram	Assignee:	Florian Barbin <florian.barbin>
Status:	CLOSED WONTFIX	QA Contact:
Severity:	normal
Priority:	P3	CC:	pierre-charles.david
Version:	4.1.0	Keywords:	triaged
Target Milestone:	4.1.3
Hardware:	All
OS:	All
See Also:	https://git.eclipse.org/r/87258 https://git.eclipse.org/r/87991 https://git.eclipse.org/r/88178 https://git.eclipse.org/r/88177
Whiteboard:	needtest
Bug Depends on:
Bug Blocks:	509970, 534273

Description Florian Barbin

2016-12-15 11:31:45 EST

When performing a go to marker, the DDiagramEditorImpl.getCorrectInput translates the FIleEditorInput into SessionEditorInput. But the method org.eclipse.sirius.ui.business.api.session.SessionEditorInput.create(URI) creates a new Sirius Session from the session resource URI each time the go to marker is executed. Those sessions are never closed.

Comment 1 Eclipse Genie

2016-12-15 12:10:46 EST

New Gerrit change created: https://git.eclipse.org/r/87258

Comment 2 Eclipse Genie

2017-01-04 08:09:13 EST

New Gerrit change created: https://git.eclipse.org/r/87991

Comment 3 Florian Barbin

2017-01-05 05:09:15 EST

We need to provide additional tests for this use case.

Comment 4 Eclipse Genie

2017-01-06 10:12:32 EST

New Gerrit change created: https://git.eclipse.org/r/88178

Comment 5 Eclipse Genie

2017-01-06 10:12:33 EST

New Gerrit change created: https://git.eclipse.org/r/88177

Comment 6 Pierre-Charles David

2017-01-13 04:06:47 EST

This will only be fixed on master for Sirius 5.0 via https://bugs.eclipse.org/bugs/show_bug.cgi?id=509970.

Comment 7 Pierre-Charles David

2017-01-31 03:48:35 EST

Available in Sirius 4.1.3, see https://wiki.eclipse.org/Sirius/4.1.3 for version details.