Merge lp:~openerp-dev/openerp-web/trunk-bug-872708-tpa into lp:openerp-web

Proposed by Turkesh Patel (openERP)
Status: Merged
Merge reported by: Xavier (Open ERP)
Merged at revision: not available
Proposed branch: lp:~openerp-dev/openerp-web/trunk-bug-872708-tpa
Merge into: lp:openerp-web
Diff against target: 12 lines (+1/-1)
1 file modified
addons/web/static/src/js/view_list.js (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/trunk-bug-872708-tpa
Reviewer Review Type Date Requested Status
OpenERP R&D Web Team Pending
Review via email: mp+79230@code.launchpad.net
To post a comment you must log in.

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 2011-10-10 14:36:34 +0000
3+++ addons/web/static/src/js/view_list.js 2011-10-13 09:19:04 +0000
4@@ -463,7 +463,7 @@
5 * @param {Array} ids the ids of the records to delete
6 */
7 do_delete: function (ids) {
8- if (!ids.length) {
9+ if (!ids.length || !confirm(" Are you sure to remove those records ?")) {
10 return;
11 }
12 var self = this;