Problème d'envoi de mail avec formulaire Ajax +Json

Marsh Posté le 28-12-2014 à 19:18:13

Bonjour,

ca fait quelques jours que je tourne en rond pour l'envoi de mail via formulaire (le truc basique quoi).
Mais cette fois ci j'ai rajouté de l'Ajax et Json pour l'envoi des données au server.

Problème: les mail de tests n'arrive plus depuis quelques jours.
Au début je pensais à un soucis de code, mais je suis de plus en plus sceptique et me demande si l'adresse ip de mon host n'est pas blacklistée, ceci dit
je n'ai aucun message d'erreur en retour, rien.
A vous de juger

Le code Jquery:

Code :

<script type="text/javascript">
$('#contact').on('submit', function(e)
{
e.preventDefault();
var error = false;
//simple validation at client's end
//loop through each field and we simply change border color to red for invalid fields
$("#contact input,#contact textarea" ).each(function(){
$(this).css('border-color','');
if(!$.trim($(this).val()))
{ //if this field is empty
$(this).css('border-color','red'); //change border color to red
error = true; //set do not proceed flag
}
//check invalid email
var email_reg = /^([w-.]+@([w-]+.)+[w-]{2,4})?$/;
if($(this).attr("type" )=="email" && !email_reg.test($.trim($(this).val())))
{
$(this).css('border-color','red'); //change border color to red
error = true; //set do not proceed flag
}
});
if(!error) //everything looks good! proceed...
{
var $this = $(this);
$.ajax({
url: $this.attr('envoi_mail.php'), // Le nom du fichier indiqué dans le formulaire
type: $this.attr('POST'), // La méthode indiquée dans le formulaire (get ou post)
data: $this.serialize(), // Je sérialise les données (j'envoie toutes les valeurs présentes dans le formulaire)
dataType: 'json', // JSON
success: function(json)
{
if(json.message_sent == true)
{
$('#contact_results').show();
$('#contact_results').addClass('.success_message');
$('#contact_results').text('Mail envoyé !');
$("form input,form textarea" ).val('');
} else {
if((json.message_sent == false) && (json.error == false) )
{
$('#contact_results').show();
$('#contact_results').addClass('.error_message');
$('#contact_results').text('Erreur lors de l'envoi n
de votre message: raison indéterminée');
$("form input,form textarea" ).val('');
}
else
{
$('#contact_results').addClass('.error_message');
$('#contact_results').text('Error: les infos rentrées dans le formulaire semblent incorrectes');
}
}
}
});
}
};
</script>

Le formulaire:

Code :

<div id="form_contact">
<div id="contact_results">
</div>
<form action="envoi_mail.php" id="contact" method="POST">
<p>
<label for="prenom" class="prenom">Prénom</label>
<br /><input id="prenom" name="prenom" type="text" autofocus>
</p>
<p>
<label for="nom" class="nom">Nom</label>
<br /><input id="nom" name="nom" type="text" autofocus>
</p>
<p>
<label for="email">Email</label>
<br /><input id="email" name="email" type="email" autofocus>
</p>
<p>
<label for="sujet" class="sujet">Sujet</label>
<br /><input id="sujet" name="sujet" type="text" autofocus>
</p>
<p>
<label for="message">Message</label>
<br /><textarea id="message" name="message" rows="10" cols="80" autofocus></textarea>
</p>
<p>
<input type="submit" class='bouton_noir' value="Envoyer" />
</p>
</form>
</div>

le code PHP:

Code :

<?php
if(isset($_POST["user_prenom"]) && isset($_POST["user_nom"])
&& isset($_POST["user_email"]) && isset($_POST["user_subject"])
&& isset($_POST["user_message"]))
{
// array to pass back data
$user_datas =[];
$return_infos = [];
$to_email = "mon adresse@yahoo.fr"; //Recipient email, Replace with own email here
$input_error = false;
$message_sent = false;
$user_datas = [
"prenom" => $_POST["user_prenom"],
"nom" => $_POST["user_nom"],
"email" => $_POST["user_email"],
"sujet" => $_POST["user_subject"],
"message" => $_POST["user_message"],
];
//var_dump($user_datas);
/*$user_prenom = filter_input(INPUT_POST, $_POST["user_prenom"], FILTER_SANITIZE_STRING);
$user_nom = filter_input(INPUT_POST, $_POST["user_nom"], FILTER_SANITIZE_STRING);
$user_email = filter_input(INPUT_POST, $_POST["user_email"], FILTER_SANITIZE_EMAIL);
$user_sujet = filter_input(INPUT_POST, $_POST["user_subject"], FILTER_SANITIZE_STRING);
$user_message = filter_input(INPUT_POST, $_POST["user_message"], FILTER_SANITIZE_STRING); */
foreach($user_datas as $k => $v)
{
if(is_string($user_datas[$v]))
{
$input_error = false;
}
else
{
$input_error = true;
break;
}
}
//var_dump($user_datas);
// return a response ===========================================================
// if there are any errors in our errors array, return a success boolean of false
if (!($input_error))
{
$user_datas_ok=$user_datas;
$headers = 'From: '.$user_datas_ok["email"]. "rn" .
'Reply-To: '.$to_email. "rn" .
'X-Mailer: PHP/' .phpversion();
if( mail($to_email, "dkdkdkdkdkdkkdkd",
",c,d,c,c,c,", 'HAL <HAL@hotmail.com>')) // tentative d'envoi du message
{
$message_sent=true;
}
else{
$message_sent=false;
}
}
header('Content-type: application/json;charset=utf-8');
echo json_encode([
'message_sent' => $message_sent,
'error' => $input_error
]);
}

Voyez vous un probleme dans mon code, qui expliquerait que le message n'arrive pas ?

Merci d'avance

Tom

---------------
collectionneur de pâtes thermiques

Reply

Marsh Posté le 28-12-2014 à 19:18:13

Reply

Marsh Posté le 29-12-2014 à 09:18:30

Bonjour,

Je pense que ton soucis viens de ta partie JS/AJAX et/ou PHP.
Tu envoie this.serialize(); mais je pense que ceci ne récupère pas le contenu de tes inputs. Un console.log sur this.serialize() s'impose.

Ensuite partie PHP tu fait utilise ces $_POST:

Code :

if(isset($_POST["user_prenom"]) && isset($_POST["user_nom"])
&& isset($_POST["user_email"]) && isset($_POST["user_subject"])
&& isset($_POST["user_message"]))

Personnellement je ne vois rien avec ces noms là ( peut-être dans ton this.serialize();

Pour finir dans ta function ajax : success ( tu devrait enlever tout les if pour avoir l'affiche des erreurs php pour débuguer )

et aussi mettre la function ajax error

Code :

error : function(rep){
}

ainsi en qu'a d'erreur de ta requête tu peux déboguer aussi.

En espérant que ça t'aiguille vers la solution à ton problème.

---------------
Made you your own sentence without believing that of the others...

Reply

Marsh Posté le 30-12-2014 à 15:58:26

Bonjour,

Je n'ai pas vraiment regardé ton code mai ça me fait penser à ce sujet :

http://forum.hardware.fr/hfr/Progr [...] 2920_1.htm

Reply

Marsh Posté le 30-12-2014 à 16:20:28

Comme le faisait remarquer OrcusZ, je doute fort que ton code ait marché avant à la vue des noms de variables dans les isset($POST['...'])

Ca me fait plus penser à qq'un qui a récupérer des boutons de code par-ci par-là et a essayé de les assembler sans les comprendre

Dans ton php, si tu fais un print_r($_POST), tu vois quoi ?

Edit : appréciation toute personnelle : je ne vois vraiment pas l'utilité d'ajax pour un simple formulaire de contact

Message édité par rufo le 30-12-2014 à 16:21:26

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

Reply

Marsh Posté le 31-12-2014 à 10:26:50

Allélouia ca marche enfin.

J'ai dormi pendant 5 jours et je viens de débuguer :whistle:
Il y avait des erreurs de syntaxes et d'autres grosses comme des maisons (le coup de $POST était énorme )

dans tous les coins

@ rufo : effectivement pas beaucoup d'intéret mais c'est surtout pour me faire la main avec ajax

Message édité par tompouss le 31-12-2014 à 10:28:11

---------------
collectionneur de pâtes thermiques

Reply

Marsh Posté le 31-12-2014 à 10:38:29

Ajax peut être une vraie plaie pour l'accessibilité des sites web aux handicapés visuels Leur lecteur d'écran ne détecte aps forcément quand une portion de la page a changé de contenu suite à une requête ajax contrairement à l'envoi d'un formulaire u un clic sur un lien qui envoie l'utilisateur sur une nouvelle url : là, le lecteur d'écran n'a pas de pb

L'autre jour, j'entends que seulement 5 à 7% des sites web étaient accessibles à ces personnes

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

Reply

Marsh Posté le 31-12-2014 à 11:39:33

Hummm, ça serait cool si on pouvais connaitre la nature de l'écran de l'utilisateur pour ce genre de cas...

En tout cas bravo si tu as résolu ton problème

Message cité 1 fois
Message édité par OrcusZ le 31-12-2014 à 11:39:58

---------------
Made you your own sentence without believing that of the others...

Reply

Marsh Posté le 31-12-2014 à 11:48:55

OrcusZ a écrit :

Hummm, ça serait cool si on pouvais connaitre la nature de l'écran de l'utilisateur pour ce genre de cas...

En tout cas bravo si tu as résolu ton problème

L'écran d'une personne malvoyante est le même que le tien. Elle a juste un soft de lecture du contenu de l'écran (ex : NVDA) qui lui lit la page affichée ou ce qui se trouve sous le curseur de la souris ou au niveau du curseur du clavier si elle utilise les raccourcis clavier pour naviguer (ce qui est souvent le cas).

D'où l'intérêt de mettre de bons libellés explicites sur les liens et dans l'attribut "title" pour l'aider dans son choix des liens à cliquer (en effet, le soft regroupe dans une sorte de menu, tous les liens de la page).
D'où l'intérêt d'utiliser les bonnes balises html pour leur sémantique et non leur rendu visuel (ex : <th> pour un entête de colonne de tableau et non un simple <td> car à chaque passage à la ligne du tableau, le soft relit l'entête de la colonne).
...

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

Reply

Marsh Posté le 31-12-2014 à 13:25:00

Re,

Merci pour l'information je ne le savais pas et ferais encore plus attention qu'avant

---------------
Made you your own sentence without believing that of the others...

Reply

Marsh Posté le 31-12-2014 à 14:18:56

Un lien caché pour aller direct au contenu de la page est très appréciable aussi. Sinon, l'utilisateur se tape la lecture du menu principal et contextuel du site à chaque fois. Ca devient très vite lourd. Pour s'en convaincre, installez-vous NVDA ( http://www.nvda-fr.org/download.php ) sur votre PC, fermez les yeux (encore que c'est pas obligé) et essayez de naviguer sur votre site sans souris (qu'avec le clavier).
Si vous arrivez à utiliser votre site sans pb, parfait, votre site est accessible. Sinon, vous allez très vite voir ce qui merde Et je vous parle même pas quand il s'agit non plus d'un site type "vitrine" ou blog mais d'un intranet.

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

Reply

Problème d'envoi de mail avec formulaire Ajax +Json

Sujets relatifs:

Leave a Replay