Bug 485832 - Sirius may open an empty popup if a SecurityException is raised
Summary: Sirius may open an empty popup if a SecurityException is raised
Status: CLOSED FIXED
Alias: None
Product: Sirius
Classification: Modeling
Component: Core (show other bugs)
Version: 3.1.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.1.4   Edit
Assignee: Florian Barbin CLA
QA Contact:
URL:
Whiteboard: needtest
Keywords: triaged
Depends on:
Blocks: 485837
  Show dependency tree
 
Reported: 2016-01-14 05:32 EST by Florian Barbin CLA
Modified: 2016-03-29 10:18 EDT (History)
2 users (show)

See Also:

Attachments
Use Case (5.49 KB, application/zip)
2016-01-14 05:32 EST, Florian Barbin CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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