Fixed handling of multiple class names for errorClass option. Test by Max Lynch. Fixes jquery-validation#280

bsrykt · jzaefferer · commit bafc78671a94 · 2012-03-29T19:20:50.000+02:00
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({
