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

Bug 486631

Summary:

EvaluationExceptions are ignored with tree

Product:

[Modeling] Sirius

Reporter:

Esteban DUGUEPEROUX <esteban.dugueperoux>

Component:

Tree

Assignee:

Pierre-Charles David <pierre-charles.david>

Status:

CLOSED FIXED

QA Contact:

Julien Dupont <julien.dupont>

Severity:

normal

Priority:

CC:

julien.dupont, laurent.redor, pierre-charles.david

Version:

3.1.0

Keywords:

triaged

Target Milestone:

4.1.0

Hardware:

All

OS:

All

See Also:

https://git.eclipse.org/r/81702
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=d176dff1cb9ce084bcaef304928cd46c3163a0da
https://git.eclipse.org/r/83088
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=c34a7a47620a97d4a4e0ad00240943e2cfe200bd

Whiteboard:

needtest

Attachments:

Description	Flags
Sample project to reproduce	none

Description Esteban DUGUEPEROUX

2016-01-27 05:31:19 EST

When a EvaluationException is thrown during semantic candidates expression evaluation for a DTree, it is ignored because in MappingBasedPartition.evaluate() we call SpecifierMonitor.warning() which do nothing.
The feedback about EvaluationException should be coherent between all dialects and possible configurable.

Comment 1 Esteban DUGUEPEROUX

2016-01-27 05:36:44 EST

Created attachment 259408 [details]
Sample project to reproduce

Scenario to reproduce:
1. Import attached project
2. Open the diagram and refresh it => many EvaluationExceptions appears in error log
3. Open the tree and refresh it => no exceptions appears because ignored by SpecifierMonitor

Comment 2 Laurent Redor

2016-01-27 10:23:36 EST

Bug confirmed.

Comment 3 Eclipse Genie

2016-09-22 10:23:33 EDT

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

Comment 4 Eclipse Genie

2016-09-26 03:37:35 EDT

Gerrit change https://git.eclipse.org/r/81702 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=d176dff1cb9ce084bcaef304928cd46c3163a0da

Comment 5 Pierre-Charles David

2016-10-04 08:42:26 EDT

Fixed.

Comment 6 Eclipse Genie

2016-10-13 04:43:24 EDT

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

Comment 7 Pierre-Charles David

2016-10-18 11:07:16 EDT

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

Comment 8 Eclipse Genie

2016-10-19 11:49:30 EDT

Gerrit change https://git.eclipse.org/r/83088 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=c34a7a47620a97d4a4e0ad00240943e2cfe200bd