diff --git a/kfet/management/commands/sendrappelsnegatifs.py b/kfet/management/commands/sendrappelsnegatifs.py index 58c47d4e..4e2f67c3 100644 --- a/kfet/management/commands/sendrappelsnegatifs.py +++ b/kfet/management/commands/sendrappelsnegatifs.py @@ -19,23 +19,31 @@ class Command(BaseCommand): def handle(self, *args, **options): now = timezone.now() + + # Le premier mail est envoyé après 24h de négatif, puis toutes les semaines first_delay = timedelta(days=1) periodic_delay = timedelta(weeks=1) - accounts_first_mail = ( - AccountNegative.objects.filter(start__lt=now - first_delay) - .filter(last_rappel__isnull=True) - .all() + + # On n'envoie des mails qu'aux comptes qui ont un négatif vraiment actif + # et dont la balance est négative + account_negatives = AccountNegative.objects.filter( + account__balance__lt=0 + ).exclude(end__lte=now) + + accounts_first_mail = account_negatives.filter( + start__lt=now - first_delay, last_rappel__isnull=True ) - accounts_periodic_mail = ( - AccountNegative.objects.filter(last_rappel__isnull=False) - .filter(last_rappel__lt=now - periodic_delay) - .all() + accounts_periodic_mail = account_negatives.filter( + last_rappel__lt=now - periodic_delay ) + for neg in accounts_first_mail: neg.send_rappel() self.stdout.write(f"Mail de rappel pour {neg.account} envoyé avec succès.") + for neg in accounts_periodic_mail: neg.send_rappel() self.stdout.write("Mail de rappel pour {neg.account} envoyé avec succès.") - if (not accounts_first_mail) and (not accounts_periodic_mail): + + if not (accounts_first_mail.exists() or accounts_periodic_mail.exists()): self.stdout.write("Aucun mail à envoyer.") diff --git a/kfet/models.py b/kfet/models.py index 7a0ec777..a71b53cd 100644 --- a/kfet/models.py +++ b/kfet/models.py @@ -260,8 +260,10 @@ class Account(models.Model): elif hasattr(self, "negative"): if self.negative.end is None: self.negative.end = timezone.now() + self.negative.save() elif timezone.now() > self.negative.end + kfet_config.cancel_duration: # Idem: on supprime le négatif après une légère période + # Nécessaire pour se souvenir du négatif après une charge annulée self.negative.delete() class UserHasAccount(Exception): diff --git a/kfet/templates/kfet/left_account.html b/kfet/templates/kfet/left_account.html index aa22cb03..98f0aa21 100644 --- a/kfet/templates/kfet/left_account.html +++ b/kfet/templates/kfet/left_account.html @@ -53,7 +53,7 @@ - {% if account.negative %} + {% if account.negative and account.balance_ukf < 0 %}

Négatif