Merge lp:~ruchir.shukla/openerp-web/7.0-opw-596129-fix-destroy into lp:~openerp-dev/openerp-web/7.0-opw-596129-msh

Proposed by Ruchir Shukla(BizzAppDev)
Status: Needs review
Proposed branch: lp:~ruchir.shukla/openerp-web/7.0-opw-596129-fix-destroy
Merge into: lp:~openerp-dev/openerp-web/7.0-opw-596129-msh
Diff against target: 13 lines (+3/-0)
1 file modified
addons/web/static/src/js/view_form.js (+3/-0)
To merge this branch: bzr merge lp:~ruchir.shukla/openerp-web/7.0-opw-596129-fix-destroy
Reviewer Review Type Date Requested Status
OpenERP R&D Team Pending
Review via email: mp+217328@code.launchpad.net

Description of the change

Fix for the multiple popup weird behavior

To post a comment you must log in.

Unmerged revisions

4026. By Ruchir Shukla(BizzAppDev)

[FIX] form.destroy() doesn\'t clean up buttons area

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js 2013-08-21 08:53:42 +0000
+++ addons/web/static/src/js/view_form.js 2014-04-26 05:58:14 +0000
@@ -147,6 +147,9 @@
147 this.$el.off('.formBlur');147 this.$el.off('.formBlur');
148 }148 }
149 this._super();149 this._super();
150 if (this.$buttons) {
151 this.$buttons.remove();
152 }
150 },153 },
151 load_form: function(data) {154 load_form: function(data) {
152 var self = this;155 var self = this;

Subscribers

People subscribed via source and target branches