Index: openacs.org-dev/packages/acs-tcl/tcl/form-processing-procs.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs.org-dev/packages/acs-tcl/tcl/form-processing-procs.tcl,v
diff -u -r1.1.1.3 -r1.1.1.4
--- openacs.org-dev/packages/acs-tcl/tcl/form-processing-procs.tcl	31 Jul 2002 16:30:56 -0000	1.1.1.3
+++ openacs.org-dev/packages/acs-tcl/tcl/form-processing-procs.tcl	8 Oct 2002 15:46:50 -0000	1.1.1.4
@@ -66,8 +66,8 @@
 
         my_table_key:key(my_table_sequence)
 
-        {value:text(textarea)             {{label "Enter text"}
-                                           {html {rows 4 cols 50}}}}
+        {value:text(textarea)             {label "Enter text"}
+                                           {html {rows 4 cols 50}}}
     } -select_query {
         select value from my_table where my_table_key = :my_table_key
     } -validate {
@@ -82,18 +82,16 @@
             values
               (:key, :value)"
         ad_returnredirect "somewhere"
-        return
+        ad_script_abort
     } -edit_data {
         db_dml do_update "
             update my_table
             set value = :value
             where my_table_key = :key"
         ad_returnredirect "somewhere"
-        return
+        ad_script_abort
     }
 
-    ad_return_template
-
     </pre></blockquote>
 
     <p>
@@ -125,6 +123,13 @@
 
     <p>
 
+    <b>General information about parameters</b>
+
+    <p>Parameters which take a name (for instance "-name" or "-select_query_name") expect a simple name
+    not surrounded by curly braces (in other words not a single-element list).  All other parameters expect
+    a single list to be passed in.
+    <p>
+
     Here's a complete list of switches that are supported by ad_form:
 
     <p>
@@ -150,11 +155,6 @@
         define multipart file handling forms.
     </dd>
 
-    <p><dt><b>-action</b></dt><p>
-    <dd>The name of the script to be called when the form is submitted.  Defaults to the name of the script
-        being served.  
-    </dd>
-
     <p><dt><b>-export</b></dt><p>
     <dd>Similar to the utility <b>export_vars</b>.  Takes a list of values to insert in the form as 
         "hidden" elements.   Each value is either a name, in which case the Tcl variable at the caller's
@@ -294,9 +294,9 @@
     </blockquote>
     
     <blockquote><pre>
-    {my_key:text(multiselect),multiple       {{label "select some values"}
+    {my_key:text(multiselect),multiple       {label "select some values"}
                                               {options {first second third fourth fifth}}
-                                              {html {size 4}}}}
+                                              {html {size 4}}}
                                   
     </pre><p>
 
@@ -306,7 +306,7 @@
     </blockquote>
     
     <blockquote><pre>
-    {hide_me:text(hidden)                     {{value 3}}}
+    {hide_me:text(hidden)                     {value 3}}
     </pre><p>
 
     Define the hidden form element "hide_me" with the value 3
@@ -615,7 +615,9 @@
                     help_text -
                     label -
                     format -
-                    value {
+                    value -
+                    before_html -
+                    after_html {
                         if { [llength $extra_arg] > 2 || [llength $extra_arg] == 1 } {
                             return -code error "element $element_name: \"$extra_arg\" requires exactly one argument"
                         }
@@ -872,7 +874,7 @@
     ad_form.
 
     @param element The name of the element
-    @parma value The value to set
+    @param value The value to set
 
 } {
     upvar #[template::adp_level] __ad_form_values__ values