openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #06162
[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