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

Bug 485832

Summary: Sirius may open an empty popup if a SecurityException is raised
Product: [Modeling] Sirius Reporter: Florian Barbin <florian.barbin>
Component: CoreAssignee: Florian Barbin <florian.barbin>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: laurent.fasani, steve.monnier
Version: 3.1.0Keywords: triaged
Target Milestone: 3.1.4   
Hardware: All   
OS: All   
See Also: https://git.eclipse.org/r/64344
https://git.eclipse.org/r/65891
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=8a70741f3cc0fab1f2db400d3733646d5a94e477
https://git.eclipse.org/r/68195
https://git.eclipse.org/r/68196
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=c04358746b1788e124b4ca99b900b95581ba4ab3
https://git.eclipse.org/r/69425
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=b38268bcbf4c1ddb41a168255d7593f1c88455ca
Whiteboard: needtest
Bug Depends on:    
Bug Blocks: 485837    
Attachments:
Description Flags
Use Case none

Description Florian Barbin CLA 2016-01-14 05:32:22 EST
Created attachment 259183 [details]
Use Case

If a SecurityException is raised without message, org.eclipse.sirius.ui.business.internal.dialect.LogThroughActiveDialectEditorLogListener open an empty popup. We should not open this popup if the SecurityException doesn't contain any message.

To reproduce, import the attached plugin deploy it and open a Sirius Session => an empty popup appears.
Comment 1 Eclipse Genie CLA 2016-01-14 06:11:18 EST
New Gerrit change created: https://git.eclipse.org/r/64344
Comment 2 Eclipse Genie CLA 2016-02-04 09:38:28 EST
New Gerrit change created: https://git.eclipse.org/r/65891
Comment 4 Florian Barbin CLA 2016-02-09 11:02:51 EST
Fix merged on 3.1.x but the junit test is still in progress
Comment 5 Laurent Fasani CLA 2016-02-09 11:19:39 EST
Validated on sirius 3.1.4
Comment 6 Pierre-Charles David CLA 2016-03-07 10:52:45 EST
Available in Sirius 3.1.4.
Comment 7 Eclipse Genie CLA 2016-03-11 04:51:34 EST
New Gerrit change created: https://git.eclipse.org/r/68195
Comment 8 Eclipse Genie CLA 2016-03-11 04:54:47 EST
New Gerrit change created: https://git.eclipse.org/r/68196
Comment 10 Eclipse Genie CLA 2016-03-29 08:51:30 EDT
New Gerrit change created: https://git.eclipse.org/r/69425