diff options
| author | Alban Gruin | 2017-02-06 18:56:28 +0100 | 
|---|---|---|
| committer | Alban Gruin | 2017-02-06 18:56:28 +0100 | 
| commit | 421748e13e41dc24a6dd53ef9e440278c1625957 (patch) | |
| tree | 577e45f24884dfa4156e418585119aa82c9997a9 | |
| parent | 56edcf086a2020cc6ad934d472b9edde1f7650a5 (diff) | |
Ne retourne pas de 404 si le token n'existe pas
| -rw-r--r-- | views.py | 14 | 
1 files changed, 8 insertions, 6 deletions
| @@ -83,12 +83,14 @@ def confirm_subscription(request, token):      return redirect("timetable", timetable_slug=subscription.group.timetable.slug, group_slug=subscription.group.slug, year=year, week=week)  def cancel_subscription(request, token): -    subscription = get_object_or_404(Subscription, token=token) -    group = subscription.group -    subscription.delete() +    subscription = Subscription.objects.filter(token=token).first() + +    if subscription is not None: +        group = subscription.group +        subscription.delete() -    template = loader.get_template("mail/mail_unsubscribed.txt") -    context = Context({"group": group}) -    send_mail("Confirmation de la désinscription", template.render(context), settings.DEFAULT_FROM_EMAIL, [subscription.email]) +        template = loader.get_template("mail/mail_unsubscribed.txt") +        context = Context({"group": group}) +        send_mail("Confirmation de la désinscription", template.render(context), settings.DEFAULT_FROM_EMAIL, [subscription.email])      return redirect("index") | 
