Turkesh Patel (openERP) has proposed merging
lp:~openerp-dev/openerp-web/trunk-bug-872708-tpa into lp:openerp-web.
Requested reviews:
OpenERP R&D Web Team (openerp-dev-web)
Related bugs:
Bug #872708 in OpenERP Web: "delete action without confirmation in list view
is dangerous"
https://bugs.launchpad.net/openerp-web/+bug/872708
For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-872708-tpa/+merge/79230
--
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-872708-tpa/+merge/79230
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openerp-web/trunk-bug-872708-tpa.
=== modified file 'addons/web/static/src/js/view_list.js'
--- addons/web/static/src/js/view_list.js 2011-10-10 14:36:34 +0000
+++ addons/web/static/src/js/view_list.js 2011-10-13 09:19:04 +0000
@@ -463,7 +463,7 @@
* @param {Array} ids the ids of the records to delete
*/
do_delete: function (ids) {
- if (!ids.length) {
+ if (!ids.length || !confirm(" Are you sure to remove those records ?")) {
return;
}
var self = this;
_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help : https://help.launchpad.net/ListHelp
