[Pieforms-commit] SF.net SVN: pieforms: [173] pieforms-php5/trunk/src/pieform/elements/ fieldset.ph

Revision: 173
          http://svn.sourceforge.net/pieforms/?rev=173&view=rev
Author:   oracleshinoda
Date:     2007-02-04 14:38:38 -0800 (Sun, 04 Feb 2007)

Log Message:
-----------
Corrected spelling of 'collapsible'. Inform the user if a collapsible fieldset does not have a legend

Modified Paths:
--------------
    pieforms-php5/trunk/src/pieform/elements/fieldset.php

Modified: pieforms-php5/trunk/src/pieform/elements/fieldset.php
===================================================================
--- pieforms-php5/trunk/src/pieform/elements/fieldset.php	2007-01-25 23:11:58 UTC (rev 172)
+++ pieforms-php5/trunk/src/pieform/elements/fieldset.php	2007-02-04 22:38:38 UTC (rev 173)
@@ -34,8 +34,11 @@
  */
 function pieform_element_fieldset(Pieform $form, $element) {
     $result = "\n<fieldset";
-    if (!empty($element['collapsable'])) {
-        $classes = array('collapsable');
+    if (!empty($element['collapsible'])) {
+        if (!isset($element['legend']) || $element['legend'] === '') {
+            Pieform::info('Collapsible fieldsets should have a legend so they can be toggled');
+        }
+        $classes = array('collapsible');
         // Work out whether any of the children have errors on them
         $error = false;
         foreach ($element['elements'] as $subelement) {
@@ -52,7 +55,7 @@
     $result .= ">\n";
     if (isset($element['legend'])) {
         $result .= '<legend';
-        if (!empty($element['collapsable'])) {
+        if (!empty($element['collapsible'])) {
             $id = substr(md5(microtime()), 0, 4);
             $result .= ' id="' . $id . '">';
             $result .= '<script type="text/javascript">';


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