[MGNLUI-1532] Logging out results in NPE Created: 04/Jun/13  Updated: 14/Jun/13  Resolved: 07/Jun/13

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 5.0
Fix Version/s: 5.0

Type: Bug Priority: Major
Reporter: Daniel Lipp Assignee: Aleksandr Pchelintcev
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
supersession
is superseded by MAGNOLIA-5071 Fix exceptions and warnings shown on ... Closed
is superseded by MGNLUI-1526 Forcing application restart throws ex... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Sprint: Beta 3, RC 2

 Description   

Logging out using the ?mgnlLogout param results in a NPE. As long as it's not fixed we have to log that exception as warning only - else we'd spoil the logs and the integration (ui) tests would fail because when crawling the logs.

[INFO] [talledLocalContainer] 2013-06-04 00:24:36,299 ERROR .magnolia.ui.admincentral.AdmincentralErrorHandler: AdmincentralUI has encountered an unhandled exception.
[INFO] [talledLocalContainer] java.lang.NullPointerException
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:624)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractField.detach(AbstractField.java:1363)
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:619)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:619)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:619)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:619)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:619)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:619)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:619)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:619)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.detach(AbstractClientConnector.java:619)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.AbstractComponent.detach(AbstractComponent.java:567)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.UI.detach(UI.java:1024)
[INFO] [talledLocalContainer] 	at info.magnolia.ui.admincentral.AdmincentralUI.detach(AdmincentralUI.java:117)
[INFO] [talledLocalContainer] 	at com.vaadin.ui.UI.setSession(UI.java:344)
[INFO] [talledLocalContainer] 	at com.vaadin.server.VaadinSession.removeUI(VaadinSession.java:626)


 Comments   
Comment by Samuli Penttilä [ 04/Jun/13 ]

This particular stack trace reveals that this has something to do with Form. While debugging it is the first TextField in a Dialog containing a Form that throws error while detaching.

Comment by Daniel Lipp [ 04/Jun/13 ]

Make sure to revert how AdmincentralUI treats exceptions once the issue is fixed - see todo in the source code.

Generated at Mon Feb 12 08:47:32 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.