From c23e10654c2fa9bdf24147ee41b3248ac0db1bcd Mon Sep 17 00:00:00 2001
From: "Xavier Bol (xbo)" <xbo@odoo.com>
Date: Mon, 5 Jun 2023 15:43:25 +0000
Subject: [PATCH] [FIX] sale_project: edit SOL in subtasks subtree if user is
 salesman

Before this commit, when the user set a subtask and wants to directly
set the SOL of that subtask in the subtree view of a task form view. He
cannot because the field is in readonly.

This commit makes the field editable if the user has access to Sales app
as it is the case for the SOL field in the form view of task.

task-3336215

closes odoo/odoo#124563

X-original-commit: 0373abdf24e1139706e6d7610873cc89ada60e74
Signed-off-by: Xavier Bol (xbo) <xbo@odoo.com>
---
 addons/sale_project/views/project_task_views.xml | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/addons/sale_project/views/project_task_views.xml b/addons/sale_project/views/project_task_views.xml
index b4aefd19fc2..39e9cc013f0 100644
--- a/addons/sale_project/views/project_task_views.xml
+++ b/addons/sale_project/views/project_task_views.xml
@@ -95,7 +95,14 @@
                 <field name="allow_billable" invisible="1"/>
             </xpath>
             <xpath expr="//field[@name='child_ids']/tree/field[@name='partner_id']" position="after">
-                <field name="sale_line_id" optional="hide" readonly="1" groups="sales_team.group_sale_salesman"/>
+                <field name="sale_line_id"
+                       optional="hide"
+                       options='{"no_create": True}'
+                       context="{'create': False, 'edit': False, 'delete': False, 'with_price_unit': True}"
+                       placeholder="Non-billable"
+                       groups="sales_team.group_sale_salesman"
+                       attrs="{'invisible': [('allow_billable', '=', False)]}"
+                />
                 <field name="sale_line_id" optional="hide" options="{'no_open': True}" readonly="1" groups="!sales_team.group_sale_salesman"/>
                 <field name="allow_billable" invisible="1"/>
             </xpath>