From e574890ea82aa5e14c95623e9dfa7fcc1112992f Mon Sep 17 00:00:00 2001
From: mohrt <mohrt@localhost>
Date: Fri, 1 Nov 2002 15:18:27 +0000
Subject: [PATCH] make html_options work with optgroup, make func modular and
 recursive.

---
 libs/plugins/function.html_options.php | 42 +++++++++++++++++---------
 plugins/function.html_options.php      | 42 +++++++++++++++++---------
 2 files changed, 54 insertions(+), 30 deletions(-)

diff --git a/libs/plugins/function.html_options.php b/libs/plugins/function.html_options.php
index 0d91e6e2..79795035 100644
--- a/libs/plugins/function.html_options.php
+++ b/libs/plugins/function.html_options.php
@@ -21,26 +21,17 @@ function smarty_function_html_options($params, &$smarty)
     if (isset($options)) {
         settype($options, 'array');
         foreach ($options as $key => $value) {
-            $html_result .= "<option value=\"$key\"";
-            if (in_array($key, $selected))
-                $html_result .= " selected=\"selected\"";
-            $html_result .= ">$value</option>\n";
+			$html_result .= smarty_function_html_options_optoutput($key, $value, $selected);
         }
     } else {
         settype($output, 'array');
         settype($values, 'array');
         for ($i = 0, $for_max = count($output); $i < $for_max; $i++) {
-            /* By default, check value against $selected */
-            $sel_check = $values[$i];
-            $html_result .= "<option";
-            if ($i < count($values))
-                $html_result .= " value=\"".$values[$i]."\"";
-            else
-                $sel_check = $output[$i];       /* if more outputs than values, then
-                                                   check output against $selected */
-            if (in_array($sel_check, $selected))
-                $html_result .= " selected=\"selected\"";
-            $html_result .= ">".$output[$i]."</option>\n";
+            if ($i < count($values)) {
+				$html_result .= smarty_function_html_options_optoutput($values[$i], $output[$i], $selected);
+			} else {
+				$html_result .= smarty_function_html_options_optoutput($output[$i], $output[$i], $selected);
+			}
         }
     }
 
@@ -50,6 +41,27 @@ function smarty_function_html_options($params, &$smarty)
         return $html_result;
 }
 
+function smarty_function_html_options_optoutput($key, $value, $selected) {
+	if(!is_array($value)) {
+    	$html_result = "<option value=\"$key\"";
+    	if (in_array($key, $selected))
+        	$html_result .= " selected=\"selected\"";
+    	$html_result .= ">$value</option>\n";
+	} else {
+		$html_result = smarty_function_html_options_optgroup($key, $value, $selected);
+	}
+		return $html_result;	
+}
+
+function smarty_function_html_options_optgroup($key, $values, $selected) {
+	$optgroup_html = "<optgroup label=\"$key\">\n";
+	foreach ($values as $key => $value) {
+		$optgroup_html .= smarty_function_html_options_optoutput($key, $value, $selected);
+	}
+	$optgroup_html .= "</optgroup>\n";
+	return $optgroup_html;
+}
+
 /* vim: set expandtab: */
 
 ?>
diff --git a/plugins/function.html_options.php b/plugins/function.html_options.php
index 0d91e6e2..79795035 100644
--- a/plugins/function.html_options.php
+++ b/plugins/function.html_options.php
@@ -21,26 +21,17 @@ function smarty_function_html_options($params, &$smarty)
     if (isset($options)) {
         settype($options, 'array');
         foreach ($options as $key => $value) {
-            $html_result .= "<option value=\"$key\"";
-            if (in_array($key, $selected))
-                $html_result .= " selected=\"selected\"";
-            $html_result .= ">$value</option>\n";
+			$html_result .= smarty_function_html_options_optoutput($key, $value, $selected);
         }
     } else {
         settype($output, 'array');
         settype($values, 'array');
         for ($i = 0, $for_max = count($output); $i < $for_max; $i++) {
-            /* By default, check value against $selected */
-            $sel_check = $values[$i];
-            $html_result .= "<option";
-            if ($i < count($values))
-                $html_result .= " value=\"".$values[$i]."\"";
-            else
-                $sel_check = $output[$i];       /* if more outputs than values, then
-                                                   check output against $selected */
-            if (in_array($sel_check, $selected))
-                $html_result .= " selected=\"selected\"";
-            $html_result .= ">".$output[$i]."</option>\n";
+            if ($i < count($values)) {
+				$html_result .= smarty_function_html_options_optoutput($values[$i], $output[$i], $selected);
+			} else {
+				$html_result .= smarty_function_html_options_optoutput($output[$i], $output[$i], $selected);
+			}
         }
     }
 
@@ -50,6 +41,27 @@ function smarty_function_html_options($params, &$smarty)
         return $html_result;
 }
 
+function smarty_function_html_options_optoutput($key, $value, $selected) {
+	if(!is_array($value)) {
+    	$html_result = "<option value=\"$key\"";
+    	if (in_array($key, $selected))
+        	$html_result .= " selected=\"selected\"";
+    	$html_result .= ">$value</option>\n";
+	} else {
+		$html_result = smarty_function_html_options_optgroup($key, $value, $selected);
+	}
+		return $html_result;	
+}
+
+function smarty_function_html_options_optgroup($key, $values, $selected) {
+	$optgroup_html = "<optgroup label=\"$key\">\n";
+	foreach ($values as $key => $value) {
+		$optgroup_html .= smarty_function_html_options_optoutput($key, $value, $selected);
+	}
+	$optgroup_html .= "</optgroup>\n";
+	return $optgroup_html;
+}
+
 /* vim: set expandtab: */
 
 ?>