Incident #1511
closed
Report Manager - deactivating window.alert deactivates commit facility - Need for internal dialog
100%
Description
Report Manager is using a window alert to ask for commit confirmation. Deactivating alerts (which is suggested by browsers with window.alert) prevents from commiting at all while it should triggers directly the commit confirmation. An internal dialog (as used for other UI components) might be used instead (not based on window.alert) to prevent this unexpected behavior.
Updated by Massimiliano Assante almost 10 years ago
Report Manager is using a standard Javascript Window.confirm, it is weird you get such a warning from the browser, what O.S. and Browser are you using, I'll try to reproduce
Updated by Massimiliano Assante almost 10 years ago
- Status changed from New to In Progress
Updated by Massimiliano Assante almost 10 years ago
- Status changed from In Progress to Feedback
Updated by Emmanuel Blondel almost 10 years ago
I use Windows 7 - Google Chrome.
First time the window appears, Chrome doesn't ask you if you want to disable window alert windows. it does it at second commit.
Updated by Massimiliano Assante over 9 years ago
- Status changed from Feedback to In Progress
Updated by Massimiliano Assante over 9 years ago
- Status changed from In Progress to Feedback
- % Done changed from 0 to 100
- Estimated time set to 4.00 h
- Infrastructure deleted (
Pre-Production)
The Reports Manager instance of development available at https://dev.d4science.org/group/devvre/report-generation contains the fix requested
In this moment however it seems VMEs do not get loaded due to an exception
Caused by: java.lang.NullPointerException
at org.gcube.application.rsg.support.builder.impl.ReportManagerReportBuilder.getProperties(ReportManagerReportBuilder.java:639)
at org.gcube.application.rsg.support.builder.impl.ReportManagerReportBuilder.getProperties(ReportManagerReportBuilder.java:583)
at org.gcube.application.rsg.support.builder.impl.ReportManagerReportBuilder.addComponent(ReportManagerReportBuilder.java:293)
at org.gcube.application.rsg.support.builder.impl.ReportManagerReportBuilder.addComponent(ReportManagerReportBuilder.java:236)
... 28 more
Updated by Massimiliano Assante over 9 years ago
- Status changed from Feedback to Closed
org.gcube.portlets-user.report-generator-portlet.5-1-0 has been included in the next release (3.10)
