[PHP] Fout in ubbparser

[Verwijderd]

Ik heb voor mn forum een ubbparser (duh ), alles erin werkt heel erg prima naar mijn mening, alleen er is 1 klein bugje...
Die ik zelf nogal vaag vind, en niet kan vinden waarom het gebeurd...

code:

<?php
  $text = preg_replace('#(^|[\n ])(http://[^ \n\r\t<]*)#is', '\1<a href="\2" target="_blank" style=text-decoration:underline;>\2</a>', $text);
  $text = preg_replace('#(^|[\n ])((www|ftp).[^ \t\n\r<]*)#is', '\1<a href="http://\2" target="_blank" style=text-decoration:underline;>\2</a>', $text);

  $text = preg_replace('/\[url=www.(.*?)\][\/url]/si', '<a href="http://www.\\1" target="_blank" style="text-decoration:underline;">\\2</a>',$text);
  $text = preg_replace('/\[url=(.*?)\][\/url]/si', '<a href="\\1" target="_blank" style="text-decoration:underline;">\\2</a>',$text);
  $text = preg_replace('/\[url=(.*?)\](.+?)\\[\/url]/isD', '<a href="\\1" target="_blank" style="text-decoration:underline;">\\2</a>',$text);
  $text = preg_replace('/\[url](.*?)\\[\/url]/si','<a href="\\1" target="_blank" style="text-decoration:underline;">\\1</a>',$text); 
?>

Zodra ik dit gebruik:

code:

[url=http://www.bla.nl]Bla[/url]

gaat hij goed, maar zodra de naam van de url met een R begint, geeft hij de url niet goed meer weer.. dus

code:

[url=http://www.bla.nl]Rare bug[/url]

Dan krijg ik iets te zien van

code:

arararara[/url]

Wie kan me helpen met dit probleem ?

[ Voor 34% gewijzigd door Verwijderd op 13-11-2004 16:27 ]

[Sjaaky]

Heb je ook last als de naam begint met een 'n' of een 't'?
Verder geldt gewoon net als altijd debuggen!
Zet na elke replace een echo $text; dan moet je er zo uit zijn. Verder kan je zoveel mogelijk code weg te halen zodanig dat het probleem zich nog steeds voordoet.

[NMe]

Ik zie zo ff niet waar je fout kan zitten, maar misschien kun je een werkende regexp van RegExLib.com halen?

[Jordi]

code:

'/\[url=(.*?)\][\/url]/si'

Je vergeet heel handig () die laatste [ en ] te escapen zodat /, u, r, l als karakterset worden beschouwd. Hij zal nu meteen op de R uit 'Rare bug' matchen.
Dat doe je trouwens op meerdere plaatsen, maar deze is de boosdoener, gok ik.

edit:
overigens begrijp ik niet wat deze regex hier doet. je verwijst in de replace naar een 2e ding dat helemaal niet gecaptured wordt.

[ Voor 37% gewijzigd door Jordi op 13-11-2004 17:17 ]

[Verwijderd]

Sjaaky schreef op zaterdag 13 november 2004 @ 16:38:
Heb je ook last als de naam begint met een 'n' of een 't'?
Verder geldt gewoon net als altijd debuggen!
Zet na elke replace een echo $text; dan moet je er zo uit zijn. Verder kan je zoveel mogelijk code weg te halen zodanig dat het probleem zich nog steeds voordoet.

Nope, het is alleen met de 'r'... ik zal eens proberen met die echo's

@ Jotti - Wat zou het wel moeten worden dan? Heb wel wat zitten escapen enzo, maar kreeg steeds errors

[ Voor 15% gewijzigd door Verwijderd op 13-11-2004 17:26 ]

[MrJay]

Je vergeet geregeld de ] van de [url] of [\url] te backspacen. Probeer deze eerst eens te backspacen...

[Thijsmans]

offtopic: ipv al die losse regexpen, is het overzichtelijker om ze in een array te zetten:

Every parameter to preg_replace() (except limit) can be an unidimensional array. When using arrays with pattern and replacement, the keys are processed in the order they appear in the array. This is not necessarily the same as the numerical index order.

PHP:

$pattern[0] = '#(^|[\n ])(http://[^ \n\r\t<]*)#is';

(...)

$replacement[0] = '\1<a href="\2" target="_blank" style=text-decoration:underline;>\2</a>';

(...)
return preg_replace($patterns, $replacement, $input);