From 2ce07c97c615c405d2171244dd47f8b52edf8c73 Mon Sep 17 00:00:00 2001
From: kbh <kbh@bde-desktop>
Date: Mon, 24 Oct 2011 14:47:30 +0530
Subject: [PATCH 1/4] [FIX] Fixed the issue of checkbutton select all/none in
 tree view.

lp bug: https://launchpad.net/bugs/878108 fixed

bzr revid: kbh@bde-desktop-20111024091730-dkzafro2f9e0k7tf
---
 addons/web/static/src/js/view_list.js | 8 +++++++-
 addons/web/static/src/xml/base.xml    | 3 ++-
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/addons/web/static/src/js/view_list.js b/addons/web/static/src/js/view_list.js
index d09ec1d3035..6fd4cdf2e03 100644
--- a/addons/web/static/src/js/view_list.js
+++ b/addons/web/static/src/js/view_list.js
@@ -201,8 +201,8 @@ openerp.web.ListView = openerp.web.View.extend( /** @lends openerp.web.ListView#
         this.setup_columns(this.fields_view.fields, grouped);
 
         this.$element.html(QWeb.render("ListView", this));
-
         // Head hook
+        this.$element.find('#all_checked').click(this.do_all_checked);
         this.$element.find('.oe-list-add')
                 .click(this.do_add_record)
                 .attr('disabled', grouped && this.options.editable);
@@ -269,6 +269,11 @@ openerp.web.ListView = openerp.web.View.extend( /** @lends openerp.web.ListView#
             this.set_common_sidebar_sections(this.sidebar);
         }
     },
+
+    do_all_checked:function (ev) {
+        var all_checkbox = this.$element.find('.oe-record-selector :checkbox')
+        all_checkbox.attr('checked', !all_checkbox.attr('checked'));
+    },
     /**
      * Configures the ListView pager based on the provided dataset's information
      *
@@ -650,6 +655,7 @@ openerp.web.ListView = openerp.web.View.extend( /** @lends openerp.web.ListView#
             $first_header.attr('colspan', parseInt(colspan, 10) + count);
         }
         // Padding for column titles, footer and data rows
+
         var $rows = this.$element
                 .find('.oe-listview-header-columns, tr:not(thead tr)')
                 .not(options['except']);
diff --git a/addons/web/static/src/xml/base.xml b/addons/web/static/src/xml/base.xml
index 7720ae2d38b..4c21a565a69 100644
--- a/addons/web/static/src/xml/base.xml
+++ b/addons/web/static/src/xml/base.xml
@@ -593,7 +593,8 @@
                     <t t-esc="column.string"/>
                 </th>
             </t>
-            <th t-if="options.selectable" width="1"/>
+            <th t-if="options.selectable" width="1"  >
+                <input type="checkbox" id ="all_checked"/> </th>
             <t t-foreach="columns" t-as="column">
                 <th t-if="!column.meta and column.invisible !== '1'" t-att-data-id="column.id"
                     t-att-class="((options.sortable and column.tag !== 'button') ? 'oe-sortable' : null)">

From 3e20652e55ce8f8eb53d3dc1edf9321a63d5c888 Mon Sep 17 00:00:00 2001
From: kbh <kbh@bde-desktop>
Date: Mon, 24 Oct 2011 18:59:59 +0530
Subject: [PATCH 2/4] [IMP] Fixed the issue of checkbutton select all/none in
 tree view.

bzr revid: kbh@bde-desktop-20111024132959-y9b3scfye91nut1c
---
 addons/web/static/src/js/view_list.js | 5 +++++
 addons/web/static/src/xml/base.xml    | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/addons/web/static/src/js/view_list.js b/addons/web/static/src/js/view_list.js
index 6fd4cdf2e03..1f8430df8a2 100644
--- a/addons/web/static/src/js/view_list.js
+++ b/addons/web/static/src/js/view_list.js
@@ -202,6 +202,11 @@ openerp.web.ListView = openerp.web.View.extend( /** @lends openerp.web.ListView#
 
         this.$element.html(QWeb.render("ListView", this));
         // Head hook
+        this.$element.find('.all-record-selector').click(function(){
+            self.$element.find(':checkbox').prop('checked', 
+                self.$element.find('.all-record-selector').prop('checked')  || false);
+        });
+
         this.$element.find('#all_checked').click(this.do_all_checked);
         this.$element.find('.oe-list-add')
                 .click(this.do_add_record)
diff --git a/addons/web/static/src/xml/base.xml b/addons/web/static/src/xml/base.xml
index 4c21a565a69..81611f0315a 100644
--- a/addons/web/static/src/xml/base.xml
+++ b/addons/web/static/src/xml/base.xml
@@ -594,7 +594,7 @@
                 </th>
             </t>
             <th t-if="options.selectable" width="1"  >
-                <input type="checkbox" id ="all_checked"/> </th>
+                <input type="checkbox" class="all-record-selector"/> </th>
             <t t-foreach="columns" t-as="column">
                 <th t-if="!column.meta and column.invisible !== '1'" t-att-data-id="column.id"
                     t-att-class="((options.sortable and column.tag !== 'button') ? 'oe-sortable' : null)">

From 276cff1807bfda7851845d977be058b388b1d67b Mon Sep 17 00:00:00 2001
From: kbh <kbh@bde-desktop>
Date: Tue, 25 Oct 2011 11:31:42 +0530
Subject: [PATCH 3/4] [IMP] Remove unnecessary code in checkbutton select
 all/none in list view.

bzr revid: kbh@bde-desktop-20111025060142-47y025oqteocdgj3
---
 addons/web/static/src/js/view_list.js | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/addons/web/static/src/js/view_list.js b/addons/web/static/src/js/view_list.js
index 1f8430df8a2..6a08b2c3160 100644
--- a/addons/web/static/src/js/view_list.js
+++ b/addons/web/static/src/js/view_list.js
@@ -275,10 +275,6 @@ openerp.web.ListView = openerp.web.View.extend( /** @lends openerp.web.ListView#
         }
     },
 
-    do_all_checked:function (ev) {
-        var all_checkbox = this.$element.find('.oe-record-selector :checkbox')
-        all_checkbox.attr('checked', !all_checkbox.attr('checked'));
-    },
     /**
      * Configures the ListView pager based on the provided dataset's information
      *

From fa94cf757893db64b840cfb011c69a76aacb794a Mon Sep 17 00:00:00 2001
From: kbh <kbh@bde-desktop>
Date: Tue, 25 Oct 2011 11:54:29 +0530
Subject: [PATCH 4/4] [IMP] Remove unnecessary code in checkbutton select
 all/none in list view.

bzr revid: kbh@bde-desktop-20111025062429-dvx4x0h1caqwg21i
---
 addons/web/static/src/js/view_list.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/addons/web/static/src/js/view_list.js b/addons/web/static/src/js/view_list.js
index 6a08b2c3160..29decd946f6 100644
--- a/addons/web/static/src/js/view_list.js
+++ b/addons/web/static/src/js/view_list.js
@@ -207,7 +207,6 @@ openerp.web.ListView = openerp.web.View.extend( /** @lends openerp.web.ListView#
                 self.$element.find('.all-record-selector').prop('checked')  || false);
         });
 
-        this.$element.find('#all_checked').click(this.do_all_checked);
         this.$element.find('.oe-list-add')
                 .click(this.do_add_record)
                 .attr('disabled', grouped && this.options.editable);