Addition: it was a battle to go back in my own brain :-) But now I think I remember why this thing was introduced.
When for example you post a straight URL in your message, like
http://www.minibb.com/forums/, and end it with some kind of punctuation symbol like dot, comma, semicolon etc., it may appear that this symbol is included in the HREF part of the URL tag itself, and it causes the URL to be opened properly. Most commonly you won't be able to just click on it - it will produce a 404 not found error on the server side.
This string in theory fixes such rare situation, it replaces the ending symbol with a quote to the quote sign only.
But as we see now this also affect image tags which is not good :-)
I think the proper update will be to patch urlMaker function like I did already here for miniBB forums. So bb_func_txt.php file will be released amongst the next version of miniBB.
Quick update for everybody who wants this immediately: replace urlMaker function of bb_func_txt.php to this:
function urlMaker($text){
$patterns=array("#(^|[ \n])(https|http|ftp)://([^<> \[\]\n\r]+)#i", "#(^|[ \n])ftp\.([^<> \[\]\n\r]+)#i", "#(^|[ \n])www\.([^<> \[\]\n\r]+)#i");
$replacements=array('\\1<a href="\\2://\\3" target="_blank" rel="nofollow">\\2://\\3</a>', '\\1<a href="ftp://ftp.\\2" target="_blank" rel="nofollow">ftp.\\2</a>', '\\1<a href="http://www.\\2" target="_blank" rel="nofollow">www.\\2</a>');
$ret=preg_replace($patterns, $replacements, $text);
if(preg_match("#<a href=\"(.+?)[.,\-:;)?!]+\"#i", $ret)) {
$ret=preg_replace("#<a href=\"(.+?)[.,\-:;)?!]+\"(.+?)>(.+?)</a>#is", '<a href="\\1"\\2>\\3</a>', $ret);
}return $ret;
}
and remove the aforementioned line in my post, from textFilter. That should work.
