[FIX] Fixed some small issues concerning event triggering.
bzr revid: vta@openerp.com-20121015075320-2zgo32vy5fengm7b
This commit is contained in:
parent
c6dfd8afc1
commit
76fe889331
|
@ -568,7 +568,7 @@ instance.web.DataSet = instance.web.CallbackEnabled.extend({
|
|||
*/
|
||||
write: function (id, data, options) {
|
||||
options = options || {};
|
||||
return this._model.call('write', [[id], data], {context: this._model.context(options.context)});
|
||||
return this._model.call('write', [[id], data], {context: this._model.context(options.context)}).then(this.trigger('dataset_changed', id, data, options));
|
||||
},
|
||||
/**
|
||||
* Deletes an existing record from the database
|
||||
|
@ -576,8 +576,7 @@ instance.web.DataSet = instance.web.CallbackEnabled.extend({
|
|||
* @param {Number|String} ids identifier of the record to delete
|
||||
*/
|
||||
unlink: function(ids) {
|
||||
this.trigger('unlink', ids);
|
||||
return this._model.call('unlink', [ids], {context: this._model.context()});
|
||||
return this._model.call('unlink', [ids], {context: this._model.context()}).then(this.trigger('dataset_changed', ids));
|
||||
},
|
||||
/**
|
||||
* Calls an arbitrary RPC method
|
||||
|
@ -779,6 +778,7 @@ instance.web.DataSetSearch = instance.web.DataSet.extend({
|
|||
self.index = self.index <= self.ids.length - 1 ?
|
||||
self.index : (self.ids.length > 0 ? self.ids.length -1 : 0);
|
||||
}
|
||||
this.trigger("dataset_changed", ids, callback, error_callback);
|
||||
});
|
||||
},
|
||||
size: function () {
|
||||
|
@ -835,7 +835,7 @@ instance.web.BufferedDataSet = instance.web.DataSetStatic.extend({
|
|||
}
|
||||
$.extend(cached.values, record.values);
|
||||
if (dirty)
|
||||
this.trigger("dataset_changed", id);
|
||||
this.trigger("dataset_changed", id, data, options);
|
||||
return $.Deferred().resolve(true).promise();
|
||||
},
|
||||
unlink: function(ids, callback, error_callback) {
|
||||
|
@ -849,7 +849,7 @@ instance.web.BufferedDataSet = instance.web.DataSetStatic.extend({
|
|||
this.to_write = _.reject(this.to_write, function(x) { return _.include(ids, x.id);});
|
||||
this.cache = _.reject(this.cache, function(x) { return _.include(ids, x.id);});
|
||||
this.set_ids(_.without.apply(_, [this.ids].concat(ids)));
|
||||
this.trigger("dataset_changed", ids);
|
||||
this.trigger("dataset_changed", ids, callback, error_callback);
|
||||
return $.async_when({result: true}).then(callback);
|
||||
},
|
||||
reset_ids: function(ids) {
|
||||
|
|
|
@ -749,7 +749,7 @@ instance.web.FormView = instance.web.View.extend(instance.web.form.FieldManagerM
|
|||
on_button_save: function() {
|
||||
var self = this;
|
||||
return this.save().then(function(result) {
|
||||
self.trigger("save");
|
||||
self.trigger("save", result);
|
||||
self.to_view_mode();
|
||||
});
|
||||
},
|
||||
|
|
Loading…
Reference in New Issue