[FIX] website_project, website_sale: Compute correctly emails notification group

If a user is not set into a notification group, don't add him automatically in the done_ids

addons/website_project/models/project.py

@@ -26,7 +26,7 @@ class Project(models.Model):
                 continue
             if recipient.user_ids and all(recipient.user_ids.mapped('share')):
                 group_data['user'] |= recipient
-            done_ids.add(recipient.id)
+                done_ids.add(recipient.id)
         return super(Project, self)._notification_group_recipients(message, recipients, done_ids, group_data)
 
 
@@ -53,6 +53,6 @@ class Task(models.Model):
                 continue
             if recipient.user_ids and all(recipient.user_ids.mapped('share')):
                 group_data['user'] |= recipient
-            done_ids.add(recipient.id)
+                done_ids.add(recipient.id)
         return super(Task, self)._notification_group_recipients(message, recipients, done_ids, group_data)
 

addons/website_project_issue/models/project_issue.py

@@ -31,7 +31,8 @@ class Issue(models.Model):
                 continue
             if recipient.user_ids and group_project_user_id in recipient.user_ids[0].groups_id.ids:
                 group_data['group_project_user'] |= recipient
+                done_ids.add(recipient.id)
             elif recipient.user_ids and all(recipient.user_ids.mapped('share')):
                 group_data['user'] |= recipient
-            done_ids.add(recipient.id)
+                done_ids.add(recipient.id)
         return super(Issue, self)._notification_group_recipients(message, recipients, done_ids, group_data)

addons/website_sale/models/account_invoice.py

@@ -15,7 +15,7 @@ class AccountInvoice(models.Model):
                 continue
             if recipient.user_ids and all(recipient.user_ids.mapped('share')):
                 group_data['user'] |= recipient
-            done_ids.add(recipient.id)
+                done_ids.add(recipient.id)
         return super(AccountInvoice, self)._notification_group_recipients(message, recipients, done_ids, group_data)
 
     @api.multi

addons/website_sale/models/sale_order.py

@@ -131,7 +131,7 @@ class SaleOrder(models.Model):
                 continue
             if recipient.user_ids and all(recipient.user_ids.mapped('share')):
                 group_data['user'] |= recipient
-            done_ids.add(recipient.id)
+                done_ids.add(recipient.id)
         return super(SaleOrder, self)._notification_group_recipients(message, recipients, done_ids, group_data)