Fixed not(jQuery) (need. more. tests.)
authorJörn Zaefferer <joern.zaefferer@gmail.com>
Tue, 23 Jan 2007 10:32:41 +0000 (10:32 +0000)
committerJörn Zaefferer <joern.zaefferer@gmail.com>
Tue, 23 Jan 2007 10:32:41 +0000 (10:32 +0000)
src/jquery/coreTest.js
src/jquery/jquery.js

index 28ba007..a9c3e26 100644 (file)
@@ -494,10 +494,11 @@ test("filter()", function() {
        isSet( $("p").filter(function() { return !$("a", this).length }).get(), q("sndp", "first"), "filter(Function)" );\r
 });\r
 \r
-test("not(String)", function() {\r
-       expect(2);\r
+test("not()", function() {\r
+       expect(3);\r
        ok( $("#main > p#ap > a").not("#google").length == 2, "not('selector')" );\r
        isSet( $("p").not("#ap, #sndp, .result").get(), q("firstp", "en", "sap", "first"), "not('selector, selector')" );\r
+       isSet( $("p").not($("#ap, #sndp, .result")).get(), q("firstp", "en", "sap", "first"), "not(jQuery)" );\r
 });\r
 \r
 \r
index 38dee9c..19ee7c1 100644 (file)
@@ -927,14 +927,14 @@ jQuery.fn = jQuery.prototype = {
        not: function(t) {
                return this.pushStack(
                        t.constructor == String &&
-                       jQuery.multiFilter(t,this,true) ||
-
-                       jQuery.grep(this,function(a){
-                                       if ( t.constructor == Array || t.jquery )
-                                               return jQuery.inArray( t, a ) < 0;
-                                       else
-                                               return a != t;
-                       }) );
+                       jQuery.multiFilter(t, this, true) ||
+
+                       jQuery.grep(this, function(a) {
+                               return ( t.constructor == Array || t.jquery )
+                                       ? console.log("t: %o a: %o", t, a) | jQuery.inArray( a, t ) < 0
+                                       : a != t;
+                       })
+               );
        },
 
        /**