e se fosse un sito in html come lo faresti...?
Il codice è corretto, ma più semplicemente FB ha un codice pronto che fa tutto per te...

Codice PHP:
<script>function fbs_click() {u=location.href;t=document.title;window.open('http://www.facebook.com/sharer.php?u='+encodeURIComponent(u)+'&t='+encodeURIComponent(t),'sharer','toolbar=0,status=0,width=626,height=436');return false;}</script>[url="http://www.facebook.com/share.php?u=<url>"][img]/img/facebook.png[/img][/url]