search

nicmarti / LesZindeps

Projet pour le site leszindeps.fr
www.leszindeps.fr
Other
11 stars 4 forks source link

Contact membre #3

Closed jblemee closed 13 years ago

jblemee commented 13 years ago

Le formulaire de contact d'un membre renvoie sur une erreur. Le message n'est jamais reçu.

lauvigne commented 13 years ago

je viens de tester sur mon profil est cela fonctionne pour moi.

Sur le profil de Jean baptiste j'ai :

Oops, an error occured This exception has been logged with id 65bl1k17m.

lauvigne commented 13 years ago

Jean Baptiste, est ce que tu as une adresse mail secondaire dans ton profil ?

Car à la lecture du code il y celui la qui pourrait lancé une exception non prévue : Mails.sendMessageToUser(message,zindep.email); if(zindep.emailBackup!=null){ Mails.sendMessageToUser(message,zindep.emailBackup); }

Profils qui ont aussi le même problème : Thierry Balla Jean Louis Boudart

Fonctionne pour moi et Nico Chéné!

lauvigne commented 13 years ago

Après réponse de Jean Louis, il a bien reçu le mail mais j'ai eu une page d'erreur, à suivre ...

jblemee commented 13 years ago

Non, je n'ai pas renseigné mon email secondaire.

jblemee commented 13 years ago

Rahh le boulet, j'ai close l'issue.. Je ne trouve pas comment réouvrir.

lauvigne commented 13 years ago

C'est bon, j'ai corrig en local. A+

2011/3/11 jblemee < reply@reply.github.com>

Rahh le boulet, j'ai close l'issue.. Je ne trouve pas comment rouvrir.

https://github.com/nicmarti/LesZindeps/issues/3#comment_861597

nicmarti commented 13 years ago

Salut

D'après la console PlayApps c'est parce que l'adresse email serait invalide... Bizarre. Est-ce que tu as mis une adresse ou est-ce qu'elle est vide ?

Nicolas

nicmarti commented 13 years ago

J'ai ajouté un bout de code pour vérifier que emailBackup est bien un email valide. On retestera

Nicolas

jblemee commented 13 years ago

J'ai un email vide. Que j'ai laissé vide pour tester.

lauvigne commented 13 years ago

Je viens de tester et j'ai le même style d'erreur C'est relativement simple à tester, supprimer sont email secondaire et on a l'erreur.

Dans le nouveau code, il faut sortir du cas si l'email est vide : if(zindep.emailBackup!=null) { if(zindep.emailBackup.trim().equals("")){ // email invalide. J'en profite pour le nettoyer et sauver la fiche proprement zindep.emailBackup=null; zindep.save(); } else { // Verifie l'email validation.email(zindep.emailBackup); // si le mail est valide alors envoie la copie if(!validation.hasErrors()){ Mails.sendMessageToUser(message,zindep.emailBackup); } } }

nicmarti commented 13 years ago

Oups Ok j'ai corrigé et redéployé la nouvelle version ce matin

Merci pour le coup de main

Nicolas

Le 18 mars 2011 19:26, lauvigne reply@reply.github.com a écrit :

Je viens de tester et j'ai le même style d'erreur C'est relativement simple à tester, supprimer sont email secondaire et on a l'erreur.

Dans le nouveau code, il faut sortir du cas si l'email est vide : if(zindep.emailBackup!=null){            if(zindep.emailBackup.trim().equals("")){               // email invalide. J'en profite pour le nettoyer et sauver la fiche proprement               zindep.emailBackup=null;               zindep.save();            } else {            // Verifie l'email            validation.email(zindep.emailBackup);            // si le mail est valide alors envoie la copie              if(!validation.hasErrors()){                  Mails.sendMessageToUser(message,zindep.emailBackup);              }           }        }

Reply to this email directly or view it on GitHub: https://github.com/nicmarti/LesZindeps/issues/3#comment_890384