[REM] Get rid of the ugly 'default_groups_ref' context key
Because it is useless now with the new behavior brought by the previous commit. All the necessary details are explained in this one.
This commit is contained in:
Yannick Tivisse
@@ -312,7 +312,7 @@
|
||||
<field name="journal_id" groups="account.group_account_user" options="{'no_create': True}"
|
||||
attrs="{'readonly':[('move_name','!=',False)]}"/>
|
||||
<field domain="[('partner_id', '=', partner_id)]" name="partner_bank_id" invisible="1"/>
|
||||
<field name="user_id" string="Responsible" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'account.group_account_invoice']}"/>
|
||||
<field name="user_id" string="Responsible"/>
|
||||
<field domain="[('company_id', '=', company_id), ('internal_type', '=', 'payable')]"
|
||||
name="account_id" groups="account.group_account_manager"/>
|
||||
<field name="name" invisible="1"/>
|
||||
@@ -379,7 +379,7 @@
|
||||
<group>
|
||||
<field name="date_invoice"/>
|
||||
<field name="move_name" invisible="1"/>
|
||||
<field name="user_id" groups="base.group_user" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'account.group_account_invoice']}"/>
|
||||
<field name="user_id" groups="base.group_user"/>
|
||||
<label for="currency_id" groups="base.group_multi_currency"/>
|
||||
<div groups="base.group_multi_currency">
|
||||
<field name="currency_id" options="{'no_create': True, 'no_open': True}" class="oe_inline"/>
|
||||
|
||||
@@ -29,7 +29,7 @@
|
||||
<separator string="Conditions" colspan="4"/>
|
||||
<field name="product_id"/>
|
||||
<field name="partner_id"/>
|
||||
<field name="user_id" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'account.group_account_invoice', 'base.group_sale_salesman']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="company_id" options="{'no_create': True}" groups="base.group_multi_company"/>
|
||||
<field name="date_start"/>
|
||||
<field name="date_stop"/>
|
||||
|
||||
@@ -81,7 +81,7 @@
|
||||
</div>
|
||||
<group>
|
||||
<group>
|
||||
<field name="creating_user_id" attrs="{'readonly':[('state','!=','draft')]}" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'account.group_account_user']}"/>
|
||||
<field name="creating_user_id" attrs="{'readonly':[('state','!=','draft')]}"/>
|
||||
</group>
|
||||
<group>
|
||||
<label for="date_from" string="Period"/>
|
||||
|
||||
@@ -157,8 +157,7 @@
|
||||
<field name="fax"/>
|
||||
</group>
|
||||
<group>
|
||||
<field name="user_id" on_change="on_change_user(user_id, context)"
|
||||
context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'base.group_sale_salesman_all_leads'] }" widget="selection"/>
|
||||
<field name="user_id" on_change="on_change_user(user_id, context)" widget="selection"/>
|
||||
<field name="team_id" widget="selection" domain="[('use_leads','=',True)]"/>
|
||||
<field name="type" invisible="1"/>
|
||||
</group>
|
||||
@@ -507,7 +506,7 @@
|
||||
</group>
|
||||
|
||||
<group>
|
||||
<field name="user_id" on_change="on_change_user(user_id, context)" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'base.group_sale_salesman_all_leads']}" widget="selection"/>
|
||||
<field name="user_id" on_change="on_change_user(user_id, context)" widget="selection"/>
|
||||
<field name="team_id" widget="selection"/>
|
||||
</group>
|
||||
<group>
|
||||
|
||||
@@ -129,7 +129,7 @@
|
||||
<field name="date"/>
|
||||
</group>
|
||||
<group colspan="4" col="4" groups="base.group_user">
|
||||
<field name="user_id" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'base.group_sale_salesman_all_leads']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="priority" widget="priority"/>
|
||||
<field name="team_id"/>
|
||||
<field name="date_deadline"/>
|
||||
|
||||
@@ -133,7 +133,6 @@
|
||||
<field name="view_type">form</field>
|
||||
<field name="target">current</field>
|
||||
<field name="view_id" ref="base.view_users_simple_form"/>
|
||||
<field name="context">{'default_groups_ref': ['base.group_user']}</field>
|
||||
<field name="help">Create and manage users that will connect to the system. Users can be deactivated should there be a period of time during which they will/should not connect to the system. You can assign them groups in order to give them specific access to the applications they need to use in the system.</field>
|
||||
</record>
|
||||
|
||||
|
||||
@@ -82,8 +82,7 @@
|
||||
<group>
|
||||
<group string='Status' name="active_group">
|
||||
<field name="company_id" groups="base.group_multi_company" on_change="onchange_company(company_id)"/>
|
||||
<field name="user_id" on_change="onchange_user(name, image, user_id)" string="Related User"
|
||||
context="{'default_groups_ref': ['base.group_user']}"/>
|
||||
<field name="user_id" on_change="onchange_user(name, image, user_id)" string="Related User"/>
|
||||
</group>
|
||||
</group>
|
||||
</page>
|
||||
|
||||
@@ -136,8 +136,7 @@
|
||||
<field name="type_id" placeholder="Degree"/>
|
||||
</group>
|
||||
<group>
|
||||
<field name="user_id"
|
||||
context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'base.group_hr_manager']}"/>
|
||||
<field name="user_id"/>
|
||||
<label for="title_action"/>
|
||||
<div>
|
||||
<field name="date_action"/>
|
||||
|
||||
@@ -8,8 +8,7 @@
|
||||
<field name="arch" type="xml">
|
||||
<tree editable="top" string="Timesheet Activities">
|
||||
<field name="date"/>
|
||||
<field name="user_id" required="1" options='{"no_open": True}'
|
||||
context="{'default_groups_ref': ['base.group_user']}"/>
|
||||
<field name="user_id" required="1" options='{"no_open": True}'/>
|
||||
<field name="name"/>
|
||||
<field name="account_id" context="{'default_type': 'contract'}" options="{'no_open': True, 'no_create': True}"/>
|
||||
<field name="unit_amount" string="Duration" sum="Total time" widget="float_time"/>
|
||||
@@ -26,8 +25,7 @@
|
||||
<group>
|
||||
<group>
|
||||
<field name="name"/>
|
||||
<field name="user_id" required="1"
|
||||
context="{'default_groups_ref': ['base.group_user']}"/>
|
||||
<field name="user_id" required="1"/>
|
||||
<field name="is_timesheet" invisible="1"/>
|
||||
</group>
|
||||
<group>
|
||||
|
||||
@@ -108,8 +108,7 @@
|
||||
<form string="cashmove form">
|
||||
<sheet>
|
||||
<group>
|
||||
<field name="user_id"
|
||||
context="{'default_groups_ref': ['base.group_user', 'lunch.group_lunch_user']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="date"/>
|
||||
<field name="amount"/>
|
||||
</group>
|
||||
@@ -388,7 +387,7 @@
|
||||
|
||||
<group>
|
||||
<group>
|
||||
<field name='user_id' context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'lunch.group_lunch_user']}" class="oe_inline"/>
|
||||
<field name='user_id' class="oe_inline"/>
|
||||
<field name='date'/>
|
||||
</group>
|
||||
<div>
|
||||
|
||||
@@ -617,7 +617,7 @@
|
||||
('product_id','=',False)]"
|
||||
context="{'default_product_id': product_id, 'default_product_tmpl_id': product_tmpl_id}" on_change="bom_id_change(bom_id)" required="1"/>
|
||||
<field name="routing_id" groups="mrp.group_mrp_routings"/>
|
||||
<field name="user_id" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'mrp.group_mrp_user']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="origin"/>
|
||||
</group>
|
||||
<group groups="stock.group_locations">
|
||||
|
||||
@@ -79,7 +79,7 @@
|
||||
<group string="General Information">
|
||||
<field name="company_id" groups="base.group_multi_company"/>
|
||||
<field name="location_id" widget="selection" groups="stock.group_locations"/>
|
||||
<field name="user_id" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'point_of_sale.group_pos_user']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="pricelist_id" groups="product.group_sale_pricelist" domain="[('type','=','sale')]"/>
|
||||
<field name="picking_id" readonly="1"/>
|
||||
<field name="pos_reference"/>
|
||||
@@ -835,7 +835,7 @@
|
||||
<group>
|
||||
<field name="cash_control" invisible="1" />
|
||||
<group>
|
||||
<field name="user_id" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'point_of_sale.group_pos_manager', 'base.group_sale_manager']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="currency_id" invisible="1"/>
|
||||
<field name="config_id"/>
|
||||
</group>
|
||||
|
||||
@@ -114,7 +114,6 @@
|
||||
<field name="partner_id" on_change="onchange_partner_id(partner_id)" string="Customer"/>
|
||||
<field name="user_id" string="Project Manager"
|
||||
attrs="{'readonly':[('state','in',['close', 'cancelled'])]}"
|
||||
context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'project.group_project_manager']}"
|
||||
groups="base.group_no_one"/>
|
||||
</group>
|
||||
<group name="misc">
|
||||
@@ -264,8 +263,7 @@
|
||||
<group>
|
||||
<field name="project_id" domain="[('state','not in', ('close', 'cancelled'))]" on_change="onchange_project(project_id)" context="{'default_use_tasks':1}"/>
|
||||
<field name="user_id"
|
||||
options='{"no_open": True}'
|
||||
context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'project.group_project_user']}"/>
|
||||
options='{"no_open": True}'/>
|
||||
<field name="planned_hours" widget="float_time"
|
||||
groups="project.group_time_work_estimation_tasks"
|
||||
on_change="onchange_planned(planned_hours)"/>
|
||||
|
||||
@@ -25,8 +25,7 @@
|
||||
</div>
|
||||
<group>
|
||||
<group>
|
||||
<field name="user_id"
|
||||
context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'project.group_project_user']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="priority" groups="base.group_user" widget="priority"/>
|
||||
<field name="tag_ids" widget="many2many_tags"/>
|
||||
</group>
|
||||
|
||||
@@ -53,7 +53,7 @@
|
||||
</div>
|
||||
<group>
|
||||
<group>
|
||||
<field name="user_id" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'purchase.group_purchase_user', 'account.group_account_invoice']}" attrs="{'readonly': [('state','not in',('draft'))]}"/>
|
||||
<field name="user_id" attrs="{'readonly': [('state','not in',('draft'))]}"/>
|
||||
<field name="exclusive" attrs="{'readonly': [('state','not in',('draft'))]}"/>
|
||||
<field name="account_analytic_id" colspan="2" groups="purchase.group_analytic_accounting" attrs="{'readonly': [('state','not in',('draft'))]}"/>
|
||||
<field name="multiple_rfq_per_supplier" attrs="{'readonly': [('state','not in',('draft'))]}"/>
|
||||
|
||||
@@ -307,7 +307,7 @@
|
||||
<page string="Other Information">
|
||||
<group>
|
||||
<group string="Sales Information" name="sales_person">
|
||||
<field name="user_id" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'account.group_account_invoice', 'base.group_sale_salesman_all_leads']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="team_id" options="{'no_create': True}"/>
|
||||
<field name="client_order_ref"/>
|
||||
<field name="company_id" options="{'no_create': True}" groups="base.group_multi_company"/>
|
||||
|
||||
@@ -59,7 +59,7 @@
|
||||
</div>
|
||||
<group>
|
||||
<group name="left">
|
||||
<field name="user_id" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'base.group_sale_salesman_all_leads']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="code" groups="base.group_no_one"/>
|
||||
</group>
|
||||
<group name="right">
|
||||
|
||||
@@ -18,7 +18,7 @@
|
||||
<group col="4">
|
||||
<field name="name"/>
|
||||
<field name="partner_id"/>
|
||||
<field name="user_id" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'account.group_account_invoice', 'base.group_sale_salesman']}"/>
|
||||
<field name="user_id"/>
|
||||
<field name="active"/>
|
||||
</group>
|
||||
<notebook>
|
||||
|
||||
@@ -71,7 +71,7 @@
|
||||
<field name="ranking" invisible="1"/>
|
||||
</group>
|
||||
<group string="Technical" groups="base.group_no_one">
|
||||
<field name="write_uid" context="{'default_groups_ref': ['base.group_user', 'base.group_partner_manager', 'base.group_website_designer']}"/>
|
||||
<field name="write_uid"/>
|
||||
<field name="write_date"/>
|
||||
</group>
|
||||
</sheet>
|
||||
|
||||
@@ -310,8 +310,7 @@
|
||||
<group name="container_row_2">
|
||||
<group string="Sale" name="sale">
|
||||
<field name="customer"/>
|
||||
<field name="user_id"
|
||||
context="{'default_groups_ref': ['base.group_partner_manager']}"/>
|
||||
<field name="user_id"/>
|
||||
</group>
|
||||
<group string="Purchase" name="purchase">
|
||||
<field name="supplier"/>
|
||||
|
||||
@@ -869,20 +869,6 @@ class users_view(osv.osv):
|
||||
values = super(users_view, self).default_get(cr, uid, fields1, context)
|
||||
self._add_reified_groups(group_fields, values)
|
||||
|
||||
# add "default_groups_ref" inside the context to set default value for group_id with xml values
|
||||
if 'groups_id' in fields and isinstance(context.get("default_groups_ref"), list):
|
||||
groups = []
|
||||
ir_model_data = self.pool.get('ir.model.data')
|
||||
for group_xml_id in context["default_groups_ref"]:
|
||||
group_split = group_xml_id.split('.')
|
||||
if len(group_split) != 2:
|
||||
raise UserError(_('Invalid context default_groups_ref value (model.name_id) : "%s"') % group_xml_id)
|
||||
try:
|
||||
temp, group_id = ir_model_data.get_object_reference(cr, uid, group_split[0], group_split[1])
|
||||
except ValueError:
|
||||
group_id = False
|
||||
groups += [group_id]
|
||||
values['groups_id'] = groups
|
||||
return values
|
||||
|
||||
def read(self, cr, uid, ids, fields=None, context=None, load='_classic_read'):
|
||||
|
||||
Reference in New Issue
Block a user