From: skodak <skodak>
Date: Tue, 5 Dec 2006 21:43:33 +0000 (+0000)
Subject: MDL-7807 duplicate error messages in client side validation, fixed the rest of proble... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=e35c9eeb5f7ac6a8331b3774eecbc334e8da44fa;p=moodle.git

MDL-7807 duplicate error messages in client side validation, fixed the rest of problems I hope
---

diff --git a/lib/formslib.php b/lib/formslib.php
index 67d4a1e040..1e8aff0829 100644
--- a/lib/formslib.php
+++ b/lib/formslib.php
@@ -460,7 +460,7 @@ class moodleform {
         $mform->addElement('submit', $addfieldsname, get_string('addfields', 'form', $addfieldsno),
                             array('onclick'=>'this.form.submit();'));//need this to bypass client validation
 
-		$renderer =& $mform->defaultRenderer();
+        $renderer =& $mform->defaultRenderer();
         $renderer->addStopFieldsetElements($addfieldsname);
 
     }
@@ -739,17 +739,18 @@ var skipClientValidation = false;
 function qf_errorHandler(element, _qfMsg) {
   div = element.parentNode;
   if (_qfMsg != \'\') {
-    var errorDiv = document.getElementById(\'id_error_\'.element.name);
+    var errorSpan = document.getElementById(\'id_error_\'+element.name);
     if (!errorSpan) {
       errorSpan = document.createElement("span");
-      errorSpan.id = \'id_error_\'.element.name;
+      errorSpan.id = \'id_error_\'+element.name;
+      errorSpan.className = "error";
     }
     while (errorSpan.firstChild) {
       errorSpan.removeChild(errorSpan.firstChild);
     }
 
-    errorSpan.insertBefore(br, errorSpan.firstChild);
     errorSpan.appendChild(document.createTextNode(_qfMsg.substring(3)));
+    errorSpan.appendChild(document.createElement("br"));
     element.parentNode.insertBefore(errorSpan, element.parentNode.firstChild);
 
     if (div.className.substr(div.className.length - 6, 6) != " error"
@@ -759,7 +760,7 @@ function qf_errorHandler(element, _qfMsg) {
 
     return false;
   } else {
-    var errorSpan = document.getElementById(\'id_error_\'.element.name);
+    var errorSpan = document.getElementById(\'id_error_\'+element.name);
     if (errorSpan) {
       errorSpan.parentNode.removeChild(errorSpan);
     }