Nettoyage

- Moins de concaténations chaînes de caractères
- Utilisation de `reverse_lazy` pour écrire des urls
This commit is contained in:
Martin Pépin 2016-07-13 20:12:11 +02:00
parent 438be7e481
commit 79eb0ad2fc
2 changed files with 32 additions and 30 deletions

View file

@ -14,5 +14,5 @@ urlpatterns = patterns('calendrier.views',
url(r'(?P<codeus>\w{6})/(?P<codeev>\w{6})/non', 'repnondir'),
url(r'(?P<pYear>\d+)/(?P<pMonth>\d+)/(?P<id>\d+)/?', 'view_event'),
url(r'(?P<pYear>\d+)/(?P<pMonth>\d+)/?$', 'calendar'),
url(r'(?P<id>\d+)/?', 'view_eventbis'),
url(r'(?P<id>\d+)/?', 'view_eventbis', 'view-event'),
)

View file

@ -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 " \