Dotclear

Ticket #796 (closed defect: fixed)

Opened 16 years ago

Last modified 15 years ago

Erreur du parseur wiki pour les url dans du code

Reported by: kozlika Owned by: xave
Priority: normal Milestone: 2.2
Component: module:clearbricks Version: 2.1
Severity: normal Keywords:
Cc:

Description

Lorsqu'on saisit dans un commentaire une url sous la forme

@@http://url-quelconque@@

la conversion ajoute des crochets de part et d'autre de cette url.

Bizarrerie supplémentaire : le crochet ouvrant est pris dans le code et le fermant non, ce qui donne dans la source html

<code>[http://url-quelconque</code>]

Change History

comment:1 Changed 16 years ago by xave

  • Component changed from module:core to module:clearbricks

comment:2 Changed 16 years ago by xave

je réussis à le reproduire sur un dc1(!) mais pas sur un dc2, wiki activé dans les commentaires ou pas.

comment:3 Changed 16 years ago by kozlika

Je viens de le tester chez toi et chez moi (en prévisu seulement chez toi, je n'ai pas validé pour ne pas pourrir ton blog :-P), avec cette saisie :

je fais un @@http://test.com@@.

Ce que je mentionne se reproduit : ajout de crochets, dont le deuxième hors du code, wiki des commentaires activé, avec ou sans wikibar. Visible là :  http://www.kozlika.org/kozeries/post/2009/10/10/Petite-brise-rafraichissante#c115045

comment:4 Changed 16 years ago by franck

Je viens de tester avec le schéma utilisé dans Clearbricks pour la reconnaissance des URL et effectivement ça merdoie sur la détection de la fin d'URL.

Pour le test, j'utilise cet outil en ligne (drôlement bien fait au demeurant) :  http://www.gskinner.com/RegExr/

Avec le schéma repris ligne 187 de class.wiki2xhtml.php :

(?<![\[\|])(http://|https://|ftp://|news:)([^"\s\)!]+)

Et en rajoutant cette ligne au beau milieu du texte proposé :

@@http://www.domain.tld/@@.

L'outil met en surbrillance ce qu'il trouve et effectivement il donne :

http://www.domain.tld/@@.

Maintenant je suis bien incapable de corriger la regexp !

comment:5 Changed 15 years ago by xave

  • Status changed from new to closed
  • Resolution set to fixed

fixed in Clearbricks rev 245.

Note: See TracTickets for help on using tickets.

Sites map