[FIX] sale_timesheet_line_exclude: recompute so line when changing ex…

…clude flag

While the test does not reveal this, without this change, clicking on the
checkbox in the timesheet list view
does not reset the so line.

sale_timesheet_line_exclude/models/account_analytic_line.py

@@ -36,8 +36,8 @@ def _compute_timesheet_invoice_type(self):
         return res
 
     @api.depends("exclude_from_sale_order")
-    def _compute_so_line_on_exclude(self):
-        self._compute_so_line()
+    def _compute_so_line(self):
+        return super()._compute_so_line()
 
     def _timesheet_determine_sale_line(self):
         self.ensure_one()

sale_timesheet_line_exclude/tests/test_sale_timesheet_line_exclude.py

@@ -174,7 +174,9 @@ def test_write_exclude_from_sale_order(self):
                 "account_id": self.project.analytic_account_id.id,
             }
         )
+        self.assertTrue(timesheet.so_line)
         timesheet.write({"exclude_from_sale_order": True})
+        self.assertFalse(timesheet.so_line)
 
         self.assertEqual(timesheet.timesheet_invoice_type, "non_billable")
         self.assertEqual(self.sale_order_line.qty_delivered, 0)