Create a form_helper class containing the function Draw_Form so we don't have to always check to see if the DrawForm method is already defined in pages that could have multiple forms.

1 files changed, 1 insertions, 44 deletions

diff --git a/themes/default/views/form.html.php b/themes/default/views/form.html.php
index 41bdc0f9..802b1dd6 100644
--- a/themes/default/views/form.html.php
+++ b/themes/default/views/form.html.php
@@ -10,50 +10,7 @@ if ($title) {
   print "<!-- unused title in form.html.php: $title -->";
 }
 
-if (!function_exists("DrawForm")) {
-  function DrawForm($inputs, $level=1) {
-    $error_messages = array();
-    $prefix = str_repeat("  ", $level);
-
-    foreach ($inputs as $input) {
-      if ($input->type == 'group') {
-        print "$prefix<fieldset>\n";
-        print "$prefix  <legend>$input->name</legend>\n";
-        print "$prefix  <ul>\n";
-        DrawForm($input->inputs, $level + 2);
-        DrawForm($input->hidden, $level + 2);
-        print "$prefix  </ul>\n";
-        print "$prefix</fieldset>\n";
-      } else {
-        if ($input->error_messages()) {
-          print "$prefix<li class=\"gError\">\n";
-        } else if ($input->type) {
-          print "$prefix<li>\n";
-        } else {
-          // no type means its a "hidden" so don't wrap it in <li>
-        }
-        if ($input->label()) {
-          print "$prefix  {$input->label()}\n";
-        }
-        print "$prefix  {$input->render()}\n";
-        if ($input->message()) {
-          print "$prefix  <p>{$input->message()}</p>\n";
-        }
-        if ($input->error_messages()) {
-          foreach ($input->error_messages() as $error_message) {
-            print "$prefix  <p class=\"gError\">\n";
-            print "$prefix    $error_message\n";
-            print "$prefix  </p>\n";
-          }
-        }
-        if ($input->type) {
-          print "$prefix</li>\n";
-        }
-      }
-    }
-  }
-}
-DrawForm($inputs);
+print form_helper::Draw_Form($inputs);
 
 print($close);
 ?>