Warning: preg_replace() [function.preg-replace]: Unknown modifier

Question

I can;t figure out what's wrong with my line:

$ret = preg_replace( "(http://|https://|http://www.|https://www.)([[:alnum:]#?~/&=._-]+)", "<a href=\"\\1\\2\" target=\"_blank\">\\1\\2</a>", $ret);

I tried inserting a / in the first pos of the first arg but that didn't work.

I'm stumped. Thanks for any help.

ouch too many / use them as \/ u need to escape them

Abhik Chakraborty
– Abhik Chakraborty

2014-04-12 17:37:23 +00:00
Commented Apr 12, 2014 at 17:37 — Abhik Chakraborty
– Abhik Chakraborty, Commented Apr 12, 2014 at 17:37

Toto · Accepted Answer · 2014-04-13 09:55:54Z

2

Just add delimiter to your regex, choose one that is not present in your expression to avoid escaping, here I suggest !:

$ret = preg_replace( "!(http://|https://|http://www.|https://www.)([[:alnum:]#?~/&=._-]+)!", "<a href=\"$1$2\" target=\"_blank\">$1$2</a>", $ret);

You could also simplify a bit:

$ret = preg_replace( "!(https?://(?:www\.)?[\w#?~/&=.-]+)!", "<a href=\"$1\" target=\"_blank\">$1</a>", $ret);

answered Apr 13, 2014 at 9:55

Toto

91.7k63 gold badges97 silver badges135 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Jonan · Accepted Answer · 2014-04-12 18:05:03Z

0

$ret = preg_replace(
                '/((?:http:\/\/|https:\/\/)(?:www\.)?)([[:alnum:]#?~\/&=._-]+)/', 
                '<a href="$1$2" target="_blank">$1$2</a>', 
                $ret
);

It should work like this. Basically, you need to escape the forward slashes in your regex.

edited Apr 12, 2014 at 18:05

answered Apr 12, 2014 at 17:54

Jonan

2,5363 gold badges27 silver badges45 bronze badges

Collectives™ on Stack Overflow

Warning: preg_replace() [function.preg-replace]: Unknown modifier

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related