You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by je...@apache.org on 2005/04/08 18:14:26 UTC
svn commit: r160561 - in cocoon/branches/BRANCH_2_1_X/src/blocks/forms:
java/org/apache/cocoon/forms/flow/javascript/Form.js samples/sitemap.xmap
Author: jeremy
Date: Fri Apr 8 09:14:26 2005
New Revision: 160561
URL: http://svn.apache.org/viewcvs?view=rev&rev=160561
Log:
rationalising the parameter names of the handleForm function
Modified:
cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/flow/javascript/Form.js
cocoon/branches/BRANCH_2_1_X/src/blocks/forms/samples/sitemap.xmap
Modified: cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/flow/javascript/Form.js
URL: http://svn.apache.org/viewcvs/cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/flow/javascript/Form.js?view=diff&r1=160560&r2=160561
==============================================================================
--- cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/flow/javascript/Form.js (original)
+++ cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/flow/javascript/Form.js Fri Apr 8 09:14:26 2005
@@ -262,24 +262,32 @@
}
function handleForm() {
- var form = new Form(cocoon.parameters["form-definition"]);
-
- var args = [form];
-
- // set the binding on the form if there's any
+ // get the form definition
+ var def = cocoon.parameters["definitionURI"];
+ if (def == null) {
+ if (cocoon.parameters["form-definition"] != null) {
+ cocoon.log.warn("the form-definition parameter in handleForm has changed to definitionURI");
+ def = cocoon.parameters["form-definition"];
+ } else {
+ throw "Definition not configured for this form.";
+ }
+ }
+ // create the Form
+ var form = new Form(def);
+ // set the binding on the form if there is one
var bindingURI = cocoon.parameters["bindingURI"];
if (bindingURI != null) {
form.createBinding(bindingURI);
}
-
+ // get the function to call to handle the form
var funcName = cocoon.parameters["function"];
var func = this[funcName];
-
+ // check the function exists
if (!func) {
throw "Function \"" + funcName + "\" is not defined.";
} else if (!(func instanceof Function)) {
throw "\"" + funcName + "\" is not a function.";
}
-
- func.apply(this, args);
+ // call the function
+ func.apply(this, [form]);
}
Modified: cocoon/branches/BRANCH_2_1_X/src/blocks/forms/samples/sitemap.xmap
URL: http://svn.apache.org/viewcvs/cocoon/branches/BRANCH_2_1_X/src/blocks/forms/samples/sitemap.xmap?view=diff&r1=160560&r2=160561
==============================================================================
--- cocoon/branches/BRANCH_2_1_X/src/blocks/forms/samples/sitemap.xmap (original)
+++ cocoon/branches/BRANCH_2_1_X/src/blocks/forms/samples/sitemap.xmap Fri Apr 8 09:14:26 2005
@@ -102,13 +102,13 @@
<!-- Here we use an action especially created for the "form1" example
which will initialise the form with some data. -->
<map:act type="init-form">
- <map:parameter name="form-definition" value="forms/form1.xml"/>
+ <map:parameter name="definitionURI" value="forms/form1.xml"/>
<map:parameter name="attribute-name" value="form1"/>
</map:act>
</map:when>
<map:when test="POST">
<map:act type="handle-form-submit">
- <map:parameter name="form-definition" value="forms/form1.xml"/>
+ <map:parameter name="definitionURI" value="forms/form1.xml"/>
<map:parameter name="attribute-name" value="form1"/>
<map:parameter name="formhandler" value="org.apache.cocoon.forms.samples.Form1Handler"/>
@@ -149,7 +149,7 @@
<map:match pattern="form1.flow">
<map:call function="handleForm">
<map:parameter name="function" value="form1"/>
- <map:parameter name="form-definition" value="forms/form1.xml"/>
+ <map:parameter name="definitionURI" value="forms/form1.xml"/>
</map:call>
</map:match>
@@ -239,7 +239,7 @@
<map:match pattern="form2xml.flow">
<map:call function="handleForm">
<map:parameter name="function" value="form2xml"/>
- <map:parameter name="form-definition" value="forms/form2_model.xml"/>
+ <map:parameter name="definitionURI" value="forms/form2_model.xml"/>
<map:parameter name="documentURI" value="forms/form2_data.xml"/>
<map:parameter name="bindingURI" value="forms/form2_bind_xml.xml"/>
</map:call>
@@ -248,7 +248,7 @@
<map:match pattern="form2simpleXML.flow">
<map:call function="handleForm">
<map:parameter name="function" value="form2simpleXML"/>
- <map:parameter name="form-definition" value="forms/form2_model.xml"/>
+ <map:parameter name="definitionURI" value="forms/form2_model.xml"/>
<map:parameter name="documentURI" value="forms/form2_simple_data.xml"/>
</map:call>
</map:match>
@@ -256,7 +256,7 @@
<map:match pattern="form2bean.flow">
<map:call function="handleForm">
<map:parameter name="function" value="form2bean"/>
- <map:parameter name="form-definition" value="forms/form2_model.xml"/>
+ <map:parameter name="definitionURI" value="forms/form2_model.xml"/>
<map:parameter name="bindingURI" value="forms/form2_bind_bean.xml"/>
</map:call>
</map:match>
@@ -442,7 +442,7 @@
<map:match pattern="form_model_gui.flow">
<map:call function="handleForm">
<map:parameter name="function" value="form_model_gui"/>
- <map:parameter name="form-definition" value="forms/form_model_gui_model.xml"/>
+ <map:parameter name="definitionURI" value="forms/form_model_gui_model.xml"/>
<map:parameter name="attribute-name" value="form_model_gui"/>
<map:parameter name="documentURI" value="forms/form_model_gui_data.xml"/>
<map:parameter name="bindingURI" value="forms/form_model_gui_binding.xml"/>