← Back to team overview

openerp-community-reviewer team mailing list archive

[Merge] lp:~therp-nl/ocb-web/7.0-form_destroy_buttons into lp:ocb-web

 

Ruchir Shukla(BizzAppDev) has proposed merging lp:~therp-nl/ocb-web/7.0-form_destroy_buttons into lp:ocb-web.

Requested reviews:
  OpenERP Community Backports Team (ocb)
Related bugs:
  Bug #1311076 in OpenERP Community Backports (Web): "multiple popup weird behavior"
  https://bugs.launchpad.net/ocb-web/+bug/1311076

For more details, see:
https://code.launchpad.net/~therp-nl/ocb-web/7.0-form_destroy_buttons/+merge/216898
-- 
https://code.launchpad.net/~therp-nl/ocb-web/7.0-form_destroy_buttons/+merge/216898
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~therp-nl/ocb-web/7.0-form_destroy_buttons into lp:ocb-web.
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2014-04-18 09:20:33 +0000
+++ addons/web/static/src/js/view_form.js	2014-04-23 14:35:10 +0000
@@ -147,6 +147,9 @@
             this.$el.off('.formBlur');
         }
         this._super();
+        if (this.$buttons) {
+            this.$buttons.remove();
+        }
     },
     load_form: function(data) {
         var self = this;


Follow ups