Skip to main content

Bug Tracker

Side navigation

#8967 closed bug (worksforme)

Opened April 25, 2011 12:35PM UTC

Closed April 25, 2011 03:01PM UTC

Last modified April 29, 2011 08:12AM UTC

Selector [value=XXX] regress in 1.5.2

Reported by: grigorenko@mail.ru Owned by: grigorenko@mail.ru
Priority: low Milestone: 1.next
Component: selector Version: 1.5.2
Keywords: Cc:
Blocked by: Blocking:
Description

Selector "SELECT[name=FOO] option[value=BAR]" works in 1.4.4 and doesn't work in 1.5.2

In 1.5.2 there's error:

Uncaught Syntax error, unrecognized expression:

[value=D:\\Results\\BDS2007\\EXE\\MitraMessageCenterApp.exe]

k.errorjquery.js:16

k.filterjquery.js:16

kjquery.js:16

c.querySelectorAll.kjquery.js:16

d.fn.extend.findjquery.js:16

d.d.fn.d.initjquery.js:16

d.djquery.js:16

check_add:2222/:201

update:2222/:144

d.extend._Deferred.f.resolveWithjquery.js:16

vjquery.js:16

d.support.ajax.d.ajaxTransport.send.cjquery.js:16

I'm trying to append option to SELECT only if there isn't one there already:

function check_add(select_name, value) {
	if ($("SELECT[name=" + select_name + "] option[value=" + value + "]").length == 0) {
		$("SELECT[name=" + select_name + "]").append($("<option />").text(value || '').val(value || ''));
	}
}

Using Chrome 10.0.648.205 under Windows XP.

Attachments (0)
Change History (7)

Changed April 25, 2011 02:47PM UTC by timmywil comment:1

component: unfiledselector
owner: → grigorenko@mail.ru
priority: undecidedlow
status: newpending

Thanks for taking the time to contribute to the jQuery project! Please provide a reduced test case on http://jsFiddle.net that reproduces the issue experienced to help us assess your ticket.

Additionally, test against the jQuery (edge) version to ensure the issue still exists. I've created one here, but was unable to reproduce the issue described: http://jsfiddle.net/timmywil/CppFb/

Changed April 25, 2011 03:01PM UTC by rwaldron comment:2

resolution: → worksforme
status: pendingclosed

Test in chrome 10 & 12 on windows XP:

http://jsfiddle.net/rwaldron/JKZ2t/

Changed April 28, 2011 07:48AM UTC by grigorenko@mail.ru comment:3

The problem exists:

http://jsfiddle.net/JKZ2t/2/

Works in 1.4.4, doesn't work in 1.5.2 and edge:

"Syntax error, unrecognized expression: [value=D:\\Results\\BDS2007\\EXE\\TestEntityTransport.exe]"

Changed April 28, 2011 07:50AM UTC by grigorenko@mail.ru comment:4

Sorry, the right link is:

http://jsfiddle.net/JKZ2t/3/

Changed April 28, 2011 03:48PM UTC by timmywil comment:5

Quotes around the value are mandatory: http://jsfiddle.net/timmywil/JKZ2t/8/

Changed April 28, 2011 04:43PM UTC by anonymous comment:6

I got the same error. After add quotes to the value in selector, the error msg went away. Thanks, timmywil.

Changed April 29, 2011 08:12AM UTC by grigorenko@mail.ru comment:7

Forgot about quotes. Sorry, my mistake.