odoo
/
odoo
2
0
Fork 0
Code Pull Requests Releases Activity

[FIX] issues with asynchronous set_value handling 

* field could be changed from under the closure, use _.each
* There's a bug in Deferred.queue() when passing synchronous deferred
  (including non-deferred) it's instantanously resolved

bzr revid: xmo@openerp.com-20111117160009-s702939kji69v3zf
This commit is contained in:
Xavier Morel 2011-11-17 17:00:09 +01:00
parent bff6674525
commit 6bd9084740
1 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
addons/web/static/src/js/view_form.js
View File

@ -174,13 +174,17 @@ openerp.web.FormView = openerp.web.View.extend( /** @lends openerp.web.FormView#
this.$form_header.find('.oe_form_on_readonly').toggle(this.readonly);
this.$form_header.find('.oe_form_on_editable').toggle(!this.readonly);
this.datarecord = record;
for (var f in this.fields) {
var field = this.fields[f];
_(this.fields).each(function (field, f) {
field.dirty = false;
deferred_stack.push($.when(field.set_value(this.datarecord[f] || false)).then(function() {
var result = field.set_value(self.datarecord[f] || false);
if (result && _.isFunction(result.promise)) {
deferred_stack.push(result);
}
$.when(result).then(function() {
field.validate();
}));
}
});
});
deferred_stack.push('force resolution if no fields');
return deferred_stack.then(function() {
if (!record.id) {