summaryrefslogtreecommitdiff
path: root/addons/survey/security/survey_security.xml
diff options
context:
space:
mode:
authorThibault Delavallée <tde@odoo.com>2019-11-25 12:26:57 +0000
committerThibault Delavallée <tde@odoo.com>2019-12-05 15:21:28 +0000
commit2592e72f2cd80f09fb23b4b16f8eb92ba253ff67 (patch)
treeed6af2529339fc37003f28384aab8499bde85aa0 /addons/survey/security/survey_security.xml
parent84bb9c748db26656d3ecd062183afe5f042dc569 (diff)
[REF] survey: globally rename survey.label model to survey.question.answer
PURPOSE As new features are about to land in survey, notably live interactions [1] and new survey building [2] performing a pre cleaning is necessary. In this PR we clean survey models by: removing unnecessary fields, cleaning some code and finally renaming models. SPECIFICATIONS In this commit we rename the long standing ``survey.label`` model. Indeed a label is something before a question, like an input label. Labels in survey are used for suggested answers and sometimes as rows for matrix answers. After much thoughts we rename ``survey.label`` to ``survey.question.answer``. It indicates this model holds answers. Moreover it is namespaced within the ``survey.question`` model name to avoid conflict with user input / user answer model. As model naming changes, some fields also evolve. In survey.question model * ``labels_id`` is renamed to ``suggested_answer_ids`` to indicate it is used to display suggested values to the user; * ``labels_id_2`` is renamed to ``matrix_row_ids`` to indicate it is used to generate the rows of the matrix-type question. A matrix is therefore done using ``matrix_row_ids`` for rows and ``suggested_answer_ids`` for columns which seems easier to understand; In survey.question.answer (old survey.label) model * ``question_id_2`` is renamed to ``matrix_question_id`` to ease its understanding, notably that it is used for matrix questions; LINKS [0] Related to Task ID 2061901 (survey models cleaning and preparation) [1] Task ID 1972640 (live interactions) [2] Task ID 2119587 (new frontend for building surveys) PR #40765
Diffstat (limited to 'addons/survey/security/survey_security.xml')
-rw-r--r--addons/survey/security/survey_security.xml20
1 files changed, 10 insertions, 10 deletions
diff --git a/addons/survey/security/survey_security.xml b/addons/survey/security/survey_security.xml
index 46bf54e53ac..4b53a1d29fa 100644
--- a/addons/survey/security/survey_security.xml
+++ b/addons/survey/security/survey_security.xml
@@ -87,9 +87,9 @@
<field name="perm_create" eval="1"/>
</record>
- <record id="survey_label_rule_survey_manager" model="ir.rule">
- <field name="name">Survey label: manager: all</field>
- <field name="model_id" ref="survey.model_survey_label"/>
+ <record id="survey_question_answer_rule_survey_manager" model="ir.rule">
+ <field name="name">Survey question answer: manager: all</field>
+ <field name="model_id" ref="survey.model_survey_question_answer"/>
<field name="domain_force">[(1, '=', 1)]</field>
<field name="groups" eval="[(4, ref('group_survey_manager'))]"/>
<field name="perm_unlink" eval="1"/>
@@ -97,9 +97,9 @@
<field name="perm_read" eval="1"/>
<field name="perm_create" eval="1"/>
</record>
- <record id="survey_label_rule_survey_user_read" model="ir.rule">
- <field name="name">Survey label: officer: read all</field>
- <field name="model_id" ref="survey.model_survey_label"/>
+ <record id="survey_question_answer_rule_survey_user_read" model="ir.rule">
+ <field name="name">Survey question answer: officer: read all</field>
+ <field name="model_id" ref="survey.model_survey_question_answer"/>
<field name="domain_force">[(1, '=', 1)]</field>
<field name="groups" eval="[(4, ref('group_survey_user'))]"/>
<field name="perm_unlink" eval="0"/>
@@ -107,10 +107,10 @@
<field name="perm_read" eval="1"/>
<field name="perm_create" eval="0"/>
</record>
- <record id="survey_label_rule_survey_user_cw" model="ir.rule">
- <field name="name">Survey label: officer: create/write/unlink linked to own survey only</field>
- <field name="model_id" ref="survey.model_survey_question"/>
- <field name="domain_force">[('survey_id.create_uid', '=', user.id)]</field>
+ <record id="survey_question_answer_rule_survey_user_cw" model="ir.rule">
+ <field name="name">Survey question answer: officer: create/write/unlink linked to own survey only</field>
+ <field name="model_id" ref="survey.model_survey_question_answer"/>
+ <field name="domain_force">['|', ('question_id.survey_id.create_uid', '=', user.id), ('matrix_question_id.survey_id.create_uid', '=', user.id)]</field>
<field name="groups" eval="[(4, ref('group_survey_user'))]"/>
<field name="perm_unlink" eval="1"/>
<field name="perm_write" eval="1"/>