[WICKET-3477] Exception when setting session default timezone with GatherExtendedBrowserInfo

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.5-RC1
• Fix Version/s: 1.4.17, 1.5-RC3
• Component/s: wicket
• Labels: nullpointerexception
• Assignee: Martin Tzvetanov Grigorov
• Reporter: Bertrand Guay-Paquet
• Created: 23/Feb/11 22:11
• Updated: 11/Apr/11 21:33
• Resolved: 27/Feb/11 09:58

Description

I want to set a default timezone for client sessions so I added the following line at the end of my CustomSession constructor:
getClientInfo().getProperties().setTimeZone(TimeZone.getTimeZone("GMT"));

Doing this coupled with getGatherExtendedBrowserInfo() == true however yields a null pointer exception.

Some analysis:
In WebSession#getClientInfo(), there is no clientInfo yet and getGatherExtendedBrowserInfo() is true so a redirect is attempted.
The line "IRequestHandler activeRequestHandler = requestCycle.getActiveRequestHandler();" produces activeRequestHandler == null.
Since there is no request handler, the following line "String url = requestCycle.urlFor(activeRequestHandler).toString();" gives a null pointer exception as urlFor returns null.

If getGatherExtendedBrowserInfo() == false, the default timezone set in the Session constructor works fine.