Description: ------------ function _clearErrors(element) { var spans = element.getElementsByTagName('span'); for (var i = 0, span; span = spans[i]; i++) { if (qf.classes.has(span, 'error')) { span.parentNode.removeChild(span); } } }; function has followed errors: 1. after remove "span" from "spans", "spans" becomes one element less. So indexes sifts. Iterator "i" still rise and some spans are skips. 2. function does not remove the "error" class of the parent element. It's problem for chained rules.

possible solution: function _clearErrors(element) { var spans = element.getElementsByTagName('span'); for (var i = 0, span; span = spans[i]; i++) { if (qf.classes.has(span, 'error')) { qf.classes.remove(span.parentNode, ['error', 'valid']); span.parentNode.removeChild(span); i--; } } };

1. Confirmed, getElementsByTagName() returns NodeList and that indeed reflects changes in the DOM tree: http://www.w3schools.com/dom/dom_nodelist.asp 2. Can you please provide a test script where this matters? AFAIR, _clearErrors() is run immediately before validation and "error" / "valid" classes are set / reset there.

Here is a complete test case: ========== require_once 'HTML/QuickForm2.php'; require_once 'HTML/QuickForm2/Renderer.php'; $form = new HTML_QuickForm2('bug19063'); $first = $form->addText('first')->setLabel('First:'); $second = $form->addText('second')->setLabel('Second:'); // Third element is mostly needed to prevent form submit $third = $form->addText('third')->setLabel('Third:'); $form->addSubmit('send', array('value' => 'Test a bug!')); $first->addRule( 'nonempty', 'First should be not empty', null, HTML_QuickForm2_Rule::CLIENT_SERVER )->or_($second->createRule( 'nonempty', 'or second should be not empty' )); $third->addRule( 'nonempty', 'Third should be not empty', null, HTML_QuickForm2_Rule::CLIENT_SERVER ); $renderer = HTML_QuickForm2_Renderer::factory('default'); $form->render($renderer); // Output javascript libraries, needed for client-side validation echo $renderer->getJavascriptBuilder()->getLibraries(true, true); echo $renderer; ========== How to reproduce: 1) Click submit without filling any fields --- 3 errors, then 2) Input something into "First", click submit --- error on "Second" does not disappear, then 3) Click submit once again --- error on "Second" disappears, but 'error' class is kept.

This bug has been fixed in SVN. 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.