Side navigation
#8768 closed bug (fixed)
Opened April 05, 2011 11:10AM UTC
Closed July 03, 2012 10:08PM UTC
Chunker has problems with single quote in attribute filter
| Reported by: | orslumen@gmail.com | Owned by: | timmywil |
|---|---|---|---|
| Priority: | high | Milestone: | 1.next |
| Component: | selector | Version: | 1.5.1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
Hello,
When a Browser does not support querySelectorAll (e.g. EnvJs headless browser), the "old" Sizzle implementation is used.
With that Sizzle implementation the selector
jQuery(':button[title="Don\\'t Save"]') fails. The exception is: Syntax error, unrecognized expression: Save"]It seems like the Chunker doesn't like the single quote within the attribute filter.
Note that when querySelectorAll is available (e.g. in FireFox 3.6) the selector works fine.
Attachments (0)
Change History (11)
Changed April 07, 2011 01:38PM UTC by comment:1
| component: | unfiled → selector |
|---|---|
| owner: | → orslumen@gmail.com |
| status: | new → pending |
Changed April 08, 2011 08:40AM UTC by comment:2
Ok, here it is: http://jsfiddle.net/W36Av/
Please note that you need a browser that does not support *querySelectorAll* (IE 6, IE 7, FF 3.0) to reproduce the error.
Changed April 23, 2011 07:47AM UTC by comment:3
| resolution: | → invalid |
|---|---|
| status: | pending → closed |
Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!
Changed April 23, 2011 09:29AM UTC by comment:4
Obviously I was not aware of the correct WF in this tracker. I replied promptly to the question asked two weeks ago, but should have also manually changed the status(?).
Anyways, the bug is still there, so I guess it would be best to reopen this ticket.
Changed April 23, 2011 03:44PM UTC by comment:5
| priority: | undecided → high |
|---|---|
| resolution: | invalid |
| status: | closed → reopened |
Looks like you opened the ticket as orsiumen and posted the fiddle as anonymous so the tracker didn't think you responded. Reopening. Agree that the escaped quote may be the issue.
Changed April 23, 2011 03:44PM UTC by comment:6
| status: | reopened → open |
|---|
Changed April 23, 2011 05:36PM UTC by comment:7
Aha, I see. Saved my email address as a preference. Next time I won't forget ;)
Changed July 12, 2011 06:04PM UTC by comment:8
Confirmed in bug triage.
Changed July 03, 2012 02:27AM UTC by comment:9
Confirmed in 1.8beta
Changed July 03, 2012 02:33AM UTC by comment:10
| owner: | orslumen@gmail.com → timmywil |
|---|---|
| status: | open → assigned |
Changed July 03, 2012 10:08PM UTC by comment:11
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
Thanks for taking the time to contribute to the jQuery project! Please provide a reduced jsFiddle test case to help us assess your ticket!
Additionally, test against the jQuery Edge version to ensure the issue still exists.