Bug Tracker

Ticket #3248: makeArray[5824].diff

File makeArray[5824].diff, 1.6 KB (added by flesler, 14 years ago)
  • jquery/src/core.js

     
    11251125                return (text || "").replace( /^\s+|\s+$/g, "" );
    11261126        },
    11271127
     1128        // Supports ANYTHING but functions
    11281129        makeArray: function( array ) {
    11291130                var ret = [];
    11301131
    11311132                if( array != null ){
    11321133                        var i = array.length;
    1133                         //the window, strings and functions also have 'length'
    1134                         if( i == null || array.split || array.setInterval || array.call )
     1134                        // The window, strings (and functions) also have 'length'
     1135                        if( i == null || array.split || array.setInterval )
    11351136                                ret[0] = array;
    11361137                        else
    11371138                                while( i )
  • jquery/test/unit/core.js

     
    16841684});
    16851685
    16861686test("jQuery.makeArray", function(){
    1687         expect(15);
     1687        expect(14);
    16881688
    16891689        equals( jQuery.makeArray(jQuery('html>*'))[0].nodeName, "HEAD", "Pass makeArray a jQuery object" );
    16901690
     
    17091709        ok( !!jQuery.makeArray( document.documentElement.childNodes ).slice(0,1)[0].nodeName, "Pass makeArray a childNodes array" );
    17101710
    17111711        //function, is tricky as it has length
    1712         equals( jQuery.makeArray( function(){ return 1;} )[0](), 1, "Pass makeArray a function" );
     1712        //equals( jQuery.makeArray( function(){ return 1;} )[0](), 1, "Pass makeArray a function" );
    17131713        //window, also has length
    17141714        equals( jQuery.makeArray(window)[0], window, "Pass makeArray the window" );
    17151715