Fixed typo in logic, also disabled function setters in this case to allow the functi...
[jquery.git] / src / core.js
index 0d5588c..edd1089 100644 (file)
@@ -33,7 +33,10 @@ var jQuery = function( selector, context ) {
        rsingleTag = /^<(\w+)\s*\/?>(?:<\/\1>)?$/,
 
        // Keep a UserAgent string for use with jQuery.browser
-       userAgent = navigator.userAgent.toLowerCase(),
+       userAgent = navigator.userAgent,
+
+       // For matching the engine and version of the browser
+       browserMatch,
        
        // Has the ready events already been bound?
        readyBound = false,
@@ -53,7 +56,7 @@ var jQuery = function( selector, context ) {
 
 jQuery.fn = jQuery.prototype = {
        init: function( selector, context ) {
-               var match, elem, ret, doc, parent;
+               var match, elem, ret, doc;
 
                // Handle $(""), $(null), or $(undefined)
                if ( !selector ) {
@@ -84,16 +87,17 @@ jQuery.fn = jQuery.prototype = {
                                        ret = rsingleTag.exec( selector );
 
                                        if ( ret ) {
-                                               selector = [ doc.createElement( ret[1] ) ];
+                                               if ( jQuery.isPlainObject( context ) ) {
+                                                       selector = [ document.createElement( ret[1] ) ];
+                                                       jQuery.fn.attr.call( selector, context, true );
+
+                                               } else {
+                                                       selector = [ doc.createElement( ret[1] ) ];
+                                               }
 
                                        } else {
                                                ret = buildFragment( [ match[1] ], [ doc ] );
-                                               parent = ret.cacheable ? ret.fragment.cloneNode(true) : ret.fragment;
-                                               selector = [];
-
-                                               while ( parent.firstChild ) {
-                                                       selector.push( parent.removeChild( parent.firstChild ) );
-                                               }
+                                               selector = (ret.cacheable ? ret.fragment.cloneNode(true) : ret.fragment).childNodes;
                                        }
 
                                // HANDLE: $("#id")
@@ -163,7 +167,7 @@ jQuery.fn = jQuery.prototype = {
                return this.length;
        },
 
-       toArray: function(){
+       toArray: function() {
                return slice.call( this, 0 );
        },
 
@@ -224,12 +228,12 @@ jQuery.fn = jQuery.prototype = {
                jQuery.bindReady();
 
                // If the DOM is already ready
-               if ( jQuery.isReady && !readyList ) {
+               if ( jQuery.isReady ) {
                        // Execute the function immediately
                        fn.call( document, jQuery );
 
                // Otherwise, remember the function for later
-               } else {
+               } else if ( readyList ) {
                        // Add the function to the wait list
                        readyList.push( fn );
                }
@@ -257,7 +261,7 @@ jQuery.fn = jQuery.prototype = {
        },
 
        map: function( callback ) {
-               return this.pushStack( jQuery.map(this, function(elem, i){
+               return this.pushStack( jQuery.map(this, function( elem, i ) {
                        return callback.call( elem, i, elem );
                }));
        },
@@ -312,10 +316,10 @@ jQuery.extend = jQuery.fn.extend = function() {
                                        continue;
                                }
 
-                               // Recurse if we're merging object literal values
-                               if ( deep && copy && jQuery.isPlainObject(copy) ) {
-                                       // Don't extend not object literals
-                                       var clone = src && jQuery.isPlainObject(src) ? src : {};
+                               // Recurse if we're merging object literal values or arrays
+                               if ( deep && copy && ( jQuery.isPlainObject(copy) || jQuery.isArray(copy) ) ) {
+                                       var clone = src && ( jQuery.isPlainObject(src) || jQuery.isArray(src) ) ? src
+                                               : jQuery.isArray(copy) ? [] : {};
 
                                        // Never move original objects, clone them
                                        target[ name ] = jQuery.extend( deep, clone, copy );
@@ -378,7 +382,10 @@ jQuery.extend({
        },
        
        bindReady: function() {
-               if ( readyBound ) { return; }
+               if ( readyBound ) {
+                       return;
+               }
+
                readyBound = true;
 
                // Catch cases where $(document).ready() is called after the
@@ -410,7 +417,7 @@ jQuery.extend({
 
                        try {
                                toplevel = window.frameElement == null;
-                       } catch(e){}
+                       } catch(e) {}
 
                        if ( document.documentElement.doScroll && toplevel ) {
                                doScrollCheck();
@@ -430,19 +437,22 @@ jQuery.extend({
        },
 
        isPlainObject: function( obj ) {
-               if ( toString.call(obj) !== "[object Object]" || typeof obj.nodeType === "number" ) {
+               // Must be an Object.
+               // Because of IE, we also have to check the presence of the constructor property.
+               // Make sure that DOM nodes and window objects don't pass through, as well
+               if ( !obj || toString.call(obj) !== "[object Object]" || obj.nodeType || obj.setInterval ) {
                        return false;
                }
                
-               // not own constructor property must be Object
+               // Not own constructor property must be Object
                if ( obj.constructor
                        && !hasOwnProperty.call(obj, "constructor")
                        && !hasOwnProperty.call(obj.constructor.prototype, "isPrototypeOf") ) {
                        return false;
                }
                
-               //own properties are iterated firstly,
-               //so to speed up, we can test last one if it is own or not
+               // Own properties are enumerated firstly, so to speed up,
+               // if last one is own, then all properties are own.
        
                var key;
                for ( key in obj ) {}
@@ -457,6 +467,8 @@ jQuery.extend({
                return true;
        },
 
+       noop: function() {},
+
        // Evalulates a script in a global context
        globalEval: function( data ) {
                if ( data && rnotwhite.test(data) ) {
@@ -607,19 +619,75 @@ jQuery.extend({
                return ret.concat.apply( [], ret );
        },
 
+       // A global GUID counter for objects
+       guid: 1,
+
+       proxy: function( fn, proxy, thisObject ) {
+               if ( arguments.length === 2 ) {
+                       if ( typeof proxy === "string" ) {
+                               thisObject = fn;
+                               fn = thisObject[ proxy ];
+                               proxy = undefined;
+
+                       } else if ( proxy && !jQuery.isFunction( proxy ) ) {
+                               thisObject = proxy;
+                               proxy = undefined;
+                       }
+               }
+
+               if ( !proxy && fn ) {
+                       proxy = function() {
+                               return fn.apply( thisObject || this, arguments );
+                       };
+               }
+
+               // Set the guid of unique handler to the same of original handler, so it can be removed
+               if ( fn ) {
+                       proxy.guid = fn.guid = fn.guid || proxy.guid || jQuery.guid++;
+               }
+
+               // So proxy can be declared as an argument
+               return proxy;
+       },
+
        // Use of jQuery.browser is frowned upon.
        // More details: http://docs.jquery.com/Utilities/jQuery.browser
-       browser: {
-               version: (/.*?(?:firefox|safari|opera|msie)[\/ ]([\d.]+)/.exec(userAgent) || [0,'0'])[1],
-               safari: /safari/.test( userAgent ),
-               opera: /opera/.test( userAgent ),
-               msie: /msie/.test( userAgent ) && !/opera/.test( userAgent ),
-               firefox: /firefox/.test( userAgent )
-       }
+       uaMatch: function( ua ) {
+               var ret = { browser: "" };
+
+               ua = ua.toLowerCase();
+
+               if ( /webkit/.test( ua ) ) {
+                       ret = { browser: "webkit", version: /webkit[\/ ]([\w.]+)/ };
+
+               } else if ( /opera/.test( ua ) ) {
+                       ret = { browser: "opera", version:  /version/.test( ua ) ? /version[\/ ]([\w.]+)/ : /opera[\/ ]([\w.]+)/ };
+                       
+               } else if ( /msie/.test( ua ) ) {
+                       ret = { browser: "msie", version: /msie ([\w.]+)/ };
+
+               } else if ( /mozilla/.test( ua ) && !/compatible/.test( ua ) ) {
+                       ret = { browser: "mozilla", version: /rv:([\w.]+)/ };
+               }
+
+               ret.version = (ret.version && ret.version.exec( ua ) || [0, "0"])[1];
+
+               return ret;
+       },
+
+       browser: {}
 });
 
-// Deprecated
-jQuery.browser.mozilla = /mozilla/.test( userAgent ) && !/(compatible|webkit)/.test( userAgent );
+browserMatch = jQuery.uaMatch( userAgent );
+if ( browserMatch.browser ) {
+       jQuery.browser[ browserMatch.browser ] = true;
+       jQuery.browser.version = browserMatch.version;
+}
+
+// Deprecated, use jQuery.browser.webkit instead
+if ( jQuery.browser.webkit ) {
+       jQuery.browser.safari = true;
+}
 
 if ( indexOf ) {
        jQuery.inArray = function( elem, array ) {
@@ -690,13 +758,13 @@ function evalScript( i, elem ) {
 
 // Mutifunctional method to get and set values to a collection
 // The value/s can be optionally by executed if its a function
-function access( elems, key, value, exec, fn ) {
+function access( elems, key, value, exec, fn, pass ) {
        var length = elems.length;
        
        // Setting many attributes
        if ( typeof key === "object" ) {
                for ( var k in key ) {
-                       access( elems, k, key[k], exec, fn );
+                       access( elems, k, key[k], exec, fn, value );
                }
                return elems;
        }
@@ -704,10 +772,10 @@ function access( elems, key, value, exec, fn ) {
        // Setting one attribute
        if ( value !== undefined ) {
                // Optionally, function values get executed if exec is true
-               exec = exec && jQuery.isFunction(value);
+               exec = !pass && exec && jQuery.isFunction(value);
                
                for ( var i = 0; i < length; i++ ) {
-                       fn( elems[i], key, exec ? value.call( elems[i], i ) : value );
+                       fn( elems[i], key, exec ? value.call( elems[i], i, fn( elems[i], key ) ) : value, pass );
                }
                
                return elems;