You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by hl...@apache.org on 2010/03/18 05:47:09 UTC

svn commit: r924636 - /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js

Author: hlship
Date: Thu Mar 18 04:47:08 2010
New Revision: 924636

URL: http://svn.apache.org/viewvc?rev=924636&view=rev
Log:
TAP5-1060: In IE8, forms that should submit using Ajax are triggering full page requests due to a client-side JavaScript error

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js?rev=924636&r1=924635&r2=924636&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js Thu Mar 18 04:47:08 2010
@@ -1241,6 +1241,14 @@ Tapestry.FormEventManager = Class.create
 	},
 
 	handleSubmit : function(domevent) {
+
+		/**
+		 * Necessary because we set the onsubmit property of the form, rather
+		 * than observing the event. But that's because we want to specfically
+		 * overwrite any other handlers.
+		 */
+		Event.extend(domevent);
+
 		var t = $T(this.form);
 
 		t.validationError = false;