You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2010/10/06 20:34:15 UTC

svn commit: r1005196 - /wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js

Author: mgrigorov
Date: Wed Oct  6 18:34:15 2010
New Revision: 1005196

URL: http://svn.apache.org/viewvc?rev=1005196&view=rev
Log:
WICKET-3095 Adding AjaxFormSubmitBehavior to Form leads to "Error: too much recursion" JS Error

Clean the attribute before returning.


Modified:
    wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js

Modified: wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js
URL: http://svn.apache.org/viewvc/wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js?rev=1005196&r1=1005195&r2=1005196&view=diff
==============================================================================
--- wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js (original)
+++ wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js Wed Oct  6 18:34:15 2010
@@ -1096,8 +1096,9 @@ Wicket.Ajax.Call.prototype = {
 		var submittingAttribute = 'data-wicket-submitting';
 		if (form.onsubmit && !form.getAttribute(submittingAttribute)) {
 			form.setAttribute(submittingAttribute, submittingAttribute);
-			if (!form.onsubmit()) return;
+			var retValue = form.onsubmit();
 			form.removeAttribute(submittingAttribute);
+			if (!retValue) return;
 		}
 		
 	    if (this.handleMultipart(form, submitButton)) {