views:

216

answers:

2

I have some code with the a structure similar to this

           function bbcode($Text)
           { //$Text = preg_replace("/\[video\](.+?)\[\/video\]/",embed_video($1), $Text);
    return $Text;}

    function embed_video($url){
if (preg_match("/http:\/\/www.youtube.com\/watch\?v=([0-9a-zA-Z-_]*)(.*)/i", $url, $matches)) {
    return '<object width="425" height="350">'.
           '<param name="movie" value="http://www.youtube.com/v/'.$matches[1].'" />'.
           '<param name="wmode" value="transparent" />'.
           '<embed src="http://www.youtube.com/v/'.$matches[1].'&amp;autoplay="0" type="application/x-shockwave-flash" wmode="transparent" width="425" height="350" />'.
           '</object>';
}
    return $url;
    }

$lolcakes = "[video]http://youtube.com/id/xxxxxxpron[/video]";
$lolcakesconverted = bbcode($lolcakes);

The problem is it spits an error back at me.

Parse error: syntax error, unexpected T_LNUMBER, expecting T_VARIABLE or '$'

Have any ideas on how i can call embed_video inside of the preg_replace of the bbcode function?

Thanks!

+1  A: 

try preg_replace_callback

return preg_replace_callback("/\[video\](.+?)\[\/video\]/", 'embed_video', $Text);

function embed_video($matches)
{
  return $matches[1] . 'foo';      
}
jspcal
maybe i should have been a bit more specificthe embed_video() has another preg_replace inside it
jiexi
that's fine, you can do any transformations necessary in embed_video... preg_replace_callback will call it for any matches found and take the result
jspcal
+3  A: 

You can use the "e" modifier on preg_replace() (see Pattern Modifiers)

return preg_replace("/\[video\](.+?)\[\/video\]/e", "embed_video('$1')", $Text);

which tells preg_replace() to treat the second parameter as PHP code.

Owen
thank you very much. perfect
jiexi
cool. You solved my problem ! Thanks!!
Cris