X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=plekit%2Fphp%2Fdetails.php;h=e9cbc395a86d9d5c7bc4e95a82003440780c3a8c;hb=23d8b9c0df8872a70fc9dda57eaea640d9d2b133;hp=ccf1c4abf678f90b9ee4bb8a3f0cd14f8e5c9fd8;hpb=878ce4154f4619a5d07c780bf1f76dfd46771df4;p=plewww.git

diff --git a/plekit/php/details.php b/plekit/php/details.php
index ccf1c4a..e9cbc39 100644
--- a/plekit/php/details.php
+++ b/plekit/php/details.php
@@ -1,7 +1,5 @@
 <?php
 
-// $Id$
-
 require_once 'plc_functions.php';
 require_once 'form.php';
 
@@ -24,7 +22,7 @@ drupal_set_html_head('
 // th_th : special cases, display 2 <th>
 // xxx todo : accept optional arguments as an options hash, rather than using the set_ methods which are ugly
 
-class PlcDetails {
+class PlekitDetails {
   
   var $editable;
   var $form;
@@ -34,7 +32,7 @@ class PlcDetails {
   var $height;
   var $input_type;
 
-  function PlcDetails ($editable) {
+  function PlekitDetails ($editable) {
     $this->editable=$editable;
     $this->form=NULL;
     $this->width="";
@@ -47,9 +45,9 @@ class PlcDetails {
   // start the details area, with an optional caption
   function start ($caption="") { print $this->start_html("$caption");}
   function start_html ($caption="") {
-    $html="<table class=plc_details><thead>";
-    if ($caption) $html .= "<caption>$caption</caption>";
-    $html .= "</thead><tbody>";
+    $html="<table class='plc_details'>";
+    if ($caption) $html .= "<thead><caption>$caption</caption></thead>";
+    $html .= "<tbody>";
     return $html;
   }
 
@@ -59,10 +57,10 @@ class PlcDetails {
   }
 
   // starts an inner form if the details are editable
-  // accpets same args as PlcForm
-  function form_start ($url,$values,$method="POST") { print $this->form_start_html($url,$values,$method); return $this->form; }
-  function form_start_html ($url,$values,$method="POST") {
-    $this->form = new PlcForm ($url,$values,$method);
+  // accpets same args as PlekitForm
+  function form_start ($url,$values,$options=NULL) { print $this->form_start_html($url,$values,$options); return $this->form; }
+  function form_start_html ($url,$values,$options=NULL) {
+    $this->form = new PlekitForm ($url,$values,$options);
     return $this->form->start_html();
   }
 
@@ -124,7 +122,7 @@ class PlcDetails {
       else $height=$this->height;
 
       $html="";
-      $html .= "<tr><th><label for=$form_varname>$title</label></th>";
+      $html .= "<tr><th><label for='$form_varname'>$title</label></th>";
       $html .= "<td>";
       // xxx hack: if input_type is select : user provides the input field verbatim
       if ( $input_type == "select" ) {
@@ -137,9 +135,9 @@ class PlcDetails {
       } else {
 	// set id too 
 	$html .= "<input type='$input_type' name='$form_varname' id='$form_varname' value='$value'";
-	if ($width) $html .= " size=$width";
+	if ($width) $html .= " size='$width'";
 	// handle event callbacks
-	$html .= PlcForm::attributes($options);
+	$html .= PlekitForm::attributes($options);
 	$html .= "/>";
       }
       $html .= "</td></tr>";
@@ -162,7 +160,7 @@ class PlcDetails {
   // 1 item, colspan=2
   function tr($title,$align=NULL) { print $this->tr_html($title,$align);}
   function tr_html($title,$align=NULL) {
-    $result="<tr><td colspan=2";
+    $result="<tr><td colspan='2'";
     if ($align) $result .= " style='text-align:$align'";
     $result .=">$title</td></tr>";
     return $result;
@@ -170,7 +168,7 @@ class PlcDetails {
   
   // a dummy line for getting some air
   function space () { print $this->space_html(); }
-  function space_html () { return "<tr><td colspan=2>&nbsp;</td></tr>\n"; }
+  function space_html () { return "<tr><td colspan='2'>&nbsp;</td></tr>\n"; }
 
 }