bbcode problem parse

[boburmum1]

<?php
function topic($input){ 

$BBCode = array("&" => "&amp;",
"<" => "&lt;",
">" => "&gt;",
"[b]" => "<b>",
"[/b]" => "</b>",
"[i]" => "<i>",
"[/i]" => "</i>",
"[u]" => "<u>",
"[/u]" => "</u>",
"[img]" => "<img src='",
"[/img]" => "'>",
"[code]" => "<font style=\"padding:3px;\" color=\"#CC0000\" size=\"2\">Code:</font><br><div id='code'><tt>",
"

" => "</tt></div>",
"" => "<a href=>",
"" => "</a>");

$parsedtext = str_replace(array_keys($BBCode), array_values($BBCode), $input);
return nl2br($parsedtext);

return nl2br(strip_tags(stripslashes(htmlspecialchars($input))));

}
?>[/code]

my url one is not working, it just reloads the same page. and please if you post some code to help me, keep it in the same format as above.

[dagon]

Find row
$message = preg_replace('/\[url\]|\[\/url\]/si', '', $message);
end replace it with
$message = preg_replace('/\[url\]|\[url=[a-zA-Z.:\/_\-0-9#]+\]|\[\/url\]/si', '', $message);

to replace correctly all [url] bbcodes.

From: http://www.phpbb.com/bugs/phpbb2/13341

[boburmum1]

sorry, but that didn't work. i really need it in the way that i've done it up there.

[dagon]

you can't just use str_replace for urls

[boburmum1]

<?php
function topic($input){

$BBCode = array("&" => "&amp;",
"<" => "&lt;",
">" => "&gt;",
"[b]" => "<b>",
"[/b]" => "</b>",
"[i]" => "<i>",
"[/i]" => "</i>",
"[u]" => "<u>",
"[/u]" => "</u>",
"[img]" => "<img src='",
"[/img]" => "'>",
"[code]" => "<font style=\"padding:3px;\" color=\"#CC0000\" size=\"2\">Code:</font><br><div id='code'><tt>",
"

" => "</tt></div>",
"" => "<a href=>",
"" => "</a>");

$message = preg_replace('/[url]|[url=[a-zA-Z.:\/_-0-9#]+]|[\/url]/si', '', $message);

return nl2br($message);

$parsedtext = str_replace(array_keys($BBCode), array_values($BBCode), $input);
return nl2br($parsedtext);

return nl2br(strip_tags(stripslashes(htmlspecialchars($input))));

}
?> [/code]

so would it be kinda like that?

[dagon]

everything beyond the first return wont run as return ends the function.

[nrg_alpha]

Just a side note on something I noticed:

"[/img]" => "'>",

I'm not sure if you are going to embed an alt attribute in there or not, but note that without one, your (x)html page will not validate. And if your document is xhtml and not html, then img tags are self closing ones (space and a forward slash):

"[/img]" => "' alt=\"\" />",

[Weedpacket]

Another side note:

"[code]" => "<font style=\"padding:3px;\" color=\"#CC0000\" size=\"2\">Code:</font><br><div id='code'><tt>",

Without going into detail over the invalid use of the "id" attribute, why not just use a suitably-styled <code> element? That's what they're for: to represent code.

.inline_code {
    padding:3px;
    color:#C00;
    font-size:small;
    white-space:pre;
}

"[code]" => '<code class="inline_code">',