From 5a09e97754228fa8c2f9ed337c6f7951a962d7d5 Mon Sep 17 00:00:00 2001
From: "Yogesh (OpenERP)" <ysa@tinyerp.com>
Date: Mon, 18 Jul 2011 11:54:59 +0530
Subject: [PATCH] [IMP] imporve display export button in list and form view.

bzr revid: ysa@tinyerp.com-20110718062459-2kf2z0zmgoes4ied
---
 addons/base/static/src/js/views.js | 24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)

diff --git a/addons/base/static/src/js/views.js b/addons/base/static/src/js/views.js
index 5efe27c7e21..07f1ce406f1 100644
--- a/addons/base/static/src/js/views.js
+++ b/addons/base/static/src/js/views.js
@@ -157,16 +157,6 @@ openerp.base.ViewManager =  openerp.base.Controller.extend({
                 self.on_controller_inited(view_type, controller);
             });
             this.views[view_type].controller = controller;
-            if(this.flags && this.flags.sidebar) {
-                if(this.active_view == 'list' || this.active_view == 'form') {
-                    this.views[this.active_view].controller.$element.after(QWeb.render('ExportView'))
-                    this.$element.find('#exportview').click(function(ev) {
-                        var export_view = new openerp.base_export.Export(self.session, self.dataset, self.views);
-                        export_view.start(false);
-                        ev.preventDefault();
-                    });
-                }
-            }
         }
 
 
@@ -193,6 +183,20 @@ openerp.base.ViewManager =  openerp.base.Controller.extend({
                 }
             }
         }
+        if(this.flags && this.flags.sidebar) {
+            if(this.$element.find('#exportview')){
+                this.$element.find('#exportview').remove()
+            }
+            if(this.active_view == 'list' || this.active_view == 'form') {
+                console.log("this.views[this.active_view].controller.$element::",this.views[this.active_view].controller.$element)
+                this.views[this.active_view].controller.$element.after(QWeb.render('ExportView'))
+                this.$element.find('#exportview').click(function(ev) {
+                    var export_view = new openerp.base_export.Export(self.session, self.dataset, self.views);
+                    export_view.start(false);
+                    ev.preventDefault();
+                });
+            }
+        }
         return view_promise;
     },
     /**