Fixed an issue with .not("#foo, bar") not working correctly, closes #3757.

diff --git a/src/core.js b/src/core.js
index b5de145fef513a71cd9c56699a82acdb30cff17b..92a6bd4a8c93554d7812e9d9142ea81ef43dd2b6 100644 (file)
--- a/src/core.js
+++ b/src/core.js
@@ -24,7 +24,7 @@ var jQuery = window.jQuery = window.$ = function( selector, context ) {
 var quickExpr = /^[^<]*(<(.|\s)+>)[^>]*$|^#([\w-]+)$/,
 
 // Is it a simple selector
-       isSimple = /^.[^:#\[\.]*$/,
+       isSimple = /^.[^:#\[\.,]*$/,
 
 // Will speed up references to undefined, and allows munging its name.
        undefined;

diff --git a/test/unit/core.js b/test/unit/core.js
index 1f0850c02468485b8bd770ab53696b023c308613..71bfc8118d8ba5d86f0e14e2bab6e3259a89f01e 100644 (file)
--- a/test/unit/core.js
+++ b/test/unit/core.js
@@ -1345,7 +1345,7 @@ test("closest()", function() {
 });
 
 test("not()", function() {
-       expect(8);
+       expect(11);
        equals( jQuery("#main > p#ap > a").not("#google").length, 2, "not('selector')" );
        equals( jQuery("#main > p#ap > a").not(document.getElementById("google")).length, 2, "not(DOMElement)" );
        isSet( jQuery("p").not(".result").get(), q("firstp", "ap", "sndp", "en", "sap", "first"), "not('.class')" );
@@ -1356,6 +1356,10 @@ test("not()", function() {
 
        var selects = jQuery("#form select");
        isSet( selects.not( selects[1] ), q("select1", "select3"), "filter out DOM element");
+
+       isSet( jQuery('#ap *').not('code'), q("google", "groups", "anchor1", "mark"), "not('tag selector')" );
+       isSet( jQuery('#ap *').not('code, #mark'), q("google", "groups", "anchor1"), "not('tag, ID selector')" );
+       isSet( jQuery('#ap *').not('#mark, code'), q("google", "groups", "anchor1"), "not('ID, tag selector')"); 
 });
 
 test("andSelf()", function() {