Skip to main content

Bug Tracker

Side navigation

#13968 closed bug (notabug)

Opened May 30, 2013 04:48PM UTC

Closed May 30, 2013 05:02PM UTC

Last modified June 06, 2013 05:18PM UTC

selector functionality broken

Reported by: dev.blanked@gmail.com Owned by:
Priority: undecided Milestone: None
Component: unfiled Version: 1.9.1
Keywords: Cc:
Blocked by: Blocking:
Description

Please refer : http://jsfiddle.net/NHgBP/1/

In jQuery 1.7.2 the above piece of code works perfectly. But in 1.9.1 it fails. Seems like in 1.9.1 the selector functionality has been broken and it no longer handles '[' properly. Due to this jQuery mobile is broken as well

Please refer https://github.com/jquery/jquery-mobile/issues/6052

jQuery mobile thinks this is a bug in the core

Attachments (0)
Change History (7)

Changed May 30, 2013 05:02PM UTC by dmethvin comment:1

resolution: → notabug
status: newclosed
no longer handles '[' properly

The test case reduces to $("chk['3']-1"). The [ is not escaped properly. It isn't even an id selector.

https://www.google.com/search?q=jquery+escape+id+selector

Changed May 30, 2013 05:06PM UTC by dmethvin comment:2

Changed May 31, 2013 05:01AM UTC by dev.blanked@gmail.com comment:3

This works well with jQuery < 1.8

Please see

https://github.com/jquery/jquery-mobile/issues/6052

jQuery mobile functionality is broken due to this issue.

Changed May 31, 2013 12:56PM UTC by gibson042 comment:4

Yes, we were pleased to bring in many Sizzle improvements with jQuery 1.8, particularly those around parsing selectors.

Changed June 05, 2013 04:49AM UTC by dev.blanked@gmail.com comment:5

You guys are saying this is not a bug.. jQuery mobile guys are saying it's an issue in the core.. :-) What should the users be doing.. stop using latest jQuery or stop using latest jQuery mobile

Changed June 05, 2013 01:09PM UTC by dmethvin comment:6

Us guys are trying to be nice to someone who doesn't understand and is not listening. The current version of jQuery is doing this properly. You are not escaping your selectors. I posted a working example above. Here is another example. Look at it carefully.

http://jsfiddle.net/NHgBP/4/

Changed June 06, 2013 05:18PM UTC by dev.blanked@gmail.com comment:7

Replying to [comment:6 dmethvin]:

Us guys are trying to be nice to someone who doesn't understand and is not listening. The current version of jQuery is doing this properly. You are not escaping your selectors. I posted a working example above. Here is another example. Look at it carefully. http://jsfiddle.net/NHgBP/4/

Thx people. I'm just using jQuery mobile. The sample code I used was to demonstrate the issue. This is an issue present in jQuery mobile. Just curious when suddenly things get broken which worked perfectly before...