[FIX] Bug in form view. When a on_change alters the domain of a field, the view_content_has_changed event is
triggered *before* we alter the domain of the field. So, it is not possible for the fields to really know when they have to re-calculate their domain. bzr revid: nicolas.vanhoren@openerp.com-20131022133432-l0m36mfiwx8z6014
This commit is contained in:
parent
439cc0b94b
commit
55ac10cd3d
|
@ -566,6 +566,13 @@ instance.web.FormView = instance.web.View.extend(instance.web.form.FieldManagerM
|
|||
},
|
||||
on_processed_onchange: function(result, processed) {
|
||||
try {
|
||||
var fields = this.fields;
|
||||
_(result.domain).each(function (domain, fieldname) {
|
||||
var field = fields[fieldname];
|
||||
if (!field) { return; }
|
||||
field.node.attrs.domain = domain;
|
||||
});
|
||||
|
||||
if (result.value) {
|
||||
this._internal_set_values(result.value, processed);
|
||||
}
|
||||
|
@ -579,13 +586,6 @@ instance.web.FormView = instance.web.View.extend(instance.web.form.FieldManagerM
|
|||
});
|
||||
}
|
||||
|
||||
var fields = this.fields;
|
||||
_(result.domain).each(function (domain, fieldname) {
|
||||
var field = fields[fieldname];
|
||||
if (!field) { return; }
|
||||
field.node.attrs.domain = domain;
|
||||
});
|
||||
|
||||
return $.Deferred().resolve();
|
||||
} catch(e) {
|
||||
console.error(e);
|
||||
|
|
Loading…
Reference in New Issue