From 882e4e71897d3b692db70e6e7acc92a578cdd3be Mon Sep 17 00:00:00 2001 From: Anand Doshi Date: Tue, 23 Jul 2013 19:49:30 +0530 Subject: [PATCH] [minor] [fix] [listing] Select All based checking using prop function of jquery 1.9.1, instead of attr as per http://jquery.com/upgrade-guide/1.9/#attr-versus-prop- --- public/js/wn/views/doclistview.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/public/js/wn/views/doclistview.js b/public/js/wn/views/doclistview.js index 861e538165..009a5eefcf 100644 --- a/public/js/wn/views/doclistview.js +++ b/public/js/wn/views/doclistview.js @@ -233,8 +233,8 @@ wn.views.DocListView = wn.ui.Listing.extend({ if(this.can_delete || this.listview.settings.selectable) { this.add_button(wn._('Delete'), function() { me.delete_items(); }, 'icon-remove'); this.add_button(wn._('Select All'), function() { - var checks = me.$page.find('.list-delete'); - checks.attr('checked', $(checks.get(0)).attr('checked') ? false : "checked"); + me.$page.find('.list-delete').prop("checked", + me.$page.find('.list-delete:checked').length ? false : true); }, 'icon-ok'); } if(in_list(user_roles, "System Manager")) {