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

Bug 519322

Summary: 'if' variable invalid in the dynamic mapping for extension filtering expression
Product: [Modeling] Sirius Reporter: Melanie Bats <melanie.bats>
Component: PropertiesAssignee: Project Inbox <sirius.properties-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: julien.dupont, pierre-charles.david
Version: 5.0.0Keywords: triaged
Target Milestone: 5.1.0   
Hardware: All   
OS: All   
See Also: https://git.eclipse.org/r/100850
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=c25ad5c4af16642a2307194af312a035ab6932fa
Whiteboard: backport
Bug Depends on:    
Bug Blocks: 519324    
Attachments:
Description Flags
Sample project to reproduce the issue none

Description Melanie Bats CLA 2017-07-06 10:27:55 EDT
To reproduce, create a dynamic mapping for element which extends another dynamic mapping for. The ntry to enter an Ifs Filtering Expression. The name of the variable to filter the ifs is : 'if'. AQL does not like a variable named if as it is parsing it as a condition instead of a variable.
Comment 1 Eclipse Genie CLA 2017-07-06 10:32:54 EDT
New Gerrit change created: https://git.eclipse.org/r/100850
Comment 3 Pierre-Charles David CLA 2017-08-07 04:47:19 EDT
Fixed by c25ad5c4af16642a2307194af312a035ab6932fa: all the filtering expression variables were renamed to match the following pattern: featureName+'Description'.

This change will be backported on v5.0.2 via #519324.
Comment 4 Julien Dupont CLA 2017-09-20 05:57:43 EDT
Can not be verified: missing or incomplete reproduction information.
Comment 5 Pierre-Charles David CLA 2017-09-25 10:19:48 EDT
Created attachment 270341 [details]
Sample project to reproduce the issue

Steps to reproduce:
1. Import the attched project.
2. Open the "My.aird" session with the aird editor.
3. Open the "Properties" view.
4. In the editor's left-hand pane, select the single EClass in the semantic model.
5. The properties view, the "Page" tab should show duplicate widget for "Name", "Instance Class Name" and "Instance Type Name", but *NOT* for the "Abstract" and "Interface" checkboxes.
Comment 6 Pierre-Charles David CLA 2017-09-25 10:20:09 EDT
Verified on Sirius 5.1.0rc1.
Comment 7 Pierre-Charles David CLA 2017-11-08 03:36:45 EST
Available in Sirius 5.1.0, see https://wiki.eclipse.org/Sirius/5.1.0.