Habe ich einen Fehler in bbcode.php?

[Holger]

Hallo,

in meiner bbcode.php steht:

Code: Alles auswählen

// [img]image_url_here[/img] code..
  preg_match("/^\ \[img\](http:\/\/|ftp:\/\/)?([^\/:]*)([^\?]*\/?)?(\?.*)?\[\/img\]$/",$text,$url_ary);
  if (($url_ary[2] == $_SERVER["SERVER_NAME"]) and ((substr($url_ary[3],-14) == "/album_pic.php") or (substr($url_ary[3],-20) == "/album_thumbnail.php")))
  {
    $text = preg_replace("#\[img\]((ht|f)tp://)([^\r\n\t<\"]*?)\[/img\]#sie", "'[img:$uid]\\1' . str_replace(' ', '%20', '\\3') . '[/img:$uid]'", $text);
  }
  else
  {
    $text = preg_replace("#\[img\]((ht|f)tp://)([^ \?&=\"\n\r\t<]*?(\.(jpg|jpeg|gif|png)))\[/img\]#sie", "'[img:$uid]\\1' . str_replace(' ', '%20', '\\3') . '[/img:$uid]'", $text);
  }

	// Remove our padding from the string..
	return substr($text, 1);;

} // bbencode_first_pass()

Ist das korrekt: return substr($text, 1);;

Gruss
Holger

[KeineAhnung]

nö eigl genügt ja eine ;, da die nur zum begrenzen sind damit der php parser weiß wo eine neue funktion anfängt man könnte ja auch alles in einher linie schreiben

[Holger]

Das kann also kein Problem verursachen?

[KeineAhnung]

das wies ich nicht, aber machs einfach weg ohne schadets auf jedenfall nicht und wenn kein fehler kam dann hat's mit wohl auch nicht wirklich geschadet ;D