Fixed handling of multiple class names for errorClass option. Test by…

… Max Lynch. Fixes #280
jquery-validation · Mar 29, 2012 · bafc786 · bafc786
1 parent d8210bf
commit bafc786
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/jquery.validate.js b/jquery.validate.js
@@ -490,7 +490,8 @@ $.extend($.validator, {
 		},
 
 		errors: function() {
-			return $( this.settings.errorElement + "." + this.settings.errorClass, this.errorContext );
+			var errorClass = this.settings.errorClass.replace(' ', '.');
+			return $( this.settings.errorElement + "." + errorClass, this.errorContext );
 		},
 
 		reset: function() {

diff --git a/test/test.js b/test/test.js
@@ -445,6 +445,21 @@ test("option: focusCleanup with wrapper", function() {
 	ok( !form.is(":has(span:visible:has(label.error[for=username]))") );
 });
 
+test("option: errorClass with multiple classes", function() {
+	var form = $("#userForm")
+	form.validate({
+		focusCleanup: true,
+		wrapper: "span",
+		errorClass: "error error1"
+	});
+	form.valid();
+	ok( form.is(":has(span:visible:has(label.error[for=username]))") );
+	ok( form.is(":has(span:visible:has(label.error1[for=username]))") );
+	$("#username").focus().trigger("focusin");
+	ok( !form.is(":has(span:visible:has(label.error[for=username]))") );
+	ok( !form.is(":has(span:visible:has(label.error1[for=username]))") );
+});
+
 test("elements() order", function() {
 	var container = $("#orderContainer");
 	var v = $("#elementsOrder").validate({