php regex change youtube links to videos - php

Question

I have functions that change links into clickable links and one that changes youtube links into embedded videos.
However, if you post a youtube video first then a regular link, all links turn into youtube videos.
How can I edit my functions to only change youtube links into embedded videos and ignore regular links?
function convertLinks($Link) {
$Link = preg_replace(
"#(^|[\n ])([\w]+?://[\w]+[^ \"\n\r\t< ]*)#",
"\\1\\2",
$Link);
$Link = preg_replace(
"#(^|[\n ])((www|ftp)\.[^ \"\t\n\r< ]*)#",
"\\1\\2",
$Link);
return $Link;
}
function convertYouTube($Link) {
if (preg_match(
'%(?:youtube(?:-nocookie)?\.com/(?:[^/]+/.+/|(?:v|e(?:mbed)?)/|.*[?&]v=)|youtu\.be/)([^"&?/ ]{11})%i',
$Link,
$match)) {
$video_id = $match[1];
$Link = htmlspecialchars_decode(
preg_replace(
"#(^|[\n ])([\w]+?://[\w]+[^ \"\n\r\t< ]*)#",
"\\1<iframe width=\"640\" height=\"360\" src=\"http://www.youtube.com/embed/$video_id\" frameborder=\"0\" allowfullscreen></iframe>",
$Link));
}
return $Link;
}
function convertMessages($Comment) {
$Comment = convertYouTube($Comment);
$Comment = convertLinks($Comment);
return $Comment;
}

Answer 1

I think I see where ukautz is coming from; you're finding certain youtube-only link text in your preg_match, yet in your preg_replace your regex doesn't care about youtube-specific text. If you were somehow able to isolate out the links before preg_matching them, this should solve the problem; however an easier solution might be to just combine your preg_match regex with your preg_replace regex.