From 58584c82a7a5e7132f236efeef6ceede6b0f74a4 Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Thu, 24 Mar 2011 19:22:22 +0100
Subject: [PATCH] [IMP] move group toggling from template code to JS code

bzr revid: xmo@openerp.com-20110324182222-qaymq120818jx10n
---
 addons/base/static/openerp/base.xml         | 4 +++-
 addons/base/static/openerp/js/base_views.js | 6 ++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/addons/base/static/openerp/base.xml b/addons/base/static/openerp/base.xml
index 3d3a3fe856b..f7f6d076b3a 100644
--- a/addons/base/static/openerp/base.xml
+++ b/addons/base/static/openerp/base.xml
@@ -323,7 +323,9 @@
 <t t-name="SearchView.group">
     <div t-att-class="'searchview_group ' + (attrs.expand == '0' ? 'folded' : 'expanded')"
          t-att-id="element_id">
-        <a class="searchview_group_string" href="#" onclick="$(this).parent().toggleClass('folded expanded')"><t t-esc="attrs.string"/></a>
+        <a t-if="attrs.string" class="searchview_group_string" href="#">
+            <t t-esc="attrs.string"/>
+        </a>
         <div class="searchview_group_content">
             <t t-call="SearchView.render_lines"/>
         </div>
diff --git a/addons/base/static/openerp/js/base_views.js b/addons/base/static/openerp/js/base_views.js
index b4c0cfd5019..1d60e10f51e 100644
--- a/addons/base/static/openerp/js/base_views.js
+++ b/addons/base/static/openerp/js/base_views.js
@@ -597,6 +597,12 @@ openerp.base.search.Group = openerp.base.search.Widget.extend({
             .chain()
             .flatten()
             .each(function (widget) { widget.start(); });
+        var $root = this.$element;
+        $root.find('a.searchview_group_string').click(function (e) {
+            $root.toggleClass('folded expanded');
+            e.stopPropagation();
+            e.preventDefault();
+        });
     }
 });
 openerp.base.search.Input = openerp.base.search.Widget.extend({