Changeset 11999

Timestamp:

01/11/08 01:04:00 (7 months ago)

Author:

bill

Message:

Tree: Fix label on dragged elements showing up as "undefined" on IE6. Also removed unused function. Fixes #5559. !strict

Location:

dijit/trunk

Files:

• tests/tree/test_Tree_DnD.html (modified) (2 diffs)
• _tree/dndSource.js (modified) (1 diff)

dijit/trunk/tests/tree/test_Tree_DnD.html

@@ -119 +119 @@
                         }
                         return false;
-                }
-
-                function tree2ItemCreator(nodes){
-                        var items = []
-
-                        for(var i=0;i<nodes.length;i++){
-                                items.push({
-                                        "name":nodes[i].textContent,
-                                        "id": nodes[i].id
-                                });
-                        }
-                        return items;
                 }
 
@@ -195 +183 @@
                                 <div class="dojoDndItem" id="1008">Spinach</div>
                                 <div class="dojoDndItem" id="1009">Cucumber</div>
-                                <div class="dojoDndItem" id="1010">Carot</div>
+                                <div class="dojoDndItem" id="1010">Carrot</div>
                                 <div class="dojoDndItem" id="1011">Potato</div>
                                 <div class="dojoDndItem" id="1012">Grape</div>
                                 <div class="dojoDndItem" id="1013">Lemon</div>
-                                <div class="dojoDndItem" id="1010">Letuce</div>
+                                <div class="dojoDndItem" id="1010">Lettuce</div>
                                 <div class="dojoDndItem" id="1010">Peanut</div>
                         </div>

dijit/trunk/_tree/dndSource.js

@@ -236 +236 @@
 
         itemCreator: function(nodes){
-                var items = []
-
-                for(var i=0;i<nodes.length;i++){
-                        items.push({
-                                "name":nodes[i].textContent,
-                                "id": nodes[i].id
-                        });
-                }
-
-                return items;
+                return dojo.map(nodes, function(node){
+                        return {
+                                "id": node.id,
+                                "name": node.textContent || node.innerText || ""
+                        };
+                });
         },