From 924b9dac6e723d0611b38a325bf4ec647b01f17d Mon Sep 17 00:00:00 2001 From: "pawan.sharma" Date: Mon, 11 Apr 2022 17:25:22 +0530 Subject: [PATCH 1/2] add check for update previous value --- cor_custom/models/project.py | 3 ++- cor_custom/views/project_view.xml | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/cor_custom/models/project.py b/cor_custom/models/project.py index 37fa516..461c412 100755 --- a/cor_custom/models/project.py +++ b/cor_custom/models/project.py @@ -72,6 +72,7 @@ class Project(models.Model): project_cons_hrs = fields.One2many('project.consultant.hrs', 'project_id', 'Consultant Allocation', copy=False) comment = fields.Text(string='Comment') tag_ids = fields.Many2many('custom.project.tags', string='Tags') + is_check = fields.Boolean() @api.onchange('allowed_internal_user_ids') def onchange_add_allowed_internal_users(self): @@ -109,7 +110,7 @@ class Project(models.Model): 'employee_id': rec[1], 'timesheet_hour': rec[2]}) - @api.depends('cost', 'expenses_amt', 'budgeted_revenue') + @api.depends('cost', 'expenses_amt', 'budgeted_revenue', 'is_check') def _compute_calc(self): for record in self: consultant_cost = 0.0 diff --git a/cor_custom/views/project_view.xml b/cor_custom/views/project_view.xml index f424b6f..52b08ce 100755 --- a/cor_custom/views/project_view.xml +++ b/cor_custom/views/project_view.xml @@ -59,6 +59,7 @@ + From 08e49e1a06587d655716b426db7daabc604194ab Mon Sep 17 00:00:00 2001 From: "pawan.sharma" Date: Mon, 11 Apr 2022 17:54:48 +0530 Subject: [PATCH 2/2] profit should be neg or positive --- cor_custom/models/project.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/cor_custom/models/project.py b/cor_custom/models/project.py index 461c412..ecdbdfa 100755 --- a/cor_custom/models/project.py +++ b/cor_custom/models/project.py @@ -140,8 +140,7 @@ class Project(models.Model): # record.profit_per = (record.profit_amt / record.budgeted_revenue) * 100 ##### CR Changes - if record.actual_revenue > record.total_expenses: - record.profit_amt = record.actual_revenue - record.total_expenses + record.profit_amt = record.actual_revenue - record.total_expenses if record.profit_amt > 0.0 and record.actual_revenue > 0.0: record.profit_per = (record.profit_amt / record.actual_revenue) * 100 ########################