Changeset 5500

Timestamp:

05/08/08 16:14:49 (4 days ago)

Author:

aflesler

Message:

jquery core: closes #2771
$.inArray now makes a === check because of IE.
$.inArray is used in $.fn.index, this is shorter, and breaks the loop when possible.
$.fn.index can receive a jquery object, and the first element is used

Files:

• trunk/jquery/src/core.js (modified) (2 diffs)

trunk/jquery/src/core.js

@@ -141 +141 @@
 
         // Locate the position of the desired element
-        this.each(function(i){
-            if ( this == elem )
-                ret = i;
-        });
-
-        return ret;
+        return jQuery.inArray( 
+            // If it receives a jQuery object, the first element is used
+            elem && elem.jquery ? elem[0] : elem
+        , this );
     },
 
@@ -1125 +1123 @@
     inArray: function( elem, array ) {
         for ( var i = 0, length = array.length; i < length; i++ )
-            if ( array[ i ] == elem )
+        // Use === because on IE, window == document
+            if ( array[ i ] === elem )
                 return i;