Merge PR #730 into 13.0

Signed-off-by pedrobaeza

mail_activity_team/models/mail_activity.py

@@ -63,10 +63,12 @@ class MailActivity(models.Model):
         # SUPERUSER is inactive and then even if you add it
         # to member_ids it's not taken account
         # To not be blocked we must add it to constraint condition
+        # We must consider also users that could be archived but come from
+        # an automatic scheduled activity
         for _activity in self.filtered(
             lambda a: a.user_id.id != SUPERUSER_ID
             and a.team_id
             and a.user_id
-            and a.user_id not in a.team_id.member_ids
+            and a.user_id not in a.team_id.with_context(active_test=True).member_ids
         ):
             raise ValidationError(_("The assigned user is not member of the team."))

mail_activity_team/models/mail_activity_mixin.py

@@ -21,8 +21,8 @@ class MailActivityMixin(models.AbstractModel):
                 .with_context(default_res_model=self._name,)
                 ._get_default_team_id(user_id=user_id)
             )
-            if team:
-                act_values.update({"team_id": team.id})
+            # Even if it comes empty, we don't want to mismatch the user's team
+            act_values.update({"team_id": team.id})
         return super().activity_schedule(
             act_type_xmlid=act_type_xmlid,
             date_deadline=date_deadline,