Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#7214 closed enhancement (fixed)

dojo.query matches differently with application/xhtml+xml

Reported by: dante Owned by: alex
Priority: high Milestone: 1.3
Component: Query Version: 1.1.1
Keywords: Cc: James Burke, DASPRiD
Blocked By: Blocking:

Description

There might be two bugs burried in this ticket, but:

When setting a content-type of application/xhtml+xml on a page (via server, php, whatever) a couple of things happen:

dojo.query("a.someClass").length == 0 

on a dom with <a class="someClass"> . It works as expected without the header sent.

It also seems the header causes the _base package system to fail somehow. no files beyond dojo.js + _base.js are loaded, and console throws errors about "console not being defined" ... ironically.

tested FF3 only. attached is a small PHP testcase with more notes inline.

Attachments (1)

querytest.php (1.7 KB) - added by dante 11 years ago.
trunk, build, and x-domain testcase

Download all attachments as: .zip

Change History (8)

Changed 11 years ago by dante

Attachment: querytest.php added

trunk, build, and x-domain testcase

comment:1 Changed 11 years ago by dante

Cc: DASPRiD added

comment:2 Changed 11 years ago by dylan

Milestone: tbd1.2

comment:3 Changed 11 years ago by alex

Resolution: wontfix
Status: newclosed

I spent some time looking into this, and the long and short of it is that I'm not gonna support application/xhtml+xml with Dojo Core. It's not worth doing. Much of the DOM just breaks anyway (document.write isn't available, etc.), meaning that you can't really develop with it in the first place. Trying to shoe-horn Dojo into an *intentionally* broken environment seems like it's not a great use of time.

comment:4 Changed 11 years ago by alex

Milestone: 1.21.3
Resolution: wontfix
Status: closedreopened
Type: defectenhancement

comment:5 Changed 11 years ago by alex

Status: reopenednew

comment:6 Changed 11 years ago by alex

Resolution: fixed
Status: newclosed

(In [15411]) make the default loader work in strict XHTML environments and fix Opera to correctly detect case sensitivity in XML/XHTML environments for queries. A small PHP script is added to test these. Sadly, both Safari and Opera don't observe script ordering for injected tags, so we had to make the dev loader script a bit more complicated to accomidate. Fixes #7455. Fixes #7214. !strict

comment:7 Changed 10 years ago by bill

Component: CoreQuery
Note: See TracTickets for help on using tickets.