preg_replace to replace string for matching url

Question

function makeLinksInTheContent($html)
{
    $html= preg_replace("/(^|[\n ])([\w]*?)((ht|f)tp(s)?:\/\/[\w]+[^ \,\"\n\r\t<]*)/is", "$1$2<a href=\"$3\" rel=\"nofollow\" >$3</a>", $html);
    $html= preg_replace("/(^|[\n ])([\w]*?)((www|ftp)\.[^ \,\"\t\n\r<]*)/is", "$1$2<a href=\"http://$3\" rel=\"nofollow\" >$3</a>", $html);
    $html= preg_replace("/(^|[\n ])([a-z0-9&\-_\.]+?)@([\w\-]+\.([\w\-\.]+)+)/i", "$1<a href=\"mailto:$2@$3\" rel=\"nofollow\">$2@$3</a>", $html);

    return($html);
}

this is my code.

My need is autolinking the url. Using preg_replace to find the url and set link to this url.

for example: "A page contains www.google.com." if i pass this content to makeLinksInTheContent($html), it will return "A page contains www.google.com."

But The following url format is not getting linked.

(http://www.google.com)
www.test.com()and[]&^%$#@!+|!@#$%^&()_+}{:"?><,./;'[]=-09~`.co,in,com.com
http://www.test.com()and[]&^%$#@!+|!@#$%^&()_+}{:"?><,./;'[]=-09~`.co,in,com.com
https://www.test.com()and[]&^%$#@!+|!@#$%^&()_+}{:"?><,./;'[]=-09~`.co,in,com.com
ftp://www.test.com()and[]&^%$#@!+|!@#$%^&()_+}{:"?><,./;'[]=-09~`.co,in,com.com

I think my regular expression have some mistakes. please suggest us.

I've already answered this question before...? This EXACT question — MDEV
– MDEV, Commented Jul 25, 2013 at 10:13
Duplicate of preg_replace to replace string for matching url — MDEV
– MDEV, Commented Jul 25, 2013 at 10:23

bystwn22 · Accepted Answer · 2013-07-25 10:27:14Z

3

You can use preg_replace_callback in this case. Read more

Function

<?php
  function replace_urls( $text = null ) {
    $regex  = '/((http|ftp|https):\/\/)?[\w-]+(\.[\w-]+)+([\w.,@?^=%&amp;:\/~+#-]*[\w@?^=%&amp;\/~+#-])?/';
    return preg_replace_callback( $regex, function( $m ) {
      $link = $name = $m[0];
      if ( empty( $m[1] ) ) {
        $link = "http://".$link;
      }
      return '<a href="'.$link.'" target="_blank" rel="nofollow">'.$name.'</a>';
    }, $text );
  }
?>

Usage

<?php
  $text = "http://stackoverflow.com/questions/17854971/preg-replace-to-replace-string-for-matching-url#17855054
  www.google.com
  https://twitter.com/
  http://www.somelinkwithhash.com/post/4454/?foo=bar#foo=bar";

  echo replace_urls( $text );
?>

Output

<a href="http://stackoverflow.com/questions/17854971/preg-replace-to-replace-string-for-matching-url#17855054" target="_blank" rel="nofollow">http://stackoverflow.com/questions/17854971/preg-replace-to-replace-string-for-matching-url#17855054</a>
<a href="http://www.google.com" target="_blank" rel="nofollow">www.google.com</a>
<a href="https://twitter.com/" target="_blank" rel="nofollow">https://twitter.com/</a>
<a href="http://www.somelinkwithhash.com/post/4454/?foo=bar#foo=bar" target="_blank" rel="nofollow">http://www.somelinkwithhash.com/post/4454/?foo=bar#foo=bar</a>

answered Jul 25, 2013 at 10:27

bystwn22

1,7941 gold badge10 silver badges9 bronze badges

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

1 Comment

user2349080 Over a year ago

The following urls not getting linked.

Community · Accepted Answer · 2017-05-23 11:49:14Z

0

My answer to the last time you asked this:

preg_replace to replace string for matching url

I can suggest using my function - just tested with your data and works for me

function parse_links($string,$mailto=true)
{
    $preg_r = "#(((https?|ftp)(://)?)?[a-zA-Z0-9-_.]{1,64}\.[a-zA-Z0-9-_.]{1,128}\.[a-z]{2,4}(\.[a-z]{2,4})?(/([^ ]{1,256}))?/?)#";
    $string = preg_replace($preg_r,"<a href=\"http://$1\">$1</a>",explode(" ",$string));
    if($mailto)
    {
        $preg_r2 = "/([0-9a-zA-Z]([-.\w]*[0-9a-zA-Z])*@([0-9a-zA-Z_\-\.][-\w]*[0-9a-zA-Z_\-\.]\.)+[a-zA-Z]{2,9})/";
        $string = preg_replace($preg_r2,"<a href=\"mailto:$1\">$1</a>",$string);
    }
    return implode(" ",$string);
}

Just pass false as second parameter if you don't want email links to be created

edited May 23, 2017 at 11:49

CommunityBot

11 silver badge

answered Jul 25, 2013 at 10:16

MDEV

10.9k2 gold badges35 silver badges50 bronze badges

2 Comments

user2349080 Over a year ago

The following urls not getting linked. (www.google.com), www.test.com()and[]&^%$#@!+|!@#$%^&()_+}{:"?><,./;'[]=-09~.co,in,com.com, https://www.test.com()and[]&^%$#@!+|!@#$%^&()_+}{:"?><,./;'[]=-09~.co,in,com.com

MDEV Over a year ago

I get three links when I test that string www.google.com www.test.com and https://www.test.com - what else were you hoping to capture??

Collectives™ on Stack Overflow

preg_replace to replace string for matching url

2 Answers 2

1 Comment

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related