Skip to Content
Menu
This question has been flagged
1 Atsakyti
21345 Rodiniai

Hello all,

I'm trying to override the unlink method in the class account_invoice. To be able to erase invoice when I make tests. 

I want the shortest possible code. What I do wrong?

from openerp import models, fields, api

class account_invoice(models.Model):
    _inherit = ['account.invoice']

 

    @api.multi
    def unlink(self):
        _inherit = ['unlink']
        for invoice in self:
            if invoice.state not in ('draft', 'cancel'):
                raise Warning(_('You cannot delete an invoice which is not draft or cancelled. You should refund it instead.'))
        return super(account_invoice, self).unlink()

 

thanks all

Portretas
Atmesti
Best Answer

Try this:

from openerp import models, fields, api
from openerp.exceptions import except_orm, Warning, RedirectWarning

class account_invoice(models.Model):
    _inherit = ['account.invoice']

    @api.multi
    def unlink(self):
        for invoice in self:
            if invoice.state not in ('draft', 'cancel'):
                raise Warning(_('You cannot delete an invoice which is not draft or cancelled. You should refund it instead.'))
        return models.Model.unlink(self)

PS. last line is not super()!!! because super() calls the method from account.invoice, which you want to replace.

Portretas
Atmesti
Autorius

Perfect! THANKS big big!

Related Posts Replies Rodiniai Veikla
5
vas. 22
10790
1
vas. 16
5417
1
spal. 15
4950
0
geg. 16
3408
2
gruod. 15
3482