You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by jk...@apache.org on 2006/06/10 15:00:58 UTC

svn commit: r413291 - in /tapestry/tapestry4/trunk/framework/src/js: dojo/ tapestry/form.js tests/test_form.js

Author: jkuhnert
Date: Sat Jun 10 06:00:57 2006
New Revision: 413291

URL: http://svn.apache.org/viewvc?rev=413291&view=rev
Log:
Sorry for the noise :(

Removed:
    tapestry/tapestry4/trunk/framework/src/js/dojo/
Modified:
    tapestry/tapestry4/trunk/framework/src/js/tapestry/form.js
    tapestry/tapestry4/trunk/framework/src/js/tests/test_form.js

Modified: tapestry/tapestry4/trunk/framework/src/js/tapestry/form.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/framework/src/js/tapestry/form.js?rev=413291&r1=413290&r2=413291&view=diff
==============================================================================
--- tapestry/tapestry4/trunk/framework/src/js/tapestry/form.js (original)
+++ tapestry/tapestry4/trunk/framework/src/js/tapestry/form.js Sat Jun 10 06:00:57 2006
@@ -125,7 +125,9 @@
 			return;
 		}
 		
-		form.submitname.value=submitName;
+		if (submitName){
+			form.submitname.value=submitName;
+		}
 		form.submit();
 	},
 	
@@ -134,19 +136,21 @@
 	 * but submits the request via XHR to the server asynchronously.
 	 * 
 	 * @param form The form(form id) to submit.
-	 * @param submitName Optional submit name string to use when submitting.
 	 * @param content Optional content map, mainly used to pass in browser
 	 * 				  event parameters to form submission, but can be any
 	 * 				  typical form/value pair.
+	 * @param submitName Optional submit name string to use when submitting.
 	 */
-	submitAsync:function(form, submitName, content){
+	submitAsync:function(form, content, submitName){
 		var form=dojo.byId(form);
 		if (!form) {
 			dojo.raise("Form not found with id " + id);
 			return;
 		}
 		
-		form.submitname.value=submitName;
+		if (submitName){
+			form.submitname.value=submitName;
+		}
 		dojo.io.bind({
 			formNode:form,
 			content:content,

Modified: tapestry/tapestry4/trunk/framework/src/js/tests/test_form.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/framework/src/js/tests/test_form.js?rev=413291&r1=413290&r2=413291&view=diff
==============================================================================
--- tapestry/tapestry4/trunk/framework/src/js/tests/test_form.js (original)
+++ tapestry/tapestry4/trunk/framework/src/js/tests/test_form.js Sat Jun 10 06:00:57 2006
@@ -23,7 +23,10 @@
 }
 
 function test_form_deprecated(){
-	Tapestry.register_form("null");
+	try {
+		Tapestry.register_form("null");
+		throw new JUMAssertFailure("Previous test should have failed.");
+	} catch (e) { jum.assertTrue("testFormDepre", e instanceof Error); return; }
 	jum.assertTrue("deprecated", lastMsgContains("deprecated"));
 	Tapestry.onpresubmit();
 	jum.assertTrue("deprecated", lastMsgContains("deprecated"));