X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=plugins%2Ffunction.select_db_table.php;h=80f20a0ab65ff5965fcf51b7e97fe689b969b220;hb=7e001f68b7928d80ccf32b8a8bca55d304476d3e;hp=5bc2bcb56d43cd4020e33912c15f6268d6e9d66b;hpb=5ddeb07cc787dd1dc3630a31f1528f5cc7c4d9b9;p=platal.git

diff --git a/plugins/function.select_db_table.php b/plugins/function.select_db_table.php
index 5bc2bcb..80f20a0 100644
--- a/plugins/function.select_db_table.php
+++ b/plugins/function.select_db_table.php
@@ -1,6 +1,6 @@
 <?php
 /***************************************************************************
- *  Copyright (C) 2003-2007 Polytechnique.org                              *
+ *  Copyright (C) 2003-2010 Polytechnique.org                              *
  *  http://opensource.polytechnique.org/                                   *
  *                                                                         *
  *  This program is free software; you can redistribute it and/or modify   *
@@ -19,36 +19,36 @@
  *  59 Temple Place, Suite 330, Boston, MA  02111-1307  USA                *
  ***************************************************************************/
 
-
-function select_options($table,$valeur,$champ="text",$pad=false,
-                        $where="",$join="",$group="")
+function select_options($table, $value, $field, $pad, $where, $join, $group)
 {
-    $fields = 't.id,' . $champ;
-    $order = $champ;
+    $fields = 't.id,' . $field;
+    $order = $field;
     if ($group) {
         $fields .= ',' . $group;
-        $order = $group . ',' . $order; 
-    } 
+        $order = $group . ',' . $order;
+    } else {
+        $fields .= ', NULL';
+    }
     $sql = "SELECT $fields FROM $table AS t $join $where ORDER BY $order";
     $res = XDB::iterRow($sql);
     $sel = ' selected="selected"';
 
-    // on ajoute une entree vide si $pad est vrai
-    $html = "";
+    // An empty entry is added when $pad is true.
+    $html = '';
     if ($pad) {
-        $html.= '<option value="0"'.($valeur==0?$sel:"")."></option>\n";
+        $html .= '<option value="0"' . ($value == 0 ? $sel : '') . ">&nbsp;</option>\n";
     }
     $optgrp = null;
-    while (list($my_id,$my_text,$my_grp) = $res->next()) {
+    while (list($my_id, $my_text, $my_grp) = $res->next()) {
         if ($my_grp != $optgrp) {
             if (!is_null($optgrp)) {
                 $html .= '</optgroup>';
             }
-            $html .= '<optgroup label="' . htmlentities($my_grp, ENT_QUOTES) . '">';
+            $html .= '<optgroup label="' . pl_entities($my_grp, ENT_QUOTES) . '">';
             $optgrp = $my_grp;
         }
-        $html .= sprintf("<option value=\"%s\" %s>%s</option>\n", 
-                         $my_id, ($valeur==$my_id?$sel:""), $my_text);
+        $html .= sprintf("<option value=\"%s\" %s>%s</option>\n",
+                         $my_id, $value == $my_id ? $sel : '', pl_entities($my_text));
     }
     if (!is_null($optgrp)) {
         $html .= '</optgroup>';
@@ -57,18 +57,22 @@ function select_options($table,$valeur,$champ="text",$pad=false,
 }
 
 function smarty_function_select_db_table($params, &$smarty) {
-    if(empty($params['table']))
+    if (empty($params['table']) || empty($params['valeur'])) {
         return;
-    if(empty($params['champ']))
+    }
+    if (empty($params['champ'])) {
         $params['champ'] = 'text';
-    if(empty($params['pad']) || !($params['pad']))
-        $pad = false;
-    else
-        $pad = true;
-    if(empty($params['where']))
-        $params['where'] = '';
+    }
+    foreach (array('where', 'join', 'group') as $value) {
+        if (empty($params[$value])) {
+            $params[$value] = '';
+        }
+    }
+    $pad = (!empty($params['pad']) && $params['pad']);
+
     return select_options($params['table'], $params['valeur'], $params['champ'], $pad,
                           $params['where'], $params['join'], $params['group']);
 }
 
+// vim:set et sw=4 sts=4 sws=4 foldmethod=marker enc=utf-8:
 ?>