1 files changed, 32 insertions, 19 deletions
diff --git a/httemplate/elements/select-table.html b/httemplate/elements/select-table.html
index 4bf3d1c4d..8bcbf2504 100644
--- a/httemplate/elements/select-table.html
+++ b/httemplate/elements/select-table.html
@@ -5,32 +5,38 @@ Example:
   include( '/elements/select-table.html',
 
     #required
-    'table'    => 'table_name',
-    'name_col' => 'name_column',
+    'table'          => 'table_name',
+    'name_col'       => 'name_column',
    
     #strongly recommended (you want your forms to be "sticky" on errors, right?)
-    'value'    => 'current_value',
+    'curr_value'     => 'current_value',
+    #'value' => #deprecated form of 'curr_value',
    
     #opt
-    'empty_label'   => '', #better specify it though, the default might change
-    'hashref'       => {},
-    'extra_sql'     => '',
-    'records'       => \@records, #instead of hashref
-    'pre_options'   => [ 'value' => 'option' ], #before normal options
-    'element_name ' => '', #HTML element name, defaults to the name of
-                           # the primary key column
-    'field'         => '', #synonym for element_name
-    'element_etc'   => '', #additional attributes (i.e. "DISABLED") for the
-                           #<SELECT> element
-    'multiple'      => 0, # bool
-    'disable_empty' => 0, # bool (implied by multiple)
-    'debug'         => 0, #set true to enable
-
+    'empty_label'    => '', #better specify it though, the default might change
+    'hashref'        => {},
+    'extra_sql'      => '',
+    'records'        => \@records, #instead of hashref
+    'pre_options'    => [ 'value' => 'option' ], #before normal options
+    'element_name'   => '', #HTML element name, defaults to the name of
+                            # the primary key column
+    'field'          => '', #synonym for element_name
+    'element_etc'    => '', #additional attributes (i.e. "DISABLED") for the
+                            #<SELECT> element
+    'onchange'       => '', #javascript code
+    'multiple'       => 0, # bool
+    'disable_empty'  => 0, # bool (implied by multiple)
+    'debug'          => 0, #set true to enable
+    'label_callback' => sub { my $record = shift; return "label"; },
   )
 
 </%doc>
 
-<SELECT <% $opt{'multiple'} ? 'MULTIPLE' : '' %> NAME="<% $opt{'element_name'} || $opt{'field'} || $key %>" <% $opt{'element_etc'} %>>
+<SELECT <% $opt{'multiple'} ? 'MULTIPLE' : '' %>
+        NAME = "<% $opt{'element_name'} || $opt{'field'} || $key %>"
+        <% $onchange %>
+        <% $opt{'element_etc'} %>
+>
 
 % while ( @pre_options ) { 
     <OPTION VALUE="<% shift(@pre_options) %>"><% shift(@pre_options) %>
@@ -47,7 +53,10 @@ Example:
             <% ref($value) && $value->{$recvalue} || $value == $recvalue
                ? ' SELECTED' : ''
             %>
-    ><% $record->$name_col() %>
+    ><% $opt{'label_callback'}
+          ? &{ $opt{'label_callback'} }( $record )
+          : $record->$name_col()
+     %>
 % } 
 
 </SELECT>
@@ -59,6 +68,10 @@ my( %opt ) = @_;
 warn "elements/select-table.html: \n". Dumper(%opt)
   if exists $opt{debug} && $opt{debug};
 
+my $onchange = $opt{'onchange'}
+                 ? 'onChange="'. $opt{'onchange'}. '(this)"'
+                 : '';
+
 my $key = dbdef->table($opt{'table'})->primary_key; #? $opt{'primary_key'} ||
 
 my $name_col = $opt{'name_col'};