Merge pull request #47 from akretion/14-timesheet-task-mandatory

[IMP] Make task mandatory in timesheet line views because invoicing w…
akretion · Apr 25, 2024 · 961d7be · 961d7be
2 parents 817f4d0 + 8fdcab8
commit 961d7be
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/project_invoicing_subcontractor/views/hr_timesheet_sheet.xml b/project_invoicing_subcontractor/views/hr_timesheet_sheet.xml
@@ -11,6 +11,17 @@
             >
                 <field name="discount" />
             </xpath>
+            <field name="add_line_task_id" position="attributes">
+                <attribute
+                    name="attrs"
+                >{'required': [('add_line_project_id', '!=', False)]}</attribute>
+            </field>
+            <xpath
+                expr="//field[@name='timesheet_ids']/tree/field[@name='task_id']"
+                position="attributes"
+            >
+                <attribute name="required">1</attribute>
+            </xpath>
         </field>
     </record>
 

diff --git a/project_invoicing_subcontractor/views/hr_timesheet_view.xml b/project_invoicing_subcontractor/views/hr_timesheet_view.xml
@@ -40,6 +40,9 @@
             <field name="project_id" position="before">
                 <field name="partner_id" optional="show" />
             </field>
+            <field name="task_id" position="attributes">
+                <attribute name="required">1</attribute>
+            </field>
             <field name="task_id" position="after">
                 <field name="parent_task_id" optional="hide" />
                 <field name="task_stage_id" optional="show" />