Commit 0a5c028a authored by Remi Oudin's avatar Remi Oudin Committed by Hamza Dely

On vérifie si le mail existe.

On vérifie le retour de ```adh.get_mail()``` pour éviter de tenter
d'envoyer un mail à None. De plus, meilleure gestion d'une annulation
d'envoi.
parent 9a11fd52
......@@ -33,14 +33,26 @@ def mail_sender(template, From, ldap_filter, recipientfile=None):
else:
# Récupère les mails des adhérents donnés par le filtre.
for adh in conn.search(ldap_filter, sizelimit=2000):
mailaddrs.add(adh.get_mail())
mail = adh.get_mail()
if mail:
mailaddrs.add(mail)
else:
if not SEND:
cprint("No mail known for %s %s" % \
(adh["prenom"][0], adh["nom"][0]),
'rouge'
)
echecs = []
if PREV:
print "Envoi simulé"
print "%d destinataires (Ctrl + C pour annuler l'envoi)" %len(mailaddrs)
raw_input("Envoyer ? (Ret pour envoyer)")
try:
print "%d destinataires (Ctrl + C pour annuler l'envoi)" %len(mailaddrs)
raw_input("Envoyer ? (Ret pour envoyer)\n")
except KeyboardInterrupt:
cprint("\nEnvoi annulé.", 'rouge')
exit(1)
with mail.ServerConnection() as conn_smtp:
for To in mailaddrs:
cprint(u"Envoi du mail à %s" % To)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment