From beb3aa9fbb43b430c45c066f11f1c406e6c66ecd Mon Sep 17 00:00:00 2001
From: "Kunal Chavda (OpenERP)" <kch@tinyerp.com>
Date: Wed, 9 Nov 2011 11:37:47 +0530
Subject: [PATCH] [IMP]Implement code for selecting multiple value in selection
 box.

bzr revid: kch@tinyerp.com-20111109060747-c34p3r5ablk56w24
---
 addons/web/static/src/js/view_editor.js | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/addons/web/static/src/js/view_editor.js b/addons/web/static/src/js/view_editor.js
index d690057db91..b2b6b07324c 100644
--- a/addons/web/static/src/js/view_editor.js
+++ b/addons/web/static/src/js/view_editor.js
@@ -679,7 +679,7 @@ openerp.web.ViewEditor.FieldSelect = openerp.web.ViewEditor.Field.extend({
         if (!this.dirty) {
             return false;
         }
-        var value = this.$element.find("select[id=" + this.name + "]").find("option:selected").val();
+        var value = this.$element.find("select[id=" + this.name + "]").val();
         return  value ? [this.name, value] : [this.name, ""];
     }
 });
@@ -732,8 +732,18 @@ openerp.web.ViewEditor.PositionProperty = openerp.web.ViewEditor.FieldSelect.ext
 });
 openerp.web.ViewEditor.GroupsProperty = openerp.web.ViewEditor.FieldSelect.extend({
     set_value: function(value, view_val) {
+        var self = this;
         this.$element.find("select[id="+ this.name +"]").attr('multiple', true).css('height','100px');
-        this._super.apply(this,[value, view_val]);
+        var selected_val, sel_val;
+        view_val ? selected_val=view_val[1].split(',') : selected_val = null;
+        _.each(value, function(item) {
+            _.include(selected_val,item[0])?sel_val=true:sel_val=false;
+            self.$element.find("select[id="+self.name+"]").append($("<option/>", {
+                value : item[0],
+                text : item[1],
+                selected:sel_val
+            }));
+         });
     }
 });
 var _PROPERTIES = {