improved on sourcery suggestion

Author: Gagan-Ram

app/eventyay/plugins/sendmail/forms.py

@@ -512,10 +512,16 @@ def clean_emails(self):
             if email.strip()
         ]
 
-        if len(updated_emails) > len(self.recipient_objects):
+        if len(updated_emails) == 0:
             raise ValidationError(
-                _("You cannot add new recipients. Only editing existing email addresses is allowed.")
+                _("At least one recipient must remain. You cannot remove all recipients.")
             )
+
+        if len(updated_emails) != len(self.recipient_objects):
+            raise ValidationError(
+                _("You cannot add new recipients or remove recipients. Only editing existing email addresses is allowed.")
+            )
+
         return updated_emails
 
     def save(self, commit=True):

app/eventyay/plugins/sendmail/models.py

@@ -139,10 +139,7 @@ def _build_email_context(self, order, position, position_or_address, recipient):
             return None
 
     def _finalize_send_status(self):
-        if all(r.sent for r in self.recipients.all()):
-            self.sent_at = now()
-        else:
-            self.sent_at = None
+        self.sent_at = now() if all(r.sent for r in self.recipients.all()) else None
         self.save(update_fields=["sent_at"])
 
     def _send_to_recipient(self, recipient, subject, message):