From 2247c2683454dc3d2af9d24110f5c6b87cfce20f Mon Sep 17 00:00:00 2001
From: Noe Antoine <nan@odoo.com>
Date: Wed, 19 Oct 2022 14:41:18 +0000
Subject: [PATCH] [IMP] website_event_questions: clarify event question page
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

In order to clear the tree view of questions in the event
form, shorten labels (also on event template) and add a String
to the reporting button. This makes it more explicit.

Add an action helper on the event registration answer views if
no answer exists yet (for that event question).

Task-3032546

closes odoo/odoo#103556

Signed-off-by: Warnon Aurélien (awa) <awa@odoo.com>
---
 .../views/event_registration_answer_views.xml          | 10 +++++++++-
 addons/website_event_questions/views/event_views.xml   | 10 +++++-----
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/addons/website_event_questions/views/event_registration_answer_views.xml b/addons/website_event_questions/views/event_registration_answer_views.xml
index d5cd93c2b06..876c5f1ca68 100644
--- a/addons/website_event_questions/views/event_registration_answer_views.xml
+++ b/addons/website_event_questions/views/event_registration_answer_views.xml
@@ -50,6 +50,14 @@
     <record id="action_event_registration_report" model="ir.actions.act_window">
         <field name="name">Answer Breakdown</field>
         <field name="res_model">event.registration.answer</field>
-        <field name="view_mode">search,tree,graph,pivot</field>
+        <field name="view_mode">tree,graph,pivot</field>
+        <field name="search_view_id" ref="event_registration_answer_view_search"/>
+        <field name="help" type="html">
+            <p class="o_view_nocontent_smiling_face">
+                No Answers yet!
+            </p><p>
+                Come back once you have registrations to overview answers.
+            </p>
+        </field>
     </record>
 </odoo>
diff --git a/addons/website_event_questions/views/event_views.xml b/addons/website_event_questions/views/event_views.xml
index 20ba25e192d..ea133415430 100644
--- a/addons/website_event_questions/views/event_views.xml
+++ b/addons/website_event_questions/views/event_views.xml
@@ -10,8 +10,8 @@
                      <field name="question_ids" class="w-100">
                          <tree sample="1">
                              <field name="title"/>
-                             <field name="is_mandatory_answer"/>
-                             <field name="once_per_order"/>
+                             <field name="is_mandatory_answer" string="Mandatory"/>
+                             <field name="once_per_order" string="Once per Order"/>
                              <field name="question_type" />
                              <field name="answer_ids" widget = "many2many_tags"/>
                          </tree>
@@ -33,12 +33,12 @@
                             <tree>
                                 <field name="sequence" widget="handle" />
                                 <field name="title"/>
-                                <field name="is_mandatory_answer"/>
-                                <field name="once_per_order"/>
+                                <field name="is_mandatory_answer" string="Mandatory"/>
+                                <field name="once_per_order" string="Once per Order"/>
                                 <field name="question_type" string="Type" />
                                 <field name="answer_ids" widget="many2many_tags"
                                     attrs="{'invisible': [('question_type', '!=', 'simple_choice')]}" />
-                                <button name="action_view_question_answers" type="object" class="fa fa-bar-chart p-0" title="Answer Breakdown" />
+                                <button name="action_view_question_answers" type="object" class="p-0" icon="fa-bar-chart pe-1" string="Stats" title="Answer Breakdown"/>
                             </tree>
                             <!-- Need to repeat the whole tree form here to be able to create answers properly
                                 Without this, the sub-fields of answer_ids are unknown to the web framework.