[SPIP Zone] correction d'un bug sur "Barre Typo V2"

Bonjour,

J’ai essayé les 2 plugins suivants sur mon site :

  • barre_typo_v2
  • barre_typo_generalisee

Ils sont sensés fonctionner ensemble, sauf que les boutons « preview » et « statistiques » ne fonctionnent pas dans ce cas.
Le problème vient d’une erreur d’échappement… le code buggué génère par exemple :
<a href=« javascript:toggle_preview(1,‹ document.getElementsByName (‹ descriptif ›)[0] ›); » (…)
donc erreur javascript à cause du conflit entre les apostrophes.

Le code corrigé génère ceci :
<a href="javascript:toggle_preview(1,’ document.getElementsByName(‹ descriptif ›)[0]’); (…)

La correction concerne le fichier barre_typo_v2/inc/barre.php :

  • vers la ligne 336,
    Je remplace
    $ret .= bouton_barre_racc(« toggle_preview($num_barre,’$champ’); », (…)
    par
    $ret .= bouton_barre_racc(« toggle_preview($num_barre,’ ».replace("’","\’",$champ)."’);", (…)
    à la ligne suivante, idem pour le bouton « statistiques » :
    $ret .= bouton_barre_racc(« toggle_stats($num_barre,’ ».str_replace("’","\’",$champ)."’);", (…)

  • Tant qu’à y être, pour être rigoureux (pas indispensable pour corriger le bug remarqué),
    vers la ligne 26, juste avant le ‹ return › dans la fonction bouton_barre_racc()
    , j’ajoute :
    $action = str_replace(""",""",$action);

A+,
Yolek

* Olivier Pessin tapuscrivait, le 08/10/2007 10:59:

Bonjour,

J'ai essayé les 2 plugins suivants sur mon site :
- barre_typo_v2
- barre_typo_generalisee

Ils sont sensés fonctionner ensemble, sauf que les boutons "preview" et "statistiques" ne fonctionnent pas dans ce cas.
Le problème vient d'une erreur d'échappement... le code buggué génère

Merci beaucoup !
C'est intégré :wink:

--
RealET

* RealET tapuscrivait, le 08/10/2007 11:23:

* Olivier Pessin tapuscrivait, le 08/10/2007 10:59:

Bonjour,

J'ai essayé les 2 plugins suivants sur mon site :
- barre_typo_v2
- barre_typo_generalisee

Ils sont sensés fonctionner ensemble, sauf que les boutons "preview" et "statistiques" ne fonctionnent pas dans ce cas.
Le problème vient d'une erreur d'échappement... le code buggué génère

Merci beaucoup !
C'est intégré :wink:

C'est intégré, mais je ne peux pas le tester sur une 1.9.3 : en l'état, le plugin BT généralisé ne marche pas avec la 1.9.3.

--
RealET