From 97cb5f4dc417f60e8f2598ea82a1fa3cc5a80383 Mon Sep 17 00:00:00 2001
From: niv-openerp <nicolas.vanhoren@openerp.com>
Date: Tue, 16 Oct 2012 11:58:24 +0200
Subject: [PATCH] [FIX] small problems in dialog actions

bzr revid: nicolas.vanhoren@openerp.com-20121016095824-woasfpauu7unye5n
---
 addons/web/static/src/js/chrome.js | 4 +++-
 addons/web/static/src/js/views.js  | 4 ++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/addons/web/static/src/js/chrome.js b/addons/web/static/src/js/chrome.js
index a281745faa0..54d26c5ec5b 100644
--- a/addons/web/static/src/js/chrome.js
+++ b/addons/web/static/src/js/chrome.js
@@ -70,7 +70,9 @@ instance.web.Dialog = instance.web.Widget.extend({
             autoOpen: false,
             position: [false, 40],
             buttons: {},
-            beforeClose: function () { self.trigger("closing") },
+            beforeClose: function () {
+                self.trigger("closing");
+            },
             resizeStop: this.on_resized
         };
         for (var f in this) {
diff --git a/addons/web/static/src/js/views.js b/addons/web/static/src/js/views.js
index 1554d347552..7cbc346b59f 100644
--- a/addons/web/static/src/js/views.js
+++ b/addons/web/static/src/js/views.js
@@ -290,13 +290,13 @@ instance.web.ActionManager = instance.web.Widget.extend({
         }
         var widget = executor.widget();
         if (executor.action.target === 'new') {
-            if (this.dialog === null) {
+            if (this.dialog === null || this.dialog.isDestroyed()) {
                 // These buttons will be overwrited by <footer> if any
                 this.dialog = new instance.web.Dialog(this, {
                     buttons: { "Close": function() { $(this).dialog("close"); }},
                     dialogClass: executor.klass
                 });
-                if(on_close)
+                if (on_close)
                     this.dialog.on("closing", null, on_close);
             } else {
                 this.dialog_widget.destroy();