From 1a4a081c2eff40ff0dc84c6f9a1aec412660e263 Mon Sep 17 00:00:00 2001
From: Michael M Slusarz <slusarz@curecanti.org>
Date: Sat, 17 Jan 2009 19:46:58 -0700
Subject: [PATCH] Use identify() when appropriate

---
 imp/js/src/DimpBase.js |  8 ++++----
 imp/js/src/dragdrop.js | 12 ++++++------
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/imp/js/src/DimpBase.js b/imp/js/src/DimpBase.js
index 1d8039568..c93c3e6b1 100644
--- a/imp/js/src/DimpBase.js
+++ b/imp/js/src/DimpBase.js
@@ -690,7 +690,7 @@ var DimpBase = {
         if (!elt) {
             return;
         }
-        this.msgSelect(elt.readAttribute('id'), { ctrl: true, right: true });
+        this.msgSelect(elt.identify(), { ctrl: true, right: true });
         e.stop();
     },
 
@@ -1642,7 +1642,7 @@ var DimpBase = {
                     tmp.up('LI.custom');
                 }
                 if (tmp) {
-                    this.go('app:' + tmp.down('A').readAttribute('id').substring(3));
+                    this.go('app:' + tmp.down('A').identify().substring(3));
                     e.stop();
                     return;
                 }
@@ -1650,7 +1650,7 @@ var DimpBase = {
 
             case 'tabbar':
                 if (e.element().hasClassName('applicationtab')) {
-                    this.go('app:' + e.element().readAttribute('id').substring(6));
+                    this.go('app:' + e.element().identify().substring(6));
                     e.stop();
                     return;
                 }
@@ -2242,7 +2242,7 @@ var DimpBase = {
     // c = (element) Child element
     isSubfolder: function(p, c)
     {
-        var sf = $(this.getSubFolderId(p.readAttribute('id')));
+        var sf = $(this.getSubFolderId(p.identify()));
         return sf && c.descendantOf(sf);
     },
 
diff --git a/imp/js/src/dragdrop.js b/imp/js/src/dragdrop.js
index 3060640bb..2d683e520 100644
--- a/imp/js/src/dragdrop.js
+++ b/imp/js/src/dragdrop.js
@@ -79,7 +79,7 @@ var DragDrop = {
                 $(document.body).insert(this.div);
             }
 
-            this.drags.set(obj.element.readAttribute('id'), obj);
+            this.drags.set(obj.element.identify(), obj);
         },
 
         unregister: function(obj)
@@ -88,7 +88,7 @@ var DragDrop = {
                 this.drag.deactivate();
             }
 
-            this.drags.unset(obj.element.readAttribute('id'));
+            this.drags.unset(obj.element.identify());
 
             if (!this.drags.size() && this.div) {
                 this.div.remove();
@@ -97,7 +97,7 @@ var DragDrop = {
 
         get_drag: function(el)
         {
-            return this.drags.get(Object.isElement(el) ? $(el).readAttribute('id') : el);
+            return this.drags.get(Object.isElement(el) ? $(el).identify() : el);
         },
 
         activate: function(drag)
@@ -127,7 +127,7 @@ var DragDrop = {
 
         register: function(obj)
         {
-            this.drops.set(obj.element.readAttribute('id'), obj);
+            this.drops.set(obj.element.identify(), obj);
         },
 
         unregister: function(obj)
@@ -136,12 +136,12 @@ var DragDrop = {
                 this.drop = null;
             }
 
-            this.drops.unset(obj.element.readAttribute('id'));
+            this.drops.unset(obj.element.identify());
         },
 
         get_drop: function(el)
         {
-            return this.drops.get(Object.isElement(el) ? $(el).readAttribute('id') : el);
+            return this.drops.get(Object.isElement(el) ? $(el).identify() : el);
         }
     },
 
-- 
2.11.0