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
=== modified file 'addons/web/static/src/js/view_list.js'
--- addons/web/static/src/js/view_list.js 2013-04-24 10:20:25 +0000
+++ addons/web/static/src/js/view_list.js 2013-04-25 06:31:24 +0000
@@ -962,6 +962,9 @@
962 .delegate('td.oe_list_record_delete button', 'click', function (e) {962 .delegate('td.oe_list_record_delete button', 'click', function (e) {
963 e.stopPropagation();963 e.stopPropagation();
964 var $row = $(e.target).closest('tr');964 var $row = $(e.target).closest('tr');
965 if (!confirm(_t("Do you really want to remove these records?"))) {
966 return;
967 }
965 $(self).trigger('deleted', [[self.row_id($row)]]);968 $(self).trigger('deleted', [[self.row_id($row)]]);
966 })969 })
967 .delegate('td.oe_list_field_cell button', 'click', function (e) {970 .delegate('td.oe_list_field_cell button', 'click', function (e) {