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

[IMP] move DOM event handling for action buttons in lists to ListView.List 

bzr revid: xmo@openerp.com-20110509081022-owlj38aqcdhnlr90
This commit is contained in:
Xavier Morel 2011-05-09 10:10:22 +02:00
parent 27c57ef671
commit b940e26c36
2 changed files with 26 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

49
addons/base/static/src/js/list.js
View File

@ -68,6 +68,16 @@ openerp.base.ListView = openerp.base.Controller.extend(
},
'deleted': function (e, ids) {
self.do_delete(ids);
},
'action': function (e, action_name, id) {
var action = _.detect(self.columns, function (field) {
return field.name === action_name;
});
if (!action) { return; }
// TODO: not supposed to reload everything, I think
self.execute_action(
action, self.dataset, self.session.action_manager,
id, self.do_reload);
}
});
@ -163,24 +173,6 @@ openerp.base.ListView = openerp.base.Controller.extend(
var $table = this.$element.find('table');
this.list.move_to($table);
// Cell events
$table.delegate(
'td.oe-field-cell button', 'click', function (e) {
e.stopImmediatePropagation();
var $cell = $(e.currentTarget).closest('td');
var col_index = $cell.prevAll('td').length;
var field = self.visible_columns[col_index];
var $row = $cell.parent('tr');
var row = self.rows[$row.prevAll().length];
// TODO: we should probably only reload content, also maybe diff records or something, instead of replacing every single row
self.execute_action(
field, self.dataset, self.session.action_manager,
row.data.id.value, self.do_reload);
});
// Global rows handlers
$table.delegate('tr', 'click', this.on_select_row);
@ -397,10 +389,14 @@ openerp.base.ListView.List = Class.extend(
* `selected`
* Triggered when a row is selected (using check boxes), provides an
* array of ids of all the selected records.
*
* `deleted`
* Triggered when deletion buttons are hit, provide an array of ids of
* all the records being marked for suppression.
* `action`
* Triggered when an action button is clicked, provides two parameters:
*
* * The name of the action to execute (as a string)
* * The id of the record to execute the action on
*
* @constructs
* @param {Object} opts display options, identical to those of :js:class:`openerp.base.ListView`
@ -421,11 +417,16 @@ openerp.base.ListView.List = Class.extend(
})
.delegate('td.oe-record-delete button', 'click', function (e) {
e.stopPropagation();
var $row = $(e.target).closest('tr'),
event = $.Event('deleted');
$(self).trigger(
event,
[[self.row_id($row)]]);
var $row = $(e.target).closest('tr');
$(self).trigger('deleted', [[self.row_id($row)]]);
})
.delegate('td.oe-field-cell button', 'click', function (e) {
e.stopPropagation();
var $target = $(e.currentTarget),
field = $target.closest('td').data('field'),
record_id = self.row_id($target.closest('tr'));
$(self).trigger('action', [field, record_id]);
});
},
move_to: function (element) {

2
addons/base/static/src/xml/base.xml
View File

@ -208,7 +208,7 @@
<t t-foreach="columns" t-as="column">
<t t-set="attrs" t-value="column.attrs_for(row.data)"/>
<td t-if="column.invisible !== '1'" t-att-title="column.help"
class="oe-field-cell">
class="oe-field-cell" t-att-data-field="column.id">
<t t-if="!attrs.invisible">
<t t-set="is_button" t-value="column.tag === 'button'"/>
<!-- TODO: get correct widget from form -->