[WICKET-1520] JavaScriptStripper breaks prototype.js

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.3.2, 1.3.3
• Fix Version/s: 1.3.5
• Assignee: Matej Knopp
• Reporter: Juliano Viana
• Created: 10/Apr/08 18:16
• Updated: 18/Sep/08 22:57
• Resolved: 18/Sep/08 22:57

Description

JavaScriptStripper is used to strip whitespace and comments from JavaScript files if the resource settings for the application enable that behavior.

The problem is it breaks prototype.js version 1.6.0 (a heavily used JavaScript library, used by many components like wicketstuff-scriptaculous ).
It changes the library so that the line that reads:

document.write("<script id=__onDOMContentLoaded defer src=//:><\/script>");

becomes:

document.write("<script id=__onDOMContentLoaded defer src=

This is enough to render useless any application that relies on scriptaculous . The funny part is that it only happen if the application mode is DEPLOYMENT, as in DEVELOPMENT this code is not active.

I suggest dropping this functionality altogether - the logic for doing the stripping is extremely complicated and what is the point of stripping whitespace and comments from something that is going to be gzip compressed anyway?

Issue Links

• is related to: WICKET-918 allow for pluggable javascript compression algorithms