Description: ------------ If a comment is not confirmed, two tags (<div>, <pre>) are not closed. Layout is broken then for comments that follow such an unconfirmed comment. The attached patch also fixed another notice in bug.php (was fixed in patch, but not committed).

I just saw that the bug preview layout is also broken (when logged in). The new patch also fixes this problem.

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. the first part of your patch is unnecessary. The code is: if (isset($_POST['in']) && (!isset($_POST['preview']) && $ncomment || $previous != $current)) { if (!$errors) { if (!isset($buggie)) { mail_bug_updates($bug, $_POST['in'], $from, $ncomment, $edit, $id, $previous, $current); you would change it to if (isset($_POST['in']) && (!isset($_POST['preview']) && $ncomment || $previous != $current)) { if (!$errors) { if (!isset($buggie)) { mail_bug_updates($bug, (isset($_POST['in']) ? $_POST['in'] : array()), $from, $ncomment, $edit, $id, $previous, $current); but this code is never executed if $_POST['in'] is not set (and shouldn't be, I might add).

> the first part of your patch is unnecessary. Okay, fine, I hadn't analyzed this part again, but just saw the difference between CVS and my local version. With the (new) if-condition this looks good now.