[WICKET-4592] WicketAjaxDebug.logError used in Wicket.Head.addJavascript - JS error when wicket-ajax-debug.js not loaded

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.5.7
• Fix Version/s: 1.5.8, 6.0.0-beta3
• Assignee: Martin Tzvetanov Grigorov
• Reporter: Tom Kmec
• Created: 06/Jun/12 11:08
• Updated: 12/Feb/14 11:18
• Resolved: 15/Jun/12 08:22

Description

Wicket.Head.addJavascript (defined in wicket-ajax.js) encapsulates javascript in this code:

content = 'try

{'+content+'}

catch(e)

{WicketAjaxDebug.logError(e);}

';

However, WicketAjaxDebug can be undefined as it's declared in wicket-ajax-debug.js which might not be included. As a consequence, catch block will fail and obfuscate any error catched.

I believe this catch block should call Wicket.Log.error(e) instead.