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

[FIX] can't perform action on a o2m record which is not saved in db, so forbid it. 

Also yet-another-shitty-undoc-change to the way widgets and form view communicate

bzr revid: xmo@openerp.com-20120711141402-1x7frd0v3vc3649e
This commit is contained in:
Xavier Morel 2012-07-11 16:14:02 +02:00
parent 9b570343a5
commit 6b907bb4d0
1 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -3196,6 +3196,11 @@ instance.web.form.One2ManyListView = instance.web.ListView.extend({
}));
this.on('edit:before', this, this.proxy('_beforeEdit'));
this.on('save:before cancel:before', this, this.proxy('_beforeUnEdit'));
this.records
.bind('add', this.proxy("changedRecords"))
.bind('edit', this.proxy("changedRecords"))
.bind('remove', this.proxy("changedRecords"));
},
start: function () {
var ret = this._super();
@ -3204,6 +3209,9 @@ instance.web.form.One2ManyListView = instance.web.ListView.extend({
.on('mousedown.handleButtons', 'table button', this.proxy('_buttonDown'));
return ret;
},
changedRecords: function () {
this.o2m.trigger_on_change();
},
is_valid: function () {
var form = this.editor.form;
@ -3276,13 +3284,19 @@ instance.web.form.One2ManyListView = instance.web.ListView.extend({
readonly: self.o2m.get("effective_readonly")
});
},
do_button_action: function () {
do_button_action: function (name, id, callback) {
if (!_.isNumber(id)) {
instance.webclient.notification.warn(
_t("Action Button"),
_t("The o2m record must be saved before an action can be used"));
return;
}
var parent_form = this.o2m.view;
var self = this, args = arguments;
return this.ensureSaved().pipe(function () {
var self = this;
this.ensureSaved().pipe(function () {
return parent_form.do_save();
}).then(function () {
self.handleButton.apply(self, args);
self.handleButton(name, id, callback);
});
},