Discussion :

[Bunny Toon]

Alors là les enfants accrochez-vous bien :p Je vais vous présenter ma super fonction qui envoie des variables PHP à du JavaScript qui les transforme en $_GET... m'enfin ça ne fonctionne pas tout à fait. o_O

Si vous n'avez rien compris ça ressemble à ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

<?php $html = preg_replace($pattern, '<a href="javascript:openPopUp(\'test.php?data=$1\')";>$1</a> ?>

Pour faire clair, ça remplace dans la page une partie des éléments pour les transformer en liens hypertextes qui ouvrent des pop-ups dont le contenu dépend du clic de départ (j'ai défini la fonction openPopUp dans un fichier JS et ça fonctionne bien).

Le petit souci est que parfois $pattern contient des ' (simple quotes) et forcément ça fait mongoliser JavaScript qui ne reçoit pas un argument correct (lorsqu'il n'y a pas de ', la fonction a le comportement souhaité). Le souci ne me semblait pas insurmontable j'ai tenté :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

<?php $html = preg_replace($pattern, '<a href="javascript:openPopUp(\'test.php?data=' . addslashes('$1') . '\')";>$1</a> ?>

Mais ça n'a rien changé. En fait les fonctions str_* ne fonctionnent pas à l'intérieur d'un preg_replace, elles sont tout simplement ignorées.

Quelqu'un a une idée plus intelligente que la mienne ?
Merci

[julp]

Il faut que vous utilisiez la fonction preg_replace_callback ou bien le modificateur e (n'ayant pas le motif, je ne détaillerais pas plus cette dernière solution).

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
$out = preg_replace_callback($pattern,
    function ($matches) {
        return '<a href="javascript:openPopUp(\'test.php?data=' . htmlspecialchars(addcslashes($matches[1], "'\r\n\\"), ENT_COMPAT/*, 'ISO-8859-1'*/) . '\')";>' . $matches[1] . '</a>';
    },
    $in
);

(non testé)

(PHP >= 5.3 requis pour ce dernier code, sinon il sera nécessaire de sortir la fonction de rappel ou demandera à se tourner vers une fonction lambda - create_function).

[Bunny Toon]

Ta solution est parfaite ! Merci beaucoup !

Le seul hic est que mon serveur est en PHP 5.2.12, car le site sur lequel je travaille n'est pas tout à fait compatible PHP 5.3

[julp]

Le sujet n'étant pas marqué résolu, avez-vous compris les adaptations possibles à réaliser ?

1) "Externaliser" la fonction de rappel :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
function cb_preg_replace_callback($matches) {
    return '<a href="javascript:openPopUp(\'test.php?data=' . htmlspecialchars(addcslashes($matches[1], "'\r\n\\"), ENT_COMPAT/*, 'ISO-8859-1'*/) . '\')";>' . $matches[1] . '</a>';
}
 
$out = preg_replace_callback($pattern,
    'cb_preg_replace_callback',
    $in
);

2) Créer une fonction lambda (c'est ce qui a de plus proche des fermetures mais ce n'est pas très "joli") :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
$out = preg_replace_callback($pattern,
    create_function('$matches', "return '<a href=\"javascript:openPopUp(\'test.php?data=' . htmlspecialchars(addcslashes(\$matches[1], \"'\\r\\n\\\\\"), ENT_COMPAT/*, 'ISO-8859-1'*/) . '\')\";>' . \$matches[1] . '</a>';"),
    $in
);

(avec toujours l'option supplémentaire du modificateur e, ce qui donnerait quelque chose de proche, mais un peu plus simple, que la fonction anonyme en terme de syntaxe)