From ceaaa43e28f8c6a882c73d72a8f86bceb3356e14 Mon Sep 17 00:00:00 2001
From: "Vaibhav (OpenERP)" <vda@tinyerp.com>
Date: Wed, 24 Aug 2011 16:42:26 +0530
Subject: [PATCH] [FIX] add or edit node.

bzr revid: vda@tinyerp.com-20110824111226-9nexueoxmddi7ren
---
 addons/base_diagram/static/src/js/diagram.js | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/addons/base_diagram/static/src/js/diagram.js b/addons/base_diagram/static/src/js/diagram.js
index e2e9ce316d4..ad1c5fe1040 100644
--- a/addons/base_diagram/static/src/js/diagram.js
+++ b/addons/base_diagram/static/src/js/diagram.js
@@ -104,18 +104,13 @@ openerp.base_diagram.DiagramView = openerp.base.View.extend({
 		
 		this.do_update_pager();
 		
-		this.$element.find('.oe_diagram_button_new').click(function(){self.add_node()})
+		this.$element.find('.oe_diagram_button_new').click(function(){self.add_edit_node()})
 		
         if(this.id) {
         	self.get_diagram_info();
         }
 	},
 	
-	add_node: function() {
-		var self = this;
-		console.log('self>>', self)
-	},
-	
 	get_diagram_info: function() {
 		var self = this;
         this.rpc(
@@ -249,13 +244,16 @@ openerp.base_diagram.DiagramView = openerp.base.View.extend({
             'model': this.node
         },
         function(result) {
-            self.popup_activity(result)
+            self.add_edit_node(result)
         }
         )
     },
     
-    popup_activity: function(result) {
+    add_edit_node: function(result) {
     	var self = this;
+    	var id;
+    	if(result)
+			id = result.activity_id[0];
     	var action_manager = new openerp.base.ActionManager(this);
     	var dialog = new openerp.base.Dialog(this, {
             title : 'Activity',
@@ -273,7 +271,7 @@ openerp.base_diagram.DiagramView = openerp.base.View.extend({
     	action_manager.appendTo(dialog.$element);
     	action_manager.do_action({
             res_model : this.node,
-            res_id: result.activity_id[0],
+            res_id: id,
             views : [[false, 'form']],
             type : 'ir.actions.act_window',
             auto_search : false,