Skip to Content
This question has been flagged
1 Reply
10065 Views
Avatar
Zakaria

Hello,

I have Override the unlink function of my class, as follows:

@api.multi
def unlink(self):

    #record = self.env['affichage2.pack_stock'].search(['public_serial','=',self.public_serial])

    for record in self:

      p = self.env['affichage2.pack_stock'].browse(record.id)

      if (p.vente != None or p.vente2 != None):

         raise osv.except_osv(('Invalid Action!'), ('Le pack est déjà affecté ! Vous ne pouvez pas le supprimer',p.vente2,p.vente))


    rec = super(Pack_Stock, self).unlink()
    return rec

This function shows a warning  if the records "vente" and "vente2" are not empty. The problem is the verification didn't made well. The function didn't run as i want. How i can access to the values of these  2 fields from the Unlink function ?


Avatar
Discard
Avatar
Yogendrasinh Chavda
Best Answer

Hello Zakaria,

No need to check condition with "None", Try below code.
make sure this function is override in affichage2.pack_stock Class.


@api.multi def unlink(self):

    # record = self.env['affichage2.pack_stock'].search(['public_serial','=',self.public_serial])

    for record in self:

      # p = self.env['affichage2.pack_stock'].browse(record.id)

      # You got recordset so no need to browse it again 

      if not record.vente or not record.vente2:

         raise osv.except_osv(('Invalid Action!'), ('Le pack est déjà affecté ! Vous ne pouvez pas le supprimer',p.vente2,p.vente))

    return super(Pack_Stock, self).unlink()

Thanks

Avatar
Discard
Zakaria
Author

Thank you Chavda, but i have tried it like that, it not working also. That is because the condition IF didn't get the values of "vente2" and "vente". I have declared these values as follows :

vente = fields.Many2one('res.users', 'Vendu à Distributeur')

vente2 = fields.Many2one('res.users', 'Vendu à Client')

Yogendrasinh Chavda

make sure vente or vente2 have value in the record.

Zakaria
Author

They are empty, and when they are empy i want to restrict the delete.

Yogendrasinh Chavda

Ok

then put condition

if not record.vente or not record.vente2:

This condition will raise exception when both or one of them is empty

Zakaria
Author

Ok i will try it, thank you Bro

Zakaria
Author

I have tried it, and still the same ..

Zakaria
Author

I resloved it, i made this in condition :

if ( len(record.vente)!=0 or len(record.vente2)!=0) :