php regex for detecting and replacing text links with anchor tags

Question

I got this regex off of a site forum to look for links in text and replace them with html links so that they are clickable.

the problem came when someone input one of those new youtube links that look like this:

http://youtu.be/djKd189-ag2

What happens is the link is made up to the dash and then -ag2 is left on it's own.

I don't know enough about php regex to even dare alter this to add parsing the dash so I was hoping I could get some help:

$newText = preg_replace('@((http|https)://([\w-.]+)+(:\d+)?(/([\w/_.]*(\?\S+)?)?)?)@', 
             '<a href="$1" class="link1" target="_blank">$1</a>',$text);

I am guessing it's somewhere around the \S+ part

possible duplicate of Convert plain text URLs into HTML hyperlinks in PHP is the source of your butchered regex, works, and only took one search to find(!) — mario
– mario, Commented May 8, 2012 at 13:20

Michael · Accepted Answer · 2012-05-08 13:15:39Z

2

$newText = preg_replace('@((http|https)://([\w-.]+)+(:\d+)?(/([\w/_-.]*(\?\S+)?)?)?)@', 
             '<a href="$1" class="link1" target="_blank">$1</a>',$text);

You need just a '-' where it's written '[\w/_.]'

answered May 8, 2012 at 13:15

Michael

1,3482 gold badges8 silver badges8 bronze badges

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

1 Comment

aidonsnous Over a year ago

I tried to use that but I am getting an error from php : PHP Warning 'yii\base\ErrorException' with message 'preg_replace(): Compilation failed: range out of order in character class at offset 42'

Collectives™ on Stack Overflow

php regex for detecting and replacing text links with anchor tags

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related