[OFBIZ-4647] Date Picker widget in for breaks Internet Explorer 7

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Cannot Reproduce
• Affects Version/s: Trunk
• Component/s: framework
• Labels: patch
• Sprint: Bug Crush Event - 21/2/2015
• Assignee: Unassigned
• Reporter: Christoph Neuroth
• Created: 05/Jan/12 15:27
• Updated: 18/Apr/15 12:24
• Resolved: 18/Apr/15 12:24

Description

When using a date field like in the code sample below, the generated JavaScript causes IE7 to stop loading the page with the infamous "Operation aborted" error and then redirect to an error page, making the screen unusable. For background information on the issue, see MS Knowledge Base 927917. This problem probably occurs with other js code blocks as well, however we only fixed this one for now because we urgently needed it for a client. This can be circumvented by moving the code in question into a function and binding that to the jQuery document.ready even as I did in the patch.

Example that breaks IE7 on current trunk:

<field id-name="fromDate" name="fromDate" tooltip="00:00"><date-time type="date"/></field>

Note: In the patch, I also added two missing semicolons joined some lines so that there is no newlines before that semicolon. While those did not cause errors here, I really doubt if generating JavaScript code using FreeMarker is that great an idae.