From 6ea547dc7125a5f2942483dbeae0d8ede840ec9b Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Fri, 20 May 2011 15:30:45 +0200
Subject: [PATCH] [ADD] ability to close an opened group, fix disposal of list
 (don't blow up if already disposed of), improve disposal of group body
 elements

bzr revid: xmo@openerp.com-20110520133045-l2gqs8sxso5rqt4k
---
 addons/base/static/src/js/list.js | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/addons/base/static/src/js/list.js b/addons/base/static/src/js/list.js
index 70ab57026da..cc0e4516822 100644
--- a/addons/base/static/src/js/list.js
+++ b/addons/base/static/src/js/list.js
@@ -449,6 +449,7 @@ openerp.base.ListView.List = Class.extend( /** @lends openerp.base.ListView.List
      * Death signal, cleans up list
      */
     apoptosis: function () {
+        if (!this.$current) { return; }
         this.$current.remove();
         this.$current = null;
     }
@@ -531,7 +532,16 @@ openerp.base.ListView.Groups = Class.extend( /** @lends openerp.base.ListView.Gr
         var placeholder = this.make_fragment();
         _(datagroups).each(function (group) {
             var $row = $('<tr>').click(function (e) {
-                self.open_group(e, group);
+                if (!$row.data('open')) {
+                    $row.data('open', true);
+                    self.open_group(e, group);
+                } else {
+                    $row.removeData('open')
+                        .find('span.ui-icon')
+                            .removeClass('ui-icon-triangle-1-s')
+                            .addClass('ui-icon-triangle-1-e');
+                    _(self.children).each(function (child) {child.apoptosis();});
+                }
             });
             placeholder.appendChild($row[0]);
             self.pad($row);
@@ -650,9 +660,7 @@ openerp.base.ListView.Groups = Class.extend( /** @lends openerp.base.ListView.Gr
         _(this.children).each(function (child) {
             child.apoptosis();
         });
-        $(this.elements).each(function (i, body) {
-            $(body).remove();
-        });
+        $(this.elements).remove();
     }
 });
 };