Changeset 3159

Timestamp:

09/08/07 23:31:23 (2 years ago)

Author:

jeresig

Message:

Landing the new expando management code. Completely overhauls how data is associated with elements.

Plugins will be most interested in:
- jQuery.data(elem) -> Unique ID for the element
- jQuery.data(elem, name) -> Named data store for the element
- jQuery.data(elem, name, value) -> Saves a value to the named data store
- jQuery.removeData(elem) -> Remove the expando and the complete data store
- jQuery.removeData(elem, name) -> Removes just this one named data store

jQuery's .remove() and .empty() automatically clean up after themselves. Once an element leaves a DOM document their events are no longer intact. Thus, statements like so:

  $("#foo").remove().appendTo("#bar");

should be written like so:

  $("#foo").appendTo("#bar");

in order to avoid losing the bound events.

Location:

trunk/jquery

Files:

• src/core.js (modified) (6 diffs)
• src/event.js (modified) (7 diffs)
• src/selector.js (modified) (5 diffs)
• test/data/testrunner.js (modified) (1 diff)
• test/unit/event.js (modified) (2 diffs)

trunk/jquery/src/core.js

@@ -239 +239 @@
 
             this.find("*").andSelf().each(function(i) {
-                var events = this.$events;
+                var events = jQuery.data(this, "events");
                 for ( var type in events )
                     for ( var handler in events[type] )
@@ -425 +425 @@
 };
 
+var expando = "jQuery" + (new Date()).getTime(), uuid = 0;
+
 jQuery.extend({
     noConflict: function(deep) {
@@ -463 +465 @@
     nodeName: function( elem, name ) {
         return elem.nodeName && elem.nodeName.toUpperCase() == name.toUpperCase();
+    },
+    
+    cache: {},
+    
+    data: function( elem, name, data ) {
+        var id = elem[ expando ];
+
+        // Compute a unique ID for the element
+        if ( !id ) 
+            id = elem[ expando ] = ++uuid;
+
+        // Only generate the data cache if we're
+        // trying to access or manipulate it
+        if ( name && !jQuery.cache[ id ] )
+            jQuery.cache[ id ] = {};
+        
+        // Prevent overriding the named cache with undefined values
+        if ( data != undefined )
+            jQuery.cache[ id ][ name ] = data;
+        
+        // Return the named cache data, or the ID for the element   
+        return name ? jQuery.cache[ id ][ name ] : id;
+    },
+    
+    removeData: function( elem, name ) {
+        var id = elem[ expando ];
+
+        // If we want to remove a specific section of the element's data
+        if ( name ) {
+            // Remove the section of cache data
+            delete jQuery.cache[ id ][ name ];
+
+            // If we've removed all the data, remove the element's cache
+            name = "";
+            for ( name in jQuery.cache[ id ] ) break;
+            if ( !name )
+                jQuery.removeData( elem );
+
+        // Otherwise, we want to remove all of the element's data
+        } else {
+            // Clean up the element expando
+            try {
+                delete elem[ expando ];
+            } catch(e){
+                // IE has trouble directly removing the expando
+                // but it's ok with using removeAttribute
+                elem.removeAttribute( expando );
+            }
+
+            // Completely remove the data cache
+            delete jQuery.cache[ id ];
+        }
     },
 
@@ -850 +904 @@
 
     unique: function(first) {
-        var r = [], num = jQuery.mergeNum++;
+        var r = [], done = {};
 
         try {
-            for ( var i = 0, fl = first.length; i < fl; i++ )
-                if ( num != first[i].mergeNum ) {
-                    first[i].mergeNum = num;
+            for ( var i = 0, fl = first.length; i < fl; i++ ) {
+                var id = jQuery.data(first[i]);
+                if ( !done[id] ) {
+                    done[id] = true;
                     r.push(first[i]);
                 }
+            }
         } catch(e) {
             r = first;
@@ -864 +920 @@
         return r;
     },
-
-    mergeNum: 0,
 
     grep: function(elems, fn, inv) {
@@ -991 +1045 @@
     },
     remove: function(a){
-        if ( !a || jQuery.filter( a, [this] ).r.length )
+        if ( !a || jQuery.filter( a, [this] ).r.length ) {
+            jQuery.removeData( this );
             this.parentNode.removeChild( this );
+        }
     },
     empty: function() {
+        // Clean up the cache
+        jQuery("*", this).each(function(){ jQuery.removeData(this); });
+
         while ( this.firstChild )
             this.removeChild( this.firstChild );

trunk/jquery/src/event.js

@@ -42 +42 @@
 
         // Init the element's event structure
-        if (!element.$events)
-            element.$events = {};
-        
-        if (!element.$handle)
-            element.$handle = function() {
-                // returned undefined or false
-                var val;
-
-                // Handle the second event of a trigger and when
-                // an event is called after a page has unloaded
-                if ( typeof jQuery == "undefined" || jQuery.event.triggered )
-                  return val;
-                
-                val = jQuery.event.handle.apply(element, arguments);
-                
+        var events = jQuery.data(element, "events") || jQuery.data(element, "events", {});
+        
+        var handle = jQuery.data(element, "handle", function(){
+            // returned undefined or false
+            var val;
+
+            // Handle the second event of a trigger and when
+            // an event is called after a page has unloaded
+            if ( typeof jQuery == "undefined" || jQuery.event.triggered )
                 return val;
-            };
+            
+            val = jQuery.event.handle.apply(element, arguments);
+            
+            return val;
+        });
 
         // Get the current list of functions bound to this event
-        var handlers = element.$events[type];
+        var handlers = events[type];
 
         // Init the event handler queue
         if (!handlers) {
-            handlers = element.$events[type] = {};  
+            handlers = events[type] = {};   
             
             // And bind the global event handler to the element
             if (element.addEventListener)
-                element.addEventListener(type, element.$handle, false);
+                element.addEventListener(type, handle, false);
             else
-                element.attachEvent("on" + type, element.$handle);
+                element.attachEvent("on" + type, handle);
         }
 
@@ -86 +84 @@
     // Detach an event or set of events from an element
     remove: function(element, type, handler) {
-        var events = element.$events, ret, index;
+        var events = jQuery.data(element, "events"), ret, index;
 
         // Namespaced event handlers
@@ -112 +110 @@
                 // remove all handlers for the given type
                 else
-                    for ( handler in element.$events[type] )
+                    for ( handler in events[type] )
                         // Handle the removal of namespaced events
                         if ( !parts[1] || events[type][handler].type == parts[1] )
@@ -121 +119 @@
                 if ( !ret ) {
                     if (element.removeEventListener)
-                        element.removeEventListener(type, element.$handle, false);
+                        element.removeEventListener(type, jQuery.data(element, "handle"), false);
                     else
-                        element.detachEvent("on" + type, element.$handle);
+                        element.detachEvent("on" + type, jQuery.data(element, "handle"));
                     ret = null;
                     delete events[type];
@@ -131 +129 @@
             // Remove the expando if it's no longer used
             for ( ret in events ) break;
-            if ( !ret )
-                element.$handle = element.$events = null;
+            if ( !ret ) {
+                jQuery.removeData( element, "events" );
+                jQuery.removeData( element, "handle" );
+            }
         }
     },
@@ -157 +157 @@
 
             // Trigger the event
-            if ( jQuery.isFunction( element.$handle ) )
-                val = element.$handle.apply( element, data );
+            if ( jQuery.isFunction( jQuery.data(element, "handle") ) )
+                val = jQuery.data(element, "handle").apply( element, data );
 
             // Handle triggering native .onfoo handlers
@@ -195 +195 @@
         event.type = parts[0];
 
-        var c = this.$events && this.$events[event.type], args = Array.prototype.slice.call( arguments, 1 );
+        var c = jQuery.data(this, "events") && jQuery.data(this, "events")[event.type], args = Array.prototype.slice.call( arguments, 1 );
         args.unshift( event );
 

trunk/jquery/src/selector.js

@@ -141 +141 @@
                     r = [];
 
-                    var nodeName = m[2], mergeNum = jQuery.mergeNum++;
+                    var nodeName = m[2], merge = {};
                     m = m[1];
 
@@ -148 +148 @@
                         for ( ; n; n = n.nextSibling )
                             if ( n.nodeType == 1 ) {
-                                if ( m == "~" && n.mergeNum == mergeNum ) break;
+                                var id = jQuery.data(n);
+
+                                if ( m == "~" && merge[id] ) break;
                                 
                                 if (!nodeName || n.nodeName.toUpperCase() == nodeName.toUpperCase() ) {
-                                    if ( m == "~" ) n.mergeNum = mergeNum;
+                                    if ( m == "~" ) merge[id] = true;
                                     r.push( n );
                                 }
@@ -347 +349 @@
             // We can get a speed boost by handling nth-child here
             } else if ( m[1] == ":" && m[2] == "nth-child" ) {
-                var num = jQuery.mergeNum++, tmp = [],
+                var merge = {}, tmp = [],
                     test = /(\d*)n\+?(\d*)/.exec(
                         m[3] == "even" && "2n" || m[3] == "odd" && "2n+1" ||
@@ -354 +356 @@
 
                 for ( var i = 0, rl = r.length; i < rl; i++ ) {
-                    var node = r[i], parentNode = node.parentNode;
-
-                    if ( num != parentNode.mergeNum ) {
+                    var node = r[i], parentNode = node.parentNode, id = jQuery.data(parentNode);
+
+                    if ( !merge[id] ) {
                         var c = 1;
 
@@ -363 +365 @@
                                 n.nodeIndex = c++;
 
-                        parentNode.mergeNum = num;
+                        merge[id] = true;
                     }
 

trunk/jquery/test/data/testrunner.js

@@ -162 +162 @@
  */
 function reset() {
-    document.getElementById('main').innerHTML = _config.fixture;
+    $("#main").html( _config.fixture );
 }
 

trunk/jquery/test/unit/event.js

@@ -9 +9 @@
     };
     $("#firstp").bind("click", {foo: "bar"}, handler).click().unbind("click", handler);
-    
-    ok( !$("#firstp").get(0).$events, "Event handler unbound when using data." );
+
+    ok( !jQuery.data($("#firstp")[0], "events"), "Event handler unbound when using data." );
     
     reset();
@@ -109 +109 @@
     el.click(function() { return; });
     el.unbind('change',function(){ return; });
-    for (var ret in el[0].$events['click']) break;
+    for (var ret in jQuery.data(el[0], "events")['click']) break;
     ok( ret, "Extra handlers weren't accidentally removed." );
 
     el.unbind('click');
-    ok( !el[0].$events, "Removed the events expando after all handlers are unbound." );
+    ok( !jQuery.data(el[0], "events"), "Removed the events expando after all handlers are unbound." );
 });