[FIX] Crash in base_user_role when role_id is not set on res.users.role.line

This commit is contained in:
Alexis de Lattre
2017-06-12 23:14:11 +02:00
committed by sebalix
parent ed30c31fc8
commit daa65abf91
4 changed files with 16 additions and 15 deletions

View File

@@ -8,8 +8,8 @@ msgid ""
msgstr "" msgstr ""
"Project-Id-Version: Odoo Server 10.0\n" "Project-Id-Version: Odoo Server 10.0\n"
"Report-Msgid-Bugs-To: \n" "Report-Msgid-Bugs-To: \n"
"POT-Creation-Date: 2017-02-22 00:55+0000\n" "POT-Creation-Date: 2017-08-01 02:44+0000\n"
"PO-Revision-Date: 2017-02-22 00:55+0000\n" "PO-Revision-Date: 2017-08-01 02:44+0000\n"
"Last-Translator: OCA Transbot <transbot@odoo-community.org>, 2017\n" "Last-Translator: OCA Transbot <transbot@odoo-community.org>, 2017\n"
"Language-Team: Catalan (https://www.transifex.com/oca/teams/23907/ca/)\n" "Language-Team: Catalan (https://www.transifex.com/oca/teams/23907/ca/)\n"
"MIME-Version: 1.0\n" "MIME-Version: 1.0\n"
@@ -134,7 +134,7 @@ msgid "Role"
msgstr "" msgstr ""
#. module: base_user_role #. module: base_user_role
#: model:ir.model.fields,field_description:base_user_role.field_res_users_role_line_ids_1497 #: model:ir.model.fields,field_description:base_user_role.field_res_users_role_line_ids_1510
msgid "Role lines" msgid "Role lines"
msgstr "" msgstr ""
@@ -189,7 +189,7 @@ msgstr ""
#: model:ir.model.fields,field_description:base_user_role.field_res_users_role_users #: model:ir.model.fields,field_description:base_user_role.field_res_users_role_users
#: model:ir.ui.view,arch_db:base_user_role.view_res_users_role_form #: model:ir.ui.view,arch_db:base_user_role.view_res_users_role_form
msgid "Users" msgid "Users"
msgstr "" msgstr "Usuaris"
#. module: base_user_role #. module: base_user_role
#: model:ir.model,name:base_user_role.model_res_users_role_line #: model:ir.model,name:base_user_role.model_res_users_role_line

View File

@@ -3,15 +3,15 @@
# * base_user_role # * base_user_role
# #
# Translators: # Translators:
# OCA Transbot <transbot@odoo-community.org>, 2016 # OCA Transbot <transbot@odoo-community.org>, 2017
# Pedro Castro Silva <pedrocs@sossia.pt>, 2016 # Pedro Castro Silva <pedrocs@sossia.pt>, 2017
msgid "" msgid ""
msgstr "" msgstr ""
"Project-Id-Version: Odoo Server 10.0c\n" "Project-Id-Version: Odoo Server 10.0\n"
"Report-Msgid-Bugs-To: \n" "Report-Msgid-Bugs-To: \n"
"POT-Creation-Date: 2016-12-17 02:07+0000\n" "POT-Creation-Date: 2017-08-01 02:44+0000\n"
"PO-Revision-Date: 2016-12-17 02:07+0000\n" "PO-Revision-Date: 2017-08-01 02:44+0000\n"
"Last-Translator: Pedro Castro Silva <pedrocs@sossia.pt>, 2016\n" "Last-Translator: Pedro Castro Silva <pedrocs@sossia.pt>, 2017\n"
"Language-Team: Portuguese (https://www.transifex.com/oca/teams/23907/pt/)\n" "Language-Team: Portuguese (https://www.transifex.com/oca/teams/23907/pt/)\n"
"MIME-Version: 1.0\n" "MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n" "Content-Type: text/plain; charset=UTF-8\n"
@@ -135,7 +135,7 @@ msgid "Role"
msgstr "" msgstr ""
#. module: base_user_role #. module: base_user_role
#: model:ir.model.fields,field_description:base_user_role.field_res_users_role_line_ids_1519 #: model:ir.model.fields,field_description:base_user_role.field_res_users_role_line_ids_1510
msgid "Role lines" msgid "Role lines"
msgstr "" msgstr ""
@@ -190,7 +190,7 @@ msgstr ""
#: model:ir.model.fields,field_description:base_user_role.field_res_users_role_users #: model:ir.model.fields,field_description:base_user_role.field_res_users_role_users
#: model:ir.ui.view,arch_db:base_user_role.view_res_users_role_form #: model:ir.ui.view,arch_db:base_user_role.view_res_users_role_form
msgid "Users" msgid "Users"
msgstr "" msgstr "Utilizadores"
#. module: base_user_role #. module: base_user_role
#: model:ir.model,name:base_user_role.model_res_users_role_line #: model:ir.model,name:base_user_role.model_res_users_role_line

View File

@@ -44,8 +44,9 @@ class ResUsers(models.Model):
lambda rec: rec.is_enabled) lambda rec: rec.is_enabled)
for role_line in role_lines: for role_line in role_lines:
role = role_line.role_id role = role_line.role_id
group_ids.append(role.group_id.id) if role:
group_ids.extend(role.implied_ids.ids) group_ids.append(role.group_id.id)
group_ids.extend(role.implied_ids.ids)
group_ids = list(set(group_ids)) # Remove duplicates IDs group_ids = list(set(group_ids)) # Remove duplicates IDs
vals = { vals = {
'groups_id': [(6, 0, group_ids)], 'groups_id': [(6, 0, group_ids)],

View File

@@ -12,7 +12,7 @@
<page string="Roles"> <page string="Roles">
<field name="role_line_ids" nolabel="1"> <field name="role_line_ids" nolabel="1">
<tree editable="bottom" colors="grey: not is_enabled;"> <tree editable="bottom" colors="grey: not is_enabled;">
<field name="role_id"/> <field name="role_id" required="1"/>
<field name="date_from"/> <field name="date_from"/>
<field name="date_to"/> <field name="date_to"/>
<field name="is_enabled"/> <field name="is_enabled"/>