Skip to Content
This question has been flagged
order.linecomputed-fields10.0
6 Replies
5647 Views
Avatar
Dishon Kadoh

Hello guys i have inherited the account.analytic.line model and added a selection field on whether the duration affects the sla or not i want to compute the totals for the time that affects only sla i.e when the selection is yes the duration is computed if no it is not computed how do i go about it.Any assistance will be highly appreciated.

this is the link to my view https://ibb.co/eDr2Te

Avatar
Discard
Avatar
Cyril Gaspard (GEM)
Best Answer

Hi,

code of Ibrahim  optimized

@api.multi
@api.depends('timesheet_ids', 'timesheet_ids.sla', 'timesheet_ids.duration')

def _compute_total_duration(self):

    for record in self:

        record.total_duration = sum(record.timesheet_ids.filtered(lambda rec: rec.sla == 'yes').mapped('duration'))

Regards

Avatar
Discard
Cyril Gaspard (GEM)

Hi Ibrahim,

thank you ^^,

bye

Dishon Kadoh
Author

Thanks Cyril and Ibrahim.THe code is running Perfectly

Cyril Gaspard (GEM)

Hi,

I recommand you to store the value for better performance, y adding strore=True in the definition of your field. To avoid ir.rule test, and better performance, you can add compute_sudo=True too.

Bye

Avatar
subbarao
Best Answer

add dependency to total compute field

ex:

api.dependency('your sla filed from line',....etc fields)

def _amount_total()

Avatar
Discard
Avatar
Ibrahim Boudmir
Best Answer

Hi, 
you want to calculate the Total of the time that affects only sla. 
Considering that : 
your O2M field is :  timesheet_ids

the total field is : total_duration

the affects SLA :  sla

the Duration : duration

then : 

add (compute='_compute_total_duration', store=True) to total_duration

and 

@api.multi
@api.depends('timesheet_ids', 'timesheet_ids.sla', 'timesheet_ids.duration')
def _compute_total_duration(self):
    for record in self:
        total = False
        // Filter only on record with sla = Yes
        timesheets = record.timesheet_ids.filtered(lambda rec: rec.sla == 'yes')
        if timesheets:
            for timesheet in timesheets:
                total += timesheet.duration     // here, you may want to check how to add durations based on your field's signature
        record.total_duration = total                    
Avatar
Discard
Cyril Gaspard (GEM)

Hi,

I think you forgot something in your code :

total += timesheet

should be

total += timesheet.duration

Bye

Cyril Gaspard (GEM)

second error, :

total = False

should be not before but in for record in self:

Ibrahim Boudmir

I agree Cyril. I have updated my answer

Thank you

Cyril Gaspard (GEM)

Hi,

I add @api.muti in your code

bye

Related Posts Replies Views Activity
How is caching working ?
cache compute computed-fields 10.0
Avatar
0
Oct 17
5634
Compute Fields Solved
computed-fields
Avatar
Avatar
2
Jul 24
411
Odoo computed field
computed-fields
Avatar
Avatar
1
Jan 24
421
Make stored compute filed recompute after changing it logic but not depedencies.
computed-fields
Avatar
Avatar
Avatar
2
Apr 24
1932
Compute field odoo 14
computed-fields
Avatar
Avatar
1
Jul 22
892