Envoi d'une chaine contenant un \n à une alerte javascript

Version imprimable

20/02/2014, 20h36
jd_83

Envoi d'une chaine contenant un \n à une alerte javascript
Bonsoir,

J'ai une page php de contact, avec formulaire contenant un textarea.

Cette page envoie le contenu du champ textarea à une autre page qui se charge de composer un message et de l'envoyer par une fonction php mail.

Si l'envoi s'est fait correctement, je voudrais que l'expéditeur en soit informé par une fonction javascript contenant une alert(), lui rappelant le contenu de son envoi.
Code:

1 2 3 function afficher(s,c){ alert('Vous venez d\'envoyer le message suivant : \n sujet : '+s+' \n texte : '+c); }
Code:

1 2 3 4 if(mail($to, $sujet, $corps, $entetemail)){ echo "<script>afficher(\"".addslashes($sujet)."\",\"".addslashes($comment)."\") </script>" ; }
Tout va bien, tant que le textarea ne contient pas de retour chariot.
Mais dés qu'il en contient, la transmission se faisant par php, le \n pose problème et l'alerte ne s'affiche plus.

Est-il possible de contourner la difficulté ? Merci de m'aider.

JD
21/02/2014, 02h57
Watilin
Bonsoir,

Ton problème est que quand PHP génère du code JavaScript, celui-ci est inerte. Du code source. Du texte brut. Il faut d'abord qu'il passe dans les tuyaux du Net pour arriver jusqu'au navigateur/client pour être analysé et interprété par le moteur JavaScript.

Et donc, quand ton serveur envoie un saut de ligne dans ce qui deviendra une chaîne JavaScript, il ne se doute pas que de l'autre côté du Net l'interpréteur JS ne va pas digérer la chaîne à cause du saut de ligne dedans. (Et il se mettra à raconter des trucs du genre unterminated string literal).

Ta tentative avec les addslashes était une bonne idée, mais le problème est que les "\n" n'existent que dans le code source (PHP ou JS). Il est transformé en saut de ligne « réel » (octet 0A) quand le code source est interprété. Dans ton cas, tu dois faire l'opération inverse : transformer les sauts de ligne réels (qui proviennent du textarea) en symbole "\n" (c'est-à-dire un \ suivi d'un n, soit les octets 5C et 6E – comme tu vois ça n'a rien à voir).

Je te propose d'utiliser une astuce basée sur la différence, propre à PHP, entre les chaînes ' (à guillemets simples) et les chaînes " (à guillemets doubles) : ces dernières sont interprétées et, en particulier, les "\n" sont transformés. Alors que les '\n' ne le sont pas.

(J'espère que c'est assez clair parce que vu l'heure, c'est fort possible que mes capacités d'explications ne soient pas au top :aie:)

Et comme le monde est mal foutu, il faut aussi surveiller les "\r".

On utilise ce bon vieux str_replace :
Code:

1 2 3 function escapeNewLines( $s ){ return str_replace(array("\r", "\n"), array('\r', '\n'), $s); }
J'ai mis la vieille syntaxe des tableaux mais tu peux raccourcir si tu as PHP 5.4 ou supérieur.

Donc voilà maintenant tu peux remplacer les addslashes de ton code par mes escapeNewLines :)
21/02/2014, 08h52
jd_83

Super, ça marche : l'alerte affiche bien le saut de ligne.

Mais il reste un problème : le addslashes() n'est pas compatible avec le escapeNewLines().
Si je les mets les 2, soit rien n'est affiché, soit ce sont les \r ou \n qui s'affichent tels quels.

Or il reste nécessaire que les apostrophes ou guillemets du texte saisi par l'utilisateur soient échappés...

Merci pour ta contribution.

JD
21/02/2014, 09h11
jd_83
Ce que je viens d'écrire est inexact : le problème se pose seulement avec les guillemets.

J'ai l'impression que le problème est résolu avec :
Code:

1 2 3 4 function escapeNewLines( $s ){ $s=str_replace("\"","\\\"",$s) ; return str_replace(array("\r", "\n"), array('\r', '\n'), $s); }
Il y a peut-être plus élégant...

JD

regarde ce que donne le code source généré ...

le problème se situe plutot coté serveur que coté js ... le traitement d'echappement devant selon moi se faire en php ...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 <html> <head> <?php $sujet=" un deyx trois"; echo '<script type="text/javascript"> alert("'.preg_replace('~\r\n~','\\n',$sujet).'"); </script>'; ?></head> <body></body> </html>

22/02/2014, 02h08
Watilin
Citation:

Envoyé par jd_83

le addslashes() n'est pas compatible avec le escapeNewLines().

Dans un sens en effet ça pose problème car les "\n" deviennent des "\\n", mais normalement dans l'autre sens ça marche :

Code:

escapeNewLines(addslashes($comment))

Citation:

Or il reste nécessaire que les apostrophes ou guillemets du texte saisi par l'utilisateur soient échappés...

Je ne crois pas. Pour être honnête, la seule utilité de addslashes selon moi était de protéger les requêtes SQL, mais maintenant qu'il y a les PDO elle ne sert carrément plus à rien.

Citation:

Envoyé par jd_83

Ce que je viens d'écrire est inexact : le problème se pose seulement avec les guillemets.

Oui car ce sont les guillemets que tu utilises pour ouvrir tes chaînes JavaScript :
Code:

1 2 3 echo "<script>afficher(\"".addslashes($sujet)."\",\"".addslashes($comment)."\")" // ^^ ^^ ^^ ^^
Si tu avais déclaré les chaînes JS avec des apostrophes, tu aurais eu besoin d'échapper les apostrophes.
Citation:
Envoyé par jd_83

J'ai l'impression que le problème est résolu avec :

Code:

1 2 3 4 function escapeNewLines( $s ){ $s=str_replace("\"","\\\"",$s) ; return str_replace(array("\r", "\n"), array('\r', '\n'), $s); }

Il y a peut-être plus élégant...

JD
Plus élégant je ne sais pas. Plus court, oui si tu appliques ce que je t'ai expliqué à propos de la différence entre les guillemets doubles et simples. (Et aussi, au passage, si tu utilises la syntaxe courte des tableaux.)
Code:

1 2 3 4 function escapeNewLines( $s ){ $s =str_replace('"', '\"', $s) ; return str_replace(["\r", "\n"], ['\r', '\n'], $s); }
Plus cohérent, selon moi, c'est de ne pas mettre cet échappement de guillemets dans le corps de la fonction escapeNewLines car ce besoin d'échapper les guillemets est particulier à ta chaîne JS. Si un jour tu as besoin d'utiliser une deuxième, une troisième, une n-ième chaîne JS, et que tu déclares ces chaînes avec des apostrophes, la fonction escapeNewLines te trahira car elle ne peut pas marcher avec les deux types de chaînes. C'est source d'erreurs.

Pour faciliter la maintenance, je pense que le mieux est d'échapper le guillemet (ou l'apostrophe) au plus près de l'endroit où tu déclares la chaîne, si possible sur la même ligne :
Code:

1 2 echo "<script>afficher(\"".str_replace('"', '\"', escapeNewLines($sujet))."\",\"".str_replace('"', '\"', escapeNewLines($comment))."\") </script>";
Et pour éviter la répétition, je pense pourquoi pas à une fonction qui ferait en même temps l'ajout des guillemets autour de la chaîne et l'échappement des guillemets dedans :
Code:

1 2 3 function putQuotes( $s ){ return '"' . str_replace('"', '\"', $s) . '"'; }
Comme ça, plus de question à se poser : tu appelles putQuotes et c'est plié.
En JavaScript, les guillemets simples et doubles sont exactement équivalents.
22/02/2014, 15h45
jd_83

Cette fois-ci, pour moi, tout est clair.

Merci à tous.