From fc62b07bbb749cf67b717bca953575c0c17c0b00 Mon Sep 17 00:00:00 2001
From: "Adrien Peiffer (ACSONE)" <adrien.peiffer@acsone.eu>
Date: Tue, 17 May 2016 08:27:40 +0200
Subject: [PATCH] [IMP] mis_builder: Cleanup JS widget.

---
 mis_builder/static/src/js/mis_builder.js | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/mis_builder/static/src/js/mis_builder.js b/mis_builder/static/src/js/mis_builder.js
index 8e932168..5fc85fe8 100644
--- a/mis_builder/static/src/js/mis_builder.js
+++ b/mis_builder/static/src/js/mis_builder.js
@@ -26,8 +26,6 @@ var MisReport = form_common.FormWidget.extend({
         self.mis_report_instance_id = self.getParent().datarecord.id
         if (self.mis_report_instance_id) {
             self.generate_content();
-        } else {
-            self.display_settings();
         }
     },
 
@@ -37,8 +35,6 @@ var MisReport = form_common.FormWidget.extend({
         self.mis_report_instance_id = self.getParent().datarecord.id
         if (self.mis_report_instance_id) {
             self.getParent().dataset.context['no_destroy'] = true;
-        } else {
-            self.display_settings();
         }
     },
     
@@ -130,6 +126,11 @@ var MisReport = form_common.FormWidget.extend({
 });
 
 ActionManager.include({
+    /*
+     * In the case where we would be open in modal view, this is
+     * necessary to avoid to close the popup on click on button like print,
+     * export, ...
+     */
     dialog_stop: function (reason) {
         var self = this
         if (self.dialog_widget && self.dialog_widget.dataset && self.dialog_widget.dataset.context) {