count the sales of a worker

Question

I am trying to count the sales made by a worker but I get the following error when using the code mentioned below:

TypeError: object of type 'bool' has no len()

class Movement_type (models.Model):

   _name = 'project_rc.movement_type'

   _rec_name = 'movement_type'

type_movement = fields.Selection ([('purchase', 'Purchase'), ('sale', 'Sale'), ('merma', 'Merma')], string = "Movement type", required = True)

class Worker (models.Model):

  _name = 'project_rc.worker'

  _rec_name = 'name'

sales_counter = fields.Integer (string = "Sales made", compute = "get_sales_realized", store = True)

@api.depends('move_type_ids')

  def get_sales_realized (self):

    for rec in self:

        rec.count_sale = len (rec.move_type_ids.mov_type == 'sale')

Answer 1

I'm not familiar with whatever framework you are using, but if you look at the error you are getting you can see that it is correct.

On line 3, you write rec.move_type_ids.mov_type == 'sale'. It doesn't matter what rec.move_type_ids.mov_type is, when you compare it to something with ==, the answer will either be True or False. It doesn't make sense to take the length of a boolean (t/f).

From the context, I'm guessing that rec.move_type_ids is a list of objects and you want to figure out how many of them have a mov_type property equal to 'sale'. If that's the case, then you could easily do that with a for loop:

sales = []
for thing in rec.move_type_ids:
    if thing.type == 'sale':
        sales.append(thing)
rec.count_sale = len(sales)

If you want to get a little fancier, you can do that with a filter function:

rec.count_sale = len(filter(lambda x: x.mov_type == 'sale', rec.move_type_ids))