diff options
| author | Xavier Morel <xmo@openerp.com> | 2011-10-04 17:46:50 +0200 |
|---|---|---|
| committer | Xavier Morel <xmo@openerp.com> | 2011-10-04 17:46:50 +0200 |
| commit | 74db7dd027ba87e4468875634d7fb6e7fbdf7c30 (patch) | |
| tree | 2e53d00b88d6d9d7f6a86957d77efba1714b5fc5 /addons/web/static/src/js/view_list_editable.js | |
| parent | 72b852c2c669b92297d75b8adac6a42e3a1ecf6d (diff) | |
[FIX] correctly handle insertion in empty editable list
bzr revid: xmo@openerp.com-20111004154650-hx56g1lmefhzgf21
Diffstat (limited to 'addons/web/static/src/js/view_list_editable.js')
| -rw-r--r-- | addons/web/static/src/js/view_list_editable.js | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/addons/web/static/src/js/view_list_editable.js b/addons/web/static/src/js/view_list_editable.js index 7daf2bd0685..41133c102d0 100644 --- a/addons/web/static/src/js/view_list_editable.js +++ b/addons/web/static/src/js/view_list_editable.js @@ -175,12 +175,16 @@ openerp.web.list_editable = function (openerp) { if (row) { $new_row.replaceAll(row); } else if (self.options.editable) { - if (self.options.editable === 'top') { - $new_row.insertBefore( - self.$current.find('> [data-id]:first')); + if (self.records.length) { + if (self.options.editable === 'top') { + $new_row.insertBefore( + self.$current.find('> [data-id]:first')); + } else { + $new_row.insertAfter( + self.$current.find('> [data-id]:last')); + } } else { - $new_row.insertAfter( - self.$current.find('> [data-id]:last')); + $new_row.prependTo(self.$current); } var $last_child = self.$current.find('> tr:last'); if ($last_child.is(':not([data-id])')) { |