Changeset 88 in text-formbuilder for trunk/lib/Text/FormBuilder

Timestamp:

06/29/05 16:46:56 (19 years ago)

Author:

peichman

Message:

BUGFIX: stopped crosstalk of data from one object to another (references were getting shared in the parser grammar code)

File:

• trunk/lib/Text/FormBuilder/grammar (modified) (4 diffs)

trunk/lib/Text/FormBuilder/grammar

@@ -37 +37 @@
 
 form_spec:
-    { 
-        %formspec = ();  # clear the old formspec data
+    {
+        # clear out old data, so we don't end up with old data in new objects
+        @lines    = ();
+        @sections = ();
+        %formspec = ();
     }
     (list_def | description_def | group_def | note | fb_params | unknown_block_directive | line)(s)
@@ -52 +55 @@
         }
         
+        # make copies instead of taking references, again so we
+        # don't end up with connections between objects
         $return = {
             fb_params   => $formspec{fb_params},
@@ -57 +62 @@
             author      => $formspec{author},
             description => $formspec{description},
-            lists       => \%lists,
-            patterns    => \%patterns,
-            subs        => \%subs,
-            groups      => \%groups,
-            sections    => \@sections,
-            ( @submit ? 
-                (submit => @submit == 1 ? $submit[0] : \@submit) :
-                ()
-            ),
+            lists       => { %lists },
+            patterns    => { %patterns },
+            subs        => { %subs },
+            groups      => { %groups },
+            sections    => [ @sections ],
+            ( @submit ? (submit => @submit == 1 ? $submit[0] : [ @submit ]) : () ),
             reset       => $formspec{reset},
         }
@@ -233 +235 @@
         @options = ();
         
+        #warn "$$field{name}: $field";
+        
         $field;
     }