Use the new updateNode in the nodedrop handler

Instead of a separate Ajax call, just use the updateNode method and pass the new :parent_id for the node.
dradis · Jul 5, 2011 · c1c010f · c1c010f
1 parent c6cc95b
commit c1c010f
Showing 1 changed file with 2 additions and 22 deletions.
diff --git a/public/javascripts/nodestree.js b/public/javascripts/nodestree.js
@@ -239,7 +239,7 @@ dradis.NodesTree = Ext.extend(Ext.tree.TreePanel, {
     this.on('nodedrop', function(ev) {
       var point = ev.point;
       var node = ev.dropNode;
-      var p = { };
+      var p = { parent_id: null };
       if ( point == 'append') {
         p.parent_id = ev.target.id;
       } else {
@@ -248,27 +248,7 @@ dradis.NodesTree = Ext.extend(Ext.tree.TreePanel, {
           p.parent_id = parent.id;
         }
       }
-      Ext.Ajax.request({
-        url: 'nodes/' + node.id + '.json',
-        method: 'put',
-        params: {
-          data: Ext.util.JSON.encode(p)
-        },
-        success: function(response, options) {
-                    dradisstatus.setStatus({ 
-                      text: 'Node repositioned',
-                      clear: 5000
-                  });
-        },
-        failure: function(response, options) {
-                    dradisstatus.setStatus({
-                      text: 'An error occured with the Ajax request',
-                      iconCls: 'error',
-                      clear: 5000
-                    });
-        }
-      });
-
+      this.updateNode(node, p)
     });
 
     // ==================================================== /event handlers