Changing function to use preg_replace() instead of ereg_replace

Question

Possible Duplicate:
replace ereg_replace with preg_replace

I have got the following function within a code base that takes a String and makes links active. I have noticed that ereg_replace() is Depreciated. How would I change this to use preg_replace?

 function makeActiveLink($originalString){

        $newString = ereg_replace("[[:alpha:]]+://[^<>[:space:]]+[[:alnum:]/]","<a href=\"\\0\" target=\"_blank\">\\0</a>", $originalString);
        return $newString;
    }

score 4 · Answer 1 · answered Aug 20 '11 at 00:13

4

You can keep it almost exactly the same, but it would be preferable to change some things:

function makeActiveLink($originalString){
    $newString = preg_replace('#[a-z]+://[^<>\s]+[[a-z0-9]/]#i', '<a href="\0" target="_blank">\0</a>', $originalString);

    return $newString;
}

Note that I used # as a delimiter because you have slashes inside your string.

answered Aug 20 '11 at 00:13

Ry-

209,133
54
439
449

You can use `A-z` instead of `a-zA-Z`, or use `a-z` along with the `i` modifier to tell the regular expression engine to ignore case. – Samir Talwar Aug 20 '11 at 00:15
@Samir: Thanks, changed. – Ry- Aug 20 '11 at 00:15

score 1 · Answer 2 · answered Aug 20 '11 at 00:11

1

function makeActiveLink($originalString) {
    $pattern '#[a-z]+://[^<>\s]+[[a-z0-9]/]#i';
    $newString = preg_replace($pattern, '<a href="\\0" target="_blank">\\0</a>', $originalString);

    return $newString;
}

answered Aug 20 '11 at 00:11

PeeHaa

69,318
57
185
258

Changing function to use preg_replace() instead of ereg_replace

2 Answers2