[Pieforms-commit] SF.net SVN: pieforms: [78] pieforms-php5/trunk/src/pieform/renderers

[<ora...@us...>]

Revision: 78
          http://svn.sourceforge.net/pieforms/?rev=78&view=rev
Author:   oracleshinoda
Date:     2006-12-06 21:32:18 -0800 (Wed, 06 Dec 2006)

Log Message:
-----------
Make each element have their ID prefixed by the form name, which is safer when more than one form is on a page

Modified Paths:
--------------
    pieforms-php5/trunk/src/pieform/renderers/div.php
    pieforms-php5/trunk/src/pieform/renderers/table.php

Modified: pieforms-php5/trunk/src/pieform/renderers/div.php
===================================================================
--- pieforms-php5/trunk/src/pieform/renderers/div.php	2006-12-06 22:02:27 UTC (rev 77)
+++ pieforms-php5/trunk/src/pieform/renderers/div.php	2006-12-07 05:32:18 UTC (rev 78)
@@ -35,10 +35,11 @@
  *                             container.
  */
 function pieform_renderer_div(Pieform $form, $builtelement, $rawelement) {
+    $formname = $form->get_name();
     // Set the class of the enclosing <div> to match that of the element
     $result = '<div';
     if (isset($rawelement['name'])) {
-        $result .= ' id="' . $rawelement['name'] . '_container"';
+        $result .= ' id="' . $formname . '_' . $rawelement['name'] . '_container"';
     }
     if ($rawelement['class']) {
         $result .= ' class="' . $rawelement['class'] . '"';
@@ -51,7 +52,7 @@
             $result .= Pieform::hsc($rawelement['title']);
         }
         else {
-            $result .= '<label for="' . $rawelement['id'] . '">' . Pieform::hsc($rawelement['title']) . '</label>';
+            $result .= '<label for="' . $formname . '_' . $rawelement['id'] . '">' . Pieform::hsc($rawelement['title']) . '</label>';
         }
     }
 

Modified: pieforms-php5/trunk/src/pieform/renderers/table.php
===================================================================
--- pieforms-php5/trunk/src/pieform/renderers/table.php	2006-12-06 22:02:27 UTC (rev 77)
+++ pieforms-php5/trunk/src/pieform/renderers/table.php	2006-12-07 05:32:18 UTC (rev 78)
@@ -35,6 +35,7 @@
  *                              container.
  */
 function pieform_renderer_table(Pieform $form, $builtelement, $rawelement) {
+    $formname = $form->get_name();
     if ($rawelement['type'] == 'fieldset') {
         // Add table tags to the build element, to preserve HTML compliance
         if (0 === strpos($builtelement, "\n<fieldset>\n<legend>")) {
@@ -53,7 +54,7 @@
     }
     
     $result = "\t<tr";
-    $result .= ' id="' . $rawelement['name'] . '_container"';
+    $result .= ' id="' . $formname . '_' . $rawelement['name'] . '_container"';
     // Set the class of the enclosing <tr> to match that of the element
     if ($rawelement['class']) {
         $result .= ' class="' . $rawelement['class'] . '"';
@@ -67,7 +68,7 @@
             $result .= Pieform::hsc($rawelement['title']);
         }
         else {
-            $result .= '<label for="' . $rawelement['id'] . '">' . Pieform::hsc($rawelement['title']) . '</label>';
+            $result .= '<label for="' . $formname . '_' . $rawelement['id'] . '">' . Pieform::hsc($rawelement['title']) . '</label>';
         }
     }
     $result .= "</th>\n\t\t<td>";


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.