Add element to success call with test. Fixes jquery-validation#60

mlynch · May 7, 2012 · a856823 · a856823
1 parent 851e941
commit a856823
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/jquery.validate.js b/jquery.validate.js
@@ -703,7 +703,7 @@ $.extend($.validator, {
 				if ( typeof this.settings.success === "string" ) {
 					label.addClass( this.settings.success );
 				} else {
-					this.settings.success( label );
+					this.settings.success( label, element );
 				}
 			}
 			this.toShow = this.toShow.add(label);

diff --git a/test/test.js b/test/test.js
@@ -875,6 +875,17 @@ test("success isn't called for optional elements", function() {
 	equal( 0, $("#testForm1 label").size() );
 });
 
+test("success callback with element", function() {
+	expect(1);
+	var v = $("#userForm").validate({
+		success: function( label, element ) {
+			equal( element, $('#username').get(0) );
+		}
+	});
+	$("#username").val("hi");
+	v.form();
+});
+
 test("all rules are evaluated even if one returns a dependency-mistmatch", function() {
 	expect(6);
 	equal( "", $("#firstname").removeClass().val() );