Changeset 68 in text-formbuilder for trunk/lib/Text

Timestamp:

03/14/05 14:22:20 (20 years ago)

Author:

peichman

Message:

BUGFIX: option list items had an inadvertant minimum length of 2
cleaned up 'required' option handling

Location:

trunk/lib/Text

Files:

• FormBuilder.pm (modified) (1 diff)
• FormBuilder/grammar (modified) (3 diffs)

trunk/lib/Text/FormBuilder.pm

@@ -251 +251 @@
             }
         }
-    }    
-    
-    # remove false $$_{required} params because this messes up things at
-    # the CGI::FormBuilder::field level; it seems to be marking required
-    # based on the existance of a 'required' param, not whether it is
-    # true or defined
-    # TODO: check if this is still needed
-    $$_{required} or delete $$_{required} foreach @{ $self->{form_spec}{fields} };
+    }
     
     # assign the field names to the sections

trunk/lib/Text/FormBuilder/grammar

@@ -173 +173 @@
             list     => $list_var,
             validate => $item{'validate(?)'}[0],
-            required => $required || 0,
+            required => $required,
         };
         
@@ -191 +191 @@
         
         $type = undef;
-        $required = 0;
+        $required = undef;
         $list_var = undef;
         $size = undef;
@@ -233 +233 @@
 
 # for simple multiword values not involving punctuation
-simple_multiword: <skip:''> /\w[\w\t ]+/ { $item[2] }
+simple_multiword: <skip:''> /\w[\w\t ]*/ { $item[2] }
 
 # my attempt at a single-quoted, non-interpolating string