Array PHP email

Version imprimable

Array PHP email

Bonjour à tous,

J'ai un souci au niveau de mon code et je ne comprends pas l'origine de ce problème. En effet, j'ai créé une fonction PHP contenant un tableau avec une liste des mots interdits. Il s'agit d'un système de question/réponse. Si un client saisit un mot interdit dans sa réponse, je vérifie que ce mot ou expression est dans mon tableau. Si le mot existe et est interdit j'envoie un émail à l'administrateur pour le signaler, sinon j'enregistre la réponse et je l'affiche. Même dans le cas ou le mot est interdit j'enregistre quand même la réponse et c'est juste au niveau de l'affichage que je remplace le mot par des étoiles (***). L'envoie de mail c'est pour informer l'administrateur qu'un mot interdit a été saisi et lui permettre de le modérer ou de le supprimer.

Voici mon script :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 // fonction mots interdits: function motInterdit ($mot) { $mots = array( 'mot 1', 'mot 2', 'mot 3' ); $mot_remplacement = '******'; $data = str_ireplace($mots, $mot_remplacement, $mot); return $data; }
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 // requête d'insertion ici /*.........*/ // select $q = array('lastId'=>$lastId); $sql = 'SELECT * FROM reponse WHERE reponse_id = :lastId'; $req = $db->prepare($sql); $req->execute($q) or die(print_r($db->errorInfo())); foreach ($req->fetchAll() as $data): $tab = nl2br(htmlspecialchars($data)); $resTab = motInterdit ($tab); if (is_array(resTab) && in_array($tab, $resTab)) { // envoie de mail mail($to, $subject, $message, $headers); } endforeach;
Je pense que mon problème c'est au niveau de la vérification si le mot interdit existe dans mon tableau. Car je n'arrive pas à envoyer mon mail. Mais je n'ai pas de problème au niveau du traitement de mon mail car si je fais un test d’envoi de mail avec un mot correcte ça fonctionne.

Merci pour votre aide !

20/09/2016, 12h07
Celira

Attention à la faute de frappe : il manque un $ devant le 1er resTab ici : if (is_array(resTab) && in_array($tab, $resTab)) {.
Activer la totalité des erreurs et des notices pendant les phases de développement est une bonne idée et pourrait t'aider à repérer ce genre d'erreurs ;)
20/09/2016, 12h12
mOscar007

Bonjour @Celira,

Merci pour la remarque.
Oui en effet c'est une erreur de frappe. Mais ce n'est pas ça mon problème.
Le message d'erreur que j'ai est celui là et rien d'autres :

Array
(
[0] => 00000
[1] =>
[2] =>
)
1

Cdlt,
20/09/2016, 13h40
Celira

Ah, donc ce qui plante, c'est ta requête (vu que c'est le seul endroit où je vois un print_r dans ton bout de code)
Ton abscence d'informations vient du fait que tu n'appelles pas la bonne fonction errorInfo :

Citation:

Si vous créez un objet PDOStatement avec la fonction PDO::prepare() ou la fonction PDO::query() et que vous invoquez une erreur sur le gestionnaire de requête, PDO::errorInfo() ne retournera pas l'erreur depuis le gestionnaire de requête. Vous devez appeler la fonction PDOStatement::errorInfo() pour retourner les informations sur l'erreur pour une opération exécutée sur un gestionnaire de requête particulier.

Essaye donc avec :

Code:

$req->execute($q) or die(print_r($req->errorInfo()));
20/09/2016, 14h42
mOscar007

@Celira,

J'ai toujours la même erreur.

Cdlt,
21/09/2016, 12h25
mOscar007

Re,

Tout ce que j'ai comme erreur c'est ça après plusieurs debug :

Array
(
[0] => HY000
[1] => 206
[2] => General SQL Server error: Check messages from the SQL Server [206] (severity 16) [(null)]

[3] => -1
[4] => 16
)

Cdlt,
21/09/2016, 14h23
Celira

Ah on avance : tu as bien un plantage dans l'exécution de ta requête.
Je pense que le problème vient des paramètres passés à la requête : il manque le ":" devant le nom du paramètre $q = array(':lastId'=>$lastId);. Évidemment, il faut aussi qu'il y ait quelque chose de correct dans $lastId ;)
21/09/2016, 14h45
mOscar007

L'erreur vient du fait que j'ai remplacé ce bout de code :

Code:

$req->execute($q) or die(print_r($req->errorInfo()));

Par :

Code:

$req->execute($q) or die(print_r($db->errorInfo()));

.

Si je remets $db je n'ai plus le message d'erreur précédent mais j'ai celui là :
Array
(
[0] => 00000
[1] =>
[2] =>
)
1

Donc on revient à la phase initiale.

Cdlt,

Effectivement, mais ne pas afficher le message d'erreur, ce n'est pas franchement la même chose que corriger l'erreur :roll:

Au départ, avec ce code :
Code:

1 2 3 4 $q = array('lastId'=>$lastId); $sql = 'SELECT * FROM reponse WHERE reponse_id = :lastId'; $req = $db->prepare($sql); $req->execute($q) or die(print_r($db->errorInfo()));
tu avais deux problèmes : ça ne marchait pas et on n'avait pas le message d'erreur.

Maintenant, avec ce code :
Code:

1 2 3 4 $q = array('lastId'=>$lastId); $sql = 'SELECT * FROM reponse WHERE reponse_id = :lastId'; $req = $db->prepare($sql); $req->execute($q) or die(print_r($req->errorInfo()));
ça ne marche toujours pas, mais on a un message d'erreur qui dit (en gros) que ta requête ne peut s'exécuter.

Donc, ensuite, on tente de corriger cette erreur. Pour cela, je te propose d'essayer ce code :
Code:

1 2 3 4 $q = array(':lastId'=>$lastId); $sql = 'SELECT * FROM reponse WHERE reponse_id = :lastId'; $req = $db->prepare($sql); $req->execute($q) or die(print_r($req->errorInfo()));

Avec

Code:

$q = array(':lastId'=>$lastId);

c'est toujours le statu quo.

Pour contextualiser à nouveau ce que je souhaite faire, j'ai un formulaire avec un champ texarea dans lequel un client peut saisir une réponse à une question posée précédemment. J'ai créé une fonction (motInterdit => code ci-dessus) qui contient elle-même un tableau avec une liste de mots ou expressions bannies. Quand une réponse est saisie je vérifie dans mon array qu'elle ne contient pas un des mots ou expressions. Si le test est positif j'envoie dans ce cas un mail à l'admin pour lui signaler et sinon j'envoie un autre mail à celui qui a posé sa question que elle a reçu de réponse. Dans tous les cas de figure j'enregistre la réponse dans la BD et les mots non désirés sont remplacés par des astérisque (***) pour l'affichage.

Code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 // fonction mots interdits: function motInterdit ($mot) { $mots = array( 'mot 1', 'mot 2', 'mot 3' ); $mot_remplacement = '******'; $data = str_ireplace($mots, $mot_remplacement, $mot); return $data; } // Mon insert /* .............. */ // select $q = array(':lastId'=>$lastId); $sql = 'SELECT * FROM reponse WHERE reponse_id = :lastId'; $req = $db->prepare($sql); $req->execute($q) or die(print_r($req->errorInfo())); $rows = $req->fetchAll(); foreach ($rows as $row): $tab = nl2br(htmlspecialchars($row['reponse_message'])); $resTab = motInterdit($tab); if (is_array($resTab) && in_array($tab, $resTab)) { // envoie de mail mail($to, $subject, $message, $headers); } else { // envoie de mail mail($to, $subject, $message, $headers); } endforeach;
Dans le cas de saisie d'un mot normal = non interdit (condition else), l'envoie de mail marche bien et je récupère mes infos avec les mêmes requêtes précédentes. Donc c'est au niveau de cette condition qu'il y a un problème :

Code:

if (is_array($resTab) && in_array($tab, $resTab)) {}

.

Mon envoi de mail ne marche pas à ce niveau et que j'ai un message d'erreur. Hors dans le cas de else je n'ai pas de message d'erreur pourtant j'utilise la même requête, ce que je ne comprends pas.

Cdlt,

21/09/2016, 16h12
Celira

Si la requête ne marche pas, vu que tu interceptes l'erreur avec un die, ni ta vérification de mots, ni ton envoi de mail ne vont être exécutés.
Si la même requête exactement fonctionne ailleurs dans le code, c'est qu'il y a un problème autre la requête. Par exemple, la valorisation de $lastId.

Peut-on voir un chouia plus de code ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
// requête d'insertion
$message = $_REQUEST['message'];
$client_id = $_SESSION['client_id'];
 
$q = array(':message'=>$message, ':id'=>$client_id);
$sql = 'INSERT INTO reponse (reponse_message, reponse_date, client_id) VALUES (:message, GETDATE(), :id)';
$req = $db->prepare($sql);
$req->execute($q) or die(print_r($req->errorInfo()));
 
$lastId = (int) $db->lastInsertId();
 
// select
$q = array(':lastId'=>$lastId);
$sql = 'SELECT * FROM reponse WHERE reponse_id = :lastId';
$req = $db->prepare($sql);
$req->execute($q) or die(print_r($req->errorInfo()));
$rows = $req->fetchAll();

Et un

Code:

var_dump($lastId);

m'affiche int(6) dans le cas de la condition else mais avec if rien ne s'affiche.

Cdlt,

21/09/2016, 17h35
Celira

Ok, donc la requête de récupération plante probablement parce que $lastId n'est pas renseigné. Où sont les if et else dont tu parles ?

Ce sont ceux à l'intérieur du foreach :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
foreach ($rows as $row):
    $tab = nl2br(htmlspecialchars($row['reponse_message']));
    $resTab = motInterdit($tab);
 
    if (is_array($resTab) && in_array($tab, $resTab)) {
         // envoie de mail
         mail($to, $subject, $message, $headers);
    } else {
        // envoie de mail
        mail($to, $subject, $message, $headers);
    }
endforeach;

Le script n’envoie pas de mail et n'insert pas la réponse dans la BD dans la condition if.

21/09/2016, 21h37
mOscar007

Je pense avoir compris le problème mais je ne suis pas sûr et si c'est le cas je ne sais pas comment procéder pour le corriger.
En effet, dans ma condition je teste ceci :

Code:

in_array($tab, $resTab)

.

Je teste si $tab (qui est le mot résultant de ma requête sql) est présent dans mon tableau $resTab qui est lui même résultant de la fonction qui va remplacer le mot interdit par des étoiles. Après avoir remplacer le mot interdit par des étoiles, peut-il (le mot résultant de ma requête sql) être présent (encore ?) dans le tableau de ma fonction ?

Cdlt,
22/09/2016, 10h39
mOscar007

Mon problème c'est que je disais dans le dernier post. Ma variable $tab que je vérifie dans mon tableau est déjà remplacé par des étoiles. Après avoir remplacer le mot interdit par des étoiles, le mot résultant de ma requête sql ne peut pas être présent dans le tableau de ma fonction. D’où l'erreur incompréhensible. Pour corriger le problème j'ai donc créé un nouveau tableau pour ma condition sans modifier ma fonction motInterdit() qui gère l'affichage (remplacement des étoiles).

Ça marche très bien maintenant.

Merci bcp @Celira pour ton aide !

A bientôt !