Merge lp:~openerp-dev/openerp-web/trunk-bug-1172360-ggh into lp:openerp-web

Proposed by Farid Ghanchi
Status: Rejected
Rejected by: Xavier (Open ERP)
Proposed branch: lp:~openerp-dev/openerp-web/trunk-bug-1172360-ggh
Merge into: lp:openerp-web
Diff against target: 13 lines (+3/-0)
1 file modified
addons/web/static/src/js/view_list.js (+3/-0)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/trunk-bug-1172360-ggh
Reviewer Review Type Date Requested Status
OpenERP R&D Web Team Pending
Review via email: mp+160797@code.launchpad.net

Description of the change

Hello,

[Trunk] Fixed the issue for delete record without confirmation in one2many.
When user click on a delete button (trash icon) in a one2many list, I had add confirmation message "Do you really want to remove these records?".

Thanks.
[GGH]

To post a comment you must log in.

Unmerged revisions

3723. By Farid Ghanchi

[FIX] Fixed the issue for delete record without confirmation in list view

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/js/view_list.js'
2--- addons/web/static/src/js/view_list.js 2013-04-24 10:20:25 +0000
3+++ addons/web/static/src/js/view_list.js 2013-04-25 06:31:24 +0000
4@@ -962,6 +962,9 @@
5 .delegate('td.oe_list_record_delete button', 'click', function (e) {
6 e.stopPropagation();
7 var $row = $(e.target).closest('tr');
8+ if (!confirm(_t("Do you really want to remove these records?"))) {
9+ return;
10+ }
11 $(self).trigger('deleted', [[self.row_id($row)]]);
12 })
13 .delegate('td.oe_list_field_cell button', 'click', function (e) {