Don't use uniqueID in PIE.Util.getUID to avoid the check on non-eleme…

…nts. Make getUID for objects only.
hbdev012 · Nov 25, 2011 · 576790b · 576790b
1 parent 887a062
commit 576790b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/sources/Element.js b/sources/Element.js
@@ -452,12 +452,12 @@ PIE.Element = (function() {
     }
 
     Element.getInstance = function( el ) {
-        var id = PIE.Util.getUID( el );
+        var id = el[ 'uniqueID' ];
         return wrappers[ id ] || ( wrappers[ id ] = new Element( el ) );
     };
 
     Element.destroy = function( el ) {
-        var id = PIE.Util.getUID( el ),
+        var id = el[ 'uniqueID' ],
             wrapper = wrappers[ id ];
         if( wrapper ) {
             wrapper.destroy();

diff --git a/sources/Util.js b/sources/Util.js
@@ -10,11 +10,12 @@
 
         /**
          * Generate and return a unique ID for a given object. The generated ID is stored
-         * as a property of the object for future reuse. DOM Elements use the builtin uniqueID property.
+         * as a property of the object for future reuse. For DOM Elements, don't use this
+         * but use the IE-native uniqueID property instead.
          * @param {Object} obj
          */
         getUID: function( obj ) {
-            return obj && obj[ 'uniqueID' ] || obj[ '_pieId' ] || ( obj[ '_pieId' ] = '_' + idNum++ );
+            return obj && obj[ '_pieId' ] || ( obj[ '_pieId' ] = '_' + idNum++ );
         },