From 80fd4e4bb631f41ec765826499d3f4b12229eeb9 Mon Sep 17 00:00:00 2001
From: Pankaj Joshi <pji@openerp.com>
Date: Fri, 14 Nov 2014 12:52:46 +0530
Subject: [PATCH] [FIX] web: radio buttons

removed js that let user to uncheck radiobutton and from addons removed radiobuttons used for not required field except sync_typ.
changed this field as required because of functional requirement
---
 addons/account/res_config_view.xml                              | 2 +-
 addons/account/wizard/account_report_partner_balance_view.xml   | 2 +-
 addons/base_gengo/wizard/base_gengo_translations.py             | 2 +-
 addons/calendar/calendar_view.xml                               | 2 +-
 .../crm_partner_assign/wizard/crm_forward_to_partner_view.xml   | 2 +-
 addons/sale_stock/res_config_view.xml                           | 2 +-
 addons/web/static/src/js/view_form.js                           | 1 -
 7 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/addons/account/res_config_view.xml b/addons/account/res_config_view.xml
index 435996c0039..2a7466c1244 100644
--- a/addons/account/res_config_view.xml
+++ b/addons/account/res_config_view.xml
@@ -115,7 +115,7 @@
                                 <field name="decimal_precision" class="oe_inline"/>
                             </div>
                             <div>
-                                <field name="tax_calculation_rounding_method" widget="radio" class="oe_inline" />
+                                <field name="tax_calculation_rounding_method" class="oe_inline" />
                             </div>
                         </div>
                         <label for="id" string="Features"/>
diff --git a/addons/account/wizard/account_report_partner_balance_view.xml b/addons/account/wizard/account_report_partner_balance_view.xml
index daa07494166..6df8dd474c6 100644
--- a/addons/account/wizard/account_report_partner_balance_view.xml
+++ b/addons/account/wizard/account_report_partner_balance_view.xml
@@ -10,7 +10,7 @@
             <data>
             <xpath expr="//field[@name='target_move']" position="after">
                 <field name="result_selection"/>
-                <field name="display_partner" widget="radio"/>
+                <field name="display_partner"/>
                 <newline/>
             </xpath>
             </data>
diff --git a/addons/base_gengo/wizard/base_gengo_translations.py b/addons/base_gengo/wizard/base_gengo_translations.py
index 1521ac3dce5..641bab02681 100644
--- a/addons/base_gengo/wizard/base_gengo_translations.py
+++ b/addons/base_gengo/wizard/base_gengo_translations.py
@@ -46,7 +46,7 @@ class base_gengo_translations(osv.osv_memory):
     _columns = {
         'sync_type': fields.selection([('send', 'Send New Terms'),
                                        ('receive', 'Receive Translation'),
-                                       ('both', 'Both')], "Sync Type"),
+                                       ('both', 'Both')], "Sync Type", required=True),
         'lang_id': fields.many2one('res.lang', 'Language', required=True),
         'sync_limit': fields.integer("No. of terms to sync"),
     }
diff --git a/addons/calendar/calendar_view.xml b/addons/calendar/calendar_view.xml
index 4bdfa5e0762..3dde4743cde 100644
--- a/addons/calendar/calendar_view.xml
+++ b/addons/calendar/calendar_view.xml
@@ -127,7 +127,7 @@
                                     </group>
                                 </group>
                                 <group>
-                                    <field name="class" widget="radio"/>
+                                    <field name="class"/>
                                     <field name="show_as"/>
                                     <field name="rrule" invisible="1" readonly="0" />                                    
                                     <field name="recurrent_id" invisible="1" />
diff --git a/addons/crm_partner_assign/wizard/crm_forward_to_partner_view.xml b/addons/crm_partner_assign/wizard/crm_forward_to_partner_view.xml
index 676954f317d..31da10393ea 100644
--- a/addons/crm_partner_assign/wizard/crm_forward_to_partner_view.xml
+++ b/addons/crm_partner_assign/wizard/crm_forward_to_partner_view.xml
@@ -7,7 +7,7 @@
             <field name="arch" type="xml">
                 <form string="Send Mail">
                     <group>
-                        <field name="forward_type" widget="radio" invisible="context.get('hide_forward_type',False)"/>
+                        <field name="forward_type" invisible="context.get('hide_forward_type',False)"/>
                     </group>
                     <group>
                         <group>
diff --git a/addons/sale_stock/res_config_view.xml b/addons/sale_stock/res_config_view.xml
index 4f20472d7a4..8d90fcda179 100644
--- a/addons/sale_stock/res_config_view.xml
+++ b/addons/sale_stock/res_config_view.xml
@@ -19,7 +19,7 @@
                      </xpath>
                      <xpath expr="//div[@name='div_default_options']" position="inside">
                          <div attrs="{'invisible':[('group_invoice_deli_orders','=',False)],'required': [('group_invoice_deli_orders','=',True)]}">
-                             <field name="default_order_policy" class="oe_inline" widget="radio"/>
+                             <field name="default_order_policy" class="oe_inline"/>
                          </div>
                          <div>
                              <field name="module_delivery" class="oe_inline"/>
diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js
index 8d7f8869ae2..c593a20da4e 100644
--- a/addons/web/static/src/js/view_form.js
+++ b/addons/web/static/src/js/view_form.js
@@ -3258,7 +3258,6 @@ instance.web.form.FieldRadio = instance.web.form.AbstractField.extend(instance.w
     render_value: function () {
         var self = this;
         this.$el.toggleClass("oe_readonly", this.get('effective_readonly'));
-        this.$("input:checked").prop("checked", false);
         if (this.get_value()) {
             this.$("input").filter(function () {return this.value == self.get_value();}).prop("checked", true);
             this.$(".oe_radio_readonly").text(this.get('value') ? this.get('value')[1] : "");