Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#16608 closed defect (invalid)

Parser fails to find any "dojo/require" script types on IE7 or IE8 when using double-quotes

Reported by: neville1355 Owned by: bill
Priority: undecided Milestone: tbd
Component: Parser Version: 1.8.3
Keywords: Cc:
Blocked By: Blocking:

Description

If dojo/require is used to alias an AMD module for a page declaratively, it will cause an exception in IE7 and IE8 due to an error in the dom query used in the dojo parser.

On line 733 of the parser, all dojo/require blocks are pulled out with the following statement: query("script[type='dojo/require']", root)

However, IE7 and IE8 do not consider single and double quotes as equivalent and, as a result, will not find any matching types if double quotes were used.

Recreate by: <script type="dojo/require"> at:'dojox/mvc/at' </script>

If you try to use "at" later in the page, you will get an exception.

Current workaround: Use single quotes when declaring the script type, e.g.

<script type='dojo/require'> at:'dojox/mvc/at' </script>

Change History (5)

comment:1 Changed 6 years ago by cjolif

Cc: cjolif added

comment:2 Changed 6 years ago by neville1355

I may have been a bit premature in my description of the problem.

It doesn't look like the double-quotes were the problem but rather that I set up dojo to load synchronously.

Apparently <script type="dojo/require"> is incompatible with dojo when the dojo config specifies async:false

comment:3 Changed 6 years ago by neville1355

Please close this bug. I'll open a new one with a more accurate description

comment:4 Changed 6 years ago by cjolif

Cc: cjolif removed
Resolution: invalid
Status: newclosed

per-user request

comment:5 Changed 6 years ago by bill

New ticket is #16609.

Note: See TracTickets for help on using tickets.