Discussion :

[guitz]

Bonjour,

Ces 2 lignes entrent en conflit :

Code :

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

1
2
3
4
 
$texte = preg_replace('#http://[a-z0-9._/-]+#i', '<a href="$0" target="_blank">$0</a>', $texte);
 
$texte = preg_replace('`\[img\](.+)\[/img\]`isU', '<img src="$1"/>', $texte);

Comment faire svp pour que le premier preg_replace ne remplace une url qu'à condition qu'il y ait un espace avant et après ?

Car le soucis c'est que je me retrouve avec des balises

Code :

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

<img> et <a href>

quand une image est postée sur mon forum.

Merci

[Celira]

Comment faire svp pour que le premier preg_replace ne remplace une url qu'à condition qu'il y ait un espace avant et après ?

En ajoutant les espaces à la regex ?

Code :

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

$texte = preg_replace('#\shttp://[a-z0-9._/-]+\s#i', '<a href="$0" target="_blank">$0</a>', $texte);

[guitz]

Désolé mais ça ne marche pas. C'est quoi #i svp ?

[stealth35]

Désolé mais ça ne marche pas. C'est quoi #i svp ?

tout est dans la doc
http://www.php.net/manual/fr/referen....modifiers.php

[guitz]

Ce que fait votre code c'est qu'à chaque fois que je met une url correspondant à un fichier image, il me met les balise :

Code :

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

<img src="url"/>

Or je souhaite que quand on met une url unique sans balise bbcode on ai un lien html vers cette url. Et que si on souhaiteafficher l'image à ce moment là on met les balises bbcode :

Code :

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

[img] [/img]

[guitz]

Au temps pour moi c'est moi qui m'étais emmellé les pinceaux, ça marche en fait Celira ya plus de conflit par contre à présent les liens seuls ne fonctionnent plus, une idée ?

Merci de m'avoir répondu

Edit : En fait les liens fonctionnent mais le soucis c'est que si d'entrée le mec met un lien dans son message ya pas d'espace avant mais un après, alors comment pourrais-je modifier le regex pour que "il faut soit un espace avant soit après l'url" ?

[guitz]

tout est dans la doc
http://www.php.net/manual/fr/referen....modifiers.php

Oui ça me ferait pas de mal d'apprendre les bases des regex

[guitz]

Bonjour bon bin j'ai trouvé comme un grand :

Code :

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

1
2
 
$texte = preg_replace('#\s*[^\]]http://[a-z0-9._/-]+[^\]]\s*#i', '<a href="$0" target="_blank">$0</a>', $texte);

Code :

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

\s*

de chaque coté pour qu'il y ait soit 0 soit un soit nombre strictement positif d'espaces avant et après l'url et pour pas de conflit avec les balises phpbb image j'ai mis :

Code :

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

[^\]]

de chaque coté

Et maintenant pour être vraiment tip top, il faut que j'impose au maximum arpès le point de l'url 3 caractères, le maximum c'est .com ? Car si un utilisateur met une url et oublie l'espace à la fin je vais me retrouver avec un lien du genre http://www.google.comsalut

Merci

[guitz]

Bonjour, ça ne marche pas en fait, comment svp échapper un crochet dans un regex ?

Code :

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

[^\[]

"un crochet ne doit pas apparaitre"

Code :

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

[^a]

"un a ne doit pas apparaitre"

edit: j'ai essayé ceci mais ça ne marche pas si la chaine commence par l'url sans espace avant :

Code :

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

$texte = preg_replace('#\s*[^]]http://[a-z0-9._/-]+[^[]\s*#i', '<a href="$0" target="_blank">$0</a>', $texte);

Traduction : zero ou plusieurs espaces avant, puis pas de crochet fermant puis

Code :

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

http://...

puis pas de crochet ouvrant puis zero ou plusieurs espaces LE TOUT pas sensible à la casse.

Je ne comprends pas ça devrait marcher, une idée ?

[gene69]

oui parce que ton "pas de crochet" va consommer ton h de http, apres il tombe sur le t (de http) alors que la regex s'attend au h de (http) .

tu ne peux pas tout accepter et tout corriger. conclusion si tu as des trucs trop complexe a analyser tu les refuses pis l'utilisateur corrigera tout seul.

ya quand meme une chose que je ne comprends pas. ta regex ne comporte pas de masque de capture donc c'est toute la capture qui est retournée dans $0, donc aussi les caracteres indésirables: espace-blanc, non-crocher. Ton lien ne sera pas valide ?!?

ce qui est le plus domage c'est que je vois pas comment utiliser filter_var() ici.

[guitz]

Salut gene69,

T'as raison faut pas non plus pousser le perfectionnisme jusque là, finalement j'ai trouvé une solution convenable :

Code :

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

1
2
3
$texte = preg_replace('`\[img\]http:(.+)\[/img\]`isU', '<img src="$1"/>', $texte);
 
	$texte = preg_replace('#http://[a-z0-9._/-]+#i', '<a href="$0" target="_blank">$0</a>', $texte);

pour une balise image phpbb l'url

Code :

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

http://www.machin.com/image.jpg

est transformée en

Code :

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

//www.machin.com/image.jpg

Donc a partir de la le second preg_replace ne va pas toucher à l'url qui est dans

Code :

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

<img src="//www.machin...."/>

Et donc plus de conflits, ensuite c'est sans espace donc lien peut être mis en début de post et en fin de post , ensuite c'est à l'utilisateur de penser à mettre un espace entre son texte et le lien

bonne soirée, merci à toi, résolu.

[gene69]

ok comme tu veux

sinon j'aurai vu pour les images
#\[img\](http://[^\s]*)\[img\]#i

(quelque chose qui est dans une balise img)

et pour les url en lignes.. sais pas trop.
#[^"](http://[^\s]*)

quelque chose qui ne commence pas par un "http comme dans tous les liens href="http://". Le seul probleme c'est que ça marchera pas s'il n'y a pas un caractère quelconque avant l'url... sinon même probleme.