From dd12cbe1b6b284a00ec78efe54f25bc44cef2345 Mon Sep 17 00:00:00 2001
From: Tejas Tank <tta@openerp.com>
Date: Mon, 5 Nov 2012 15:16:02 +0530
Subject: [PATCH 1/5] [IMP] Selection field with placeholder feature.

bzr revid: tta@openerp.com-20121105094602-v2a4z4m4mighxgqw
---
 addons/web/static/src/xml/base.xml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/addons/web/static/src/xml/base.xml b/addons/web/static/src/xml/base.xml
index 275d31a9fde..d4b8674430e 100644
--- a/addons/web/static/src/xml/base.xml
+++ b/addons/web/static/src/xml/base.xml
@@ -993,6 +993,7 @@
     <span class="oe_form_field oe_datepicker_root oe_form_field_datetime" t-att-style="widget.node.attrs.style"/>
 </t>
 <t t-name="FieldSelection">
+    <t t-set="placeholder" t-value="! widget.get('effective_readonly') ? widget.node.attrs.placeholder : ''"/>
     <span class="oe_form_field oe_form_field_selection" t-att-style="widget.node.attrs.style">
         <select t-if="!widget.get('effective_readonly')"
             t-att-name="widget.name"
@@ -1000,7 +1001,7 @@
             t-att-autofocus="widget.node.attrs.autofocus"
             t-att-id="widget.id_for_label">
                 <t t-foreach="widget.values" t-as="option">
-                    <option><t t-esc="option[1]"/></option>
+                    <option><t t-esc="placeholder" t-if="option[0] == false"/><t t-esc="option[1]" t-if="option[0] != false"/></option>
                 </t>
         </select>
     </span>

From e4effd65856a4f336753103ea05127f7066b97f2 Mon Sep 17 00:00:00 2001
From: Tejas Tank <tta@openerp.com>
Date: Wed, 7 Nov 2012 12:08:10 +0530
Subject: [PATCH 2/5] [IMP] Remove traceback with a warning/constraint message
 for float field.

bzr revid: tta@openerp.com-20121107063810-ulei1bq8aw1xctyl
---
 addons/web/static/src/js/view_form.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js
index 07c237ae0d8..90335c269df 100644
--- a/addons/web/static/src/js/view_form.js
+++ b/addons/web/static/src/js/view_form.js
@@ -2179,7 +2179,9 @@ instance.web.form.FieldChar = instance.web.form.AbstractField.extend(instance.we
         var self = this;
         var $input = this.$el.find('input');
         $input.change(function() {
-            self.internal_set_value(self.parse_value($input.val()));
+            if(self.is_syntax_valid()){
+                self.internal_set_value(self.parse_value($input.val()));
+            }
         });
         this.setupFocus($input);
     },

From 909285bb7d4bdad5ea94665f3e413497cd853c13 Mon Sep 17 00:00:00 2001
From: Tejas Tank <tta@openerp.com>
Date: Wed, 7 Nov 2012 12:16:23 +0530
Subject: [PATCH 3/5] [IMP] Listview.buttons template corrected for import
 button and future support to append buttons inside 'oe_list_buttons' section.

bzr revid: tta@openerp.com-20121107064623-q9oggrypyil1207w
---
 addons/web/static/src/xml/base.xml | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/addons/web/static/src/xml/base.xml b/addons/web/static/src/xml/base.xml
index d4b8674430e..99af729d04f 100644
--- a/addons/web/static/src/xml/base.xml
+++ b/addons/web/static/src/xml/base.xml
@@ -648,13 +648,15 @@
         </tr>
     </tfoot>
 </table>
-<div t-name="ListView.buttons" class="oe_list_buttons">
+<t t-name="ListView.buttons">
+    <div class="oe_list_buttons">
     <t t-if="!widget.no_leaf and widget.options.action_buttons !== false and widget.options.addable and widget.is_action_enabled('create')">
         <button type="button" class="oe_button oe_list_add oe_highlight">
             <t t-esc="widget.options.addable"/>
         </button>
     </t>
-</div>
+    </div>
+</t>
 <t t-name="ListView.pager">
     <div class="oe_list_pager" t-att-colspan="widget.columns_count">
         <t t-if="!widget.no_leaf and widget.options.pager !== false" t-call="ViewPager">

From 1a9f93fc85ea9bce1265b14f53cafd3eb1e5f99e Mon Sep 17 00:00:00 2001
From: Tejas Tank <tta@openerp.com>
Date: Wed, 7 Nov 2012 17:02:56 +0530
Subject: [PATCH 4/5] [fix] on model form tags issue fixed..

bzr revid: tta@openerp.com-20121107113256-ac6znf9m5ibzoovl
---
 addons/web/static/src/js/view_form.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js
index 69feb2757d0..23101465849 100644
--- a/addons/web/static/src/js/view_form.js
+++ b/addons/web/static/src/js/view_form.js
@@ -3920,6 +3920,8 @@ instance.web.form.FieldMany2ManyTags = instance.web.form.AbstractField.extend(in
         return this.get("value");
     },
     render_value: function() {
+        if (this.get("effective_readonly"))
+            return;
         var self = this;
         var dataset = new instance.web.DataSetStatic(this, this.field.relation, self.build_context());
         var values = self.get("value")

From 81a2e23dd60800386708b9b4986f5ad2cf9d29af Mon Sep 17 00:00:00 2001
From: Tejas Tank <tta@openerp.com>
Date: Wed, 28 Nov 2012 11:05:40 +0530
Subject: [PATCH 5/5] [FIX] clean the code.

bzr revid: tta@openerp.com-20121128053540-01z3yiooq83foyew
---
 addons/web/static/src/js/view_form.js | 2 --
 1 file changed, 2 deletions(-)

diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js
index 6dd4fc10937..ed622945d7c 100644
--- a/addons/web/static/src/js/view_form.js
+++ b/addons/web/static/src/js/view_form.js
@@ -4036,8 +4036,6 @@ instance.web.form.FieldMany2ManyTags = instance.web.form.AbstractField.extend(in
         return this.get("value");
     },
     render_value: function() {
-        if (this.get("effective_readonly"))
-            return;
         var self = this;
         var dataset = new instance.web.DataSetStatic(this, this.field.relation, self.build_context());
         var values = self.get("value");