From 79eb0ad2fc3ed68d45378c102796911b1642c7ac Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20P=C3=A9pin?= Date: Wed, 13 Jul 2016 20:12:11 +0200 Subject: [PATCH] Nettoyage MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Moins de concaténations chaînes de caractères - Utilisation de `reverse_lazy` pour écrire des urls --- calendrier/urls.py | 2 +- calendrier/views.py | 60 +++++++++++++++++++++++---------------------- 2 files changed, 32 insertions(+), 30 deletions(-) diff --git a/calendrier/urls.py b/calendrier/urls.py index 6c7419d..5badbab 100644 --- a/calendrier/urls.py +++ b/calendrier/urls.py @@ -14,5 +14,5 @@ urlpatterns = patterns('calendrier.views', url(r'(?P\w{6})/(?P\w{6})/non', 'repnondir'), url(r'(?P\d+)/(?P\d+)/(?P\d+)/?', 'view_event'), url(r'(?P\d+)/(?P\d+)/?$', 'calendar'), - url(r'(?P\d+)/?', 'view_eventbis'), + url(r'(?P\d+)/?', 'view_eventbis', 'view-event'), ) diff --git a/calendrier/views.py b/calendrier/views.py index 3d98619..e1f5145 100644 --- a/calendrier/views.py +++ b/calendrier/views.py @@ -114,25 +114,25 @@ def resend(request, id): try: for user in users: if user.mails: - Subject = "[Pouet-Pouet] " + ev.nom + ", le " \ - + ev.date.strftime('%d/%m/%Y') + Subject = "[Pouet-Pouet] %s, le %s" % ( + ev.nom, ev.date.strftime('%d/%m/%Y')) sender = 'fanfare@ens.fr' receivers = [user.user.email] - text = "Bonjour, un évémenent a été créé." if message != '': text = message - text += '\n Pour voir l\'événement, allez sur ' \ - + 'http://www.ernestophone.ens.fr/calendar/' \ - + str(ev.id) + '\n' - text += '\nPour répondre oui, allez sur ' \ - + 'http://www.ernestophone.ens.fr/calendar/' \ - + user.slug + '/' + ev.slug + '/oui' - text += '\nPour répondre non, allez sur ' \ - + 'http://www.ernestophone.ens.fr/calendar/' \ - + user.slug + '/' + ev.slug + '/non' - text += '\nPour répondre peut-être, allez sur ' \ - + 'http://www.ernestophone.ens.fr/calendar/' \ - + user.slug + '/' + ev.slug + '/pe' + else: + text = "Bonjour, un évémenent a été créé.\n" + text += 'Pour voir l\'événement, allez sur %s\n' \ + + 'Pour répondre oui allez sur %s\n' \ + + 'Pour répondre non, allez sur %s\n' \ + + 'Pour répondre peut-être, allez sur %s\n' \ + % (reverse_lazy('view-event', args=[ev.id]), + reverse_lazy('repouidir', args=[ + user.slug, ev.slug]), + reverse_lazy('repnondir', args=[ + user.slug, ev.slug]), + reverse_lazy('reppedir', args=[ + user.slug, ev.slug])) send_mail(Subject, text, sender, receivers) return redirect("calendrier.views.view_eventbis", id) except: @@ -184,24 +184,26 @@ def create_event(request): for user in users: if user.mails: try: - Subject = "[Pouet-Pouet] " + nom + ", le " + date + Subject = "[Pouet-Pouet] %s, le %s" % (nom, date) sender = 'fanfare@ens.fr' receivers = [user.user.email] - text = "Bonjour, un évémenent a été créé." if message != '': text = message - text += '\n Pour voir l\'événement, allez sur ' - text += 'http://www.ernestophone.ens.fr/calendar/' - text += str(obj.id) + '\n' - text += '\nPour répondre oui, allez sur ' - text += 'http://www.ernestophone.ens.fr/calendar/' - text += user.slug + '/' + obj.slug + '/oui' - text += '\nPour répondre non, allez sur ' - text += 'http://www.ernestophone.ens.fr/calendar/' - text += user.slug + '/' + obj.slug + '/non' - text += '\nPour répondre peut-être, allez sur ' - text += 'http://www.ernestophone.ens.fr/calendar/' - text += user.slug + '/' + obj.slug + '/pe' + else: + text = "Bonjour, un évémenent a été créé.\n" + text += "Pour voir l'événement, allez sur %s" \ + + '\nPour répondre oui allez sur %s\n' \ + + 'Pour répondre non, allez sur %s\n' \ + + 'Pour répondre peut-être, allez sur %s' \ + + '\n' \ + % (reverse_lazy('view-event', args=[ + obj.id]), + reverse_lazy('repouidir', args=[ + user.slug, obj.slug]), + reverse_lazy('repnondir', args=[ + user.slug, obj.slug]), + reverse_lazy('reppedir', args=[ + user.slug, obj.slug])) send_mail(Subject, text, sender, receivers) except: erreur = "Une erreur est survenue, le mail n'a " \