Skip to Content
Menu
This question has been flagged
3 Replies
4767 Views

Hello, I want to compute a field based on a number of records in a table.

That I want to do at the finish is :

code_tarif
numero_ordre
...
12
1
...
12
2
...
15
1
...
15
2
...
15
3
...

But with this code, I have an error if I have more than 1 record in the table :

Code :

 class miadi_lignesTarifs(models.Model):
    _name = 'miadi.lignestarifs'
  
    code_tarif = fields.Many2one('miadi.tarifs', 'Price Code', default='', required=True, ondelete='cascade')
    numero_ordre = fields.Integer('Order number', compute='_compute_numero_ordre')
    produit_id = fields.Many2one('product.product', 'Product', required=True, ondelete='cascade')
    conditionnement_id = fields.Many2one('miadi.packaging', 'Packaging', required=True, ondelete='cascade')

    @api.depends('code_tarif')
    def _compute_numero_ordre(self):
        tarifs = self.env['miadi.lignestarifs']
        self.numero_ordre = tarifs.search_count([('code_tarif', '=', self.code_tarif.id)])

Error :

ValueError: Expected singleton: miadi.lignestarifs(17, 18)


I tried to do it overriding the create method :

@api.multi
@api.model
def create(self, vals):
    tarifs = self.env['miadi.lignestarifs']
    self.numero_ordre = tarifs.search_count([('code_tarif', '=', self.code_tarif.id)])
    return super(miadi_lignesTarifs, self).create(vals)

But it gives me this :

TypeError: create() takes exactly 2 arguments (1 given)


How can I do what I want ?

I'm on Odoo 10

Thanks for answers

Avatar
Discard
Author

Hi NIKHIL, I answer you in comment because I've not enought karma.

This solution doesn't work because it will put the same order number for all records.

I found the solution, I overrided the create method. In my first post, the create method were good but I had an error because of the copy/paste. I delete the function and I rewrit it and it works.

Thanks

Best Answer

Hi 
Please use this code, it may help you. thanks

  @api.depends('code_tarif')
    def _compute_numero_ordre(self):
        tarifs = self.env['miadi.lignestarifs']
        for rec in self:
            rec.numero_ordre = tarifs.search_count([('code_tarif', '=', rec.code_tarif.id)])

the compute function may run in the time of tree view loaded.  that time 'self' contain multiple records, to solve this we can use for loom.
and  why you super the create function, there is no need of it.


Avatar
Discard
Author Best Answer

I found the solution !

I use the create method and I override it.

I don't know why it didn't work but I delete the create method and I rewrite it again and it works.

Avatar
Discard

Without this create method it wont work ?

Author

Hi Niyas, no without this, I doesn't work