If someone use the ubb img tag in his signature threads convert it to img src but without selfclosing /> on its end. This gives a verify error in W3C Markup Validation Service.

and another one: ubbthreads/templates/default/showflat.tmpl. there is a missing close ancor tag and the # is too much.

found in line 156:
Code:

---

<a name="#{$postrow[$i]['Number']}">#<span id="number$i">{$postrow[$i]['Number']}</span> - {$postrow[$i]['time']} <i>{$postrow[$i]['IP']}</i> {$postrow[$i]['filelink']}

---

should be:
Code:

---

<a name="{$postrow[$i]['Number']}">#<span id="number$i">{$postrow[$i]['Number']}</span> - {$postrow[$i]['time']} <i>{$postrow[$i]['IP']}</i> {$postrow[$i]['filelink']}</a>

---

I found the xhtml bug for the missing selfclosing tag:

in /ubbthreads/ubbt.inc.php line 806:
Code:

---

$Body = preg_replace("/(\[IMG\]|\[{$ubbt_lang['TEXT_IMAGE']}\])http([^\[]*)\.(gif|jpg|png)(\[\/IMG\]|\[\/{$ubbt_lang['TEXT_IMAGE']}\])/i","<img src=\"http\\2.\\3\">",$Body);

---

should be:
Code:

---

$Body = preg_replace("/(\[IMG\]|\[{$ubbt_lang['TEXT_IMAGE']}\])http([^\[]*)\.(gif|jpg|png)(\[\/IMG\]|\[\/{$ubbt_lang['TEXT_IMAGE']}\])/i","<img src=\"http\\2.\\3\" alt=\"\" />",$Body);

---

mmhh, this fix the bug for the first time, but a new problem comes up. If I change the preg_replace ubb code for img got converted to html if I edit the post and if you try to submit this edit and html is off in this board the image is gone and text will be displayed.

How does this work? There must be another place to change a bit of code to solve this.

function undo_markup:
Code:

---

 
// Convert image markup
if ($config['allowimages'] || ($config['sigimage'] && $type=="sig")) {
     $Body = preg_replace("/<img src=\"([^\>]*)\">/i","[{$ubbt_lang['TEXT_IMAGE']}]\\1[/{$ubbt_lang['TEXT_IMAGE']}]",$Body);
}
 

---