[Bps-public-commit] rt-extension-formtools branch, master, updated. 0.50-7-g08d8d22

? sunnavy sunnavy at bestpractical.com
Tue Jul 16 09:41:19 EDT 2019


The branch, master has been updated
       via  08d8d22ea61b8477ce38830ff3ee0c3872d16da9 (commit)
      from  7d058364a3f889c29b4cb568d99bf1a8f1fe7389 (commit)

Summary of changes:
 html/FormTools/Form | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

- Log -----------------------------------------------------------------
commit 08d8d22ea61b8477ce38830ff3ee0c3872d16da9
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Tue Jul 16 21:38:16 2019 +0800

    Don't bother adding an empty class attribute
    
    This is consistent with how we handle $form_id/$form_name

diff --git a/html/FormTools/Form b/html/FormTools/Form
index 1b670f4..7b6ba7d 100644
--- a/html/FormTools/Form
+++ b/html/FormTools/Form
@@ -10,7 +10,7 @@ $enable_persisting   => 1
 $forbid_persisting   => {}
 $form_id             => undef
 $form_name           => undef
-$form_classes        => ''
+$form_classes        => undef
 $self_service        => 0
 </%args>
 <%init>
@@ -112,7 +112,7 @@ $next_for_validation ||= $m->caller(1)->path;
     enctype="multipart/form-data"
     <% defined $form_id ? ' id="'.$form_id.'"' : '' |n %>
     <% defined $form_name ? ' name="'.$form_name.'"' : '' |n %>
-    class="<% $form_classes %>"
+    <% defined $form_classes ? ' class="'.$form_classes.'"' : '' |n %>
 >
 % if ($validation) {
 <input type="hidden" name="_form_tools_next" value="<%$next%>" />

-----------------------------------------------------------------------


More information about the Bps-public-commit mailing list