Use a for loop rather than for/in loop when copying events, so that code will work...

[jquery.git] / src / manipulation.js

diff --git a/src/manipulation.js b/src/manipulation.js
index 63203ef..9d70a7b 100644 (file)
--- a/src/manipulation.js
+++ b/src/manipulation.js
@@ -370,14 +370,14 @@ function root( elem, cur ) {
 }
 
 function cloneCopyEvent(orig, ret) {
-       var i = 0;
+       var node = 0;
 
        ret.each(function() {
-               if ( this.nodeType !== 1 || this.nodeName !== (orig[i] && orig[i].nodeName) ) {
+               if ( this.nodeType !== 1 || this.nodeName !== (orig[node] && orig[node].nodeName) ) {
                        return;
                }
 
-               var oldData = jQuery.data( orig[i++] ),
+               var oldData = jQuery.data( orig[node++] ),
                        curData = jQuery.data( this, oldData ),
                        events = oldData && oldData.events;
 
@@ -386,8 +386,8 @@ function cloneCopyEvent(orig, ret) {
                        curData.events = {};
 
                        for ( var type in events ) {
-                               for ( var handler in events[ type ] ) {
-                                       jQuery.event.add( this, type, events[ type ][ handler ], events[ type ][ handler ].data );
+                               for ( var i = 0, l = events[ type ].length; i < l; i++ ) {
+                                       jQuery.event.add( this, type, events[ type ][ i ], events[ type ][ i ].data );
                                }
                        }
                }
@@ -415,17 +415,15 @@ function cloneFixAttributes(src, dest) {
        // attribute) to identify the type of content to display
        if ( nodeName === "object" ) {
                dest.outerHTML = src.outerHTML;
-       }
 
        // IE6-8 fails to persist the checked state of a cloned checkbox
        // or radio button
-       else if ( nodeName === "input" && src.checked ) {
+       } else if ( nodeName === "input" && src.checked ) {
                dest.defaultChecked = dest.checked = src.checked;
-       }
 
        // IE6-8 fails to return the selected option to the default selected
        // state when cloning options
-       else if ( nodeName === "option" ) {
+       } else if ( nodeName === "option" ) {
                dest.selected = src.defaultSelected;
        }