From f079d7addd8e0021a2d323c0d2fdba38172d47c7 Mon Sep 17 00:00:00 2001 From: Thibault Francois Date: Sat, 1 Jul 2017 15:40:47 +0200 Subject: [PATCH] [FIX] [0024-Bug] cannot set two times the same responsible --- beesdoo_inventory/models/stock.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/beesdoo_inventory/models/stock.py b/beesdoo_inventory/models/stock.py index 5ee2cfd..cf09b0f 100644 --- a/beesdoo_inventory/models/stock.py +++ b/beesdoo_inventory/models/stock.py @@ -10,7 +10,10 @@ class StockPicking(models.Model): def _add_follower(self): if(self.responsible): types = self.env['mail.message.subtype'].search(['|',('res_model','=','stock.picking'),('name','=','Discussions')]) - self.env['mail.followers'].create({'res_model' : 'stock.picking', + if not self.env['mail.followers'].search([('res_id', '=', self.id), + ('res_model', '=', 'stock.picking'), + ('partner_id', '=', self.responsible.id)]): + self.env['mail.followers'].create({'res_model' : 'stock.picking', 'res_id' : self.id, 'partner_id' : self.responsible.id, 'subtype_ids': [(6, 0, types.ids)]}) @@ -23,7 +26,6 @@ class StockPicking(models.Model): @api.model def create(self, values): - print "Values", values picking = super(StockPicking, self).create(values) picking._add_follower() return picking