summaryrefslogtreecommitdiff
path: root/httemplate/elements/select-table.html
diff options
context:
space:
mode:
authorMark Wells <mark@freeside.biz>2014-12-06 09:50:32 -0800
committerMark Wells <mark@freeside.biz>2014-12-06 09:50:32 -0800
commit715120170b58861d2957266a2802df2e412979c8 (patch)
treef480774d2c8e005c1bb5bbbe6ed0a1467a373112 /httemplate/elements/select-table.html
parent56efb22e0fe7adb1aebad517c6975edbf6e1c921 (diff)
slightly more cautious fix for sorting the reason selector, #31702
Diffstat (limited to 'httemplate/elements/select-table.html')
-rw-r--r--httemplate/elements/select-table.html16
1 files changed, 8 insertions, 8 deletions
diff --git a/httemplate/elements/select-table.html b/httemplate/elements/select-table.html
index 5f36a36ef..3f6c8805d 100644
--- a/httemplate/elements/select-table.html
+++ b/httemplate/elements/select-table.html
@@ -9,6 +9,7 @@ Example:
##
'table' => 'table_name',
'name_col' => 'name_column', #or method if you pass an order_by
+ # order_by is currently broken, though
#strongly recommended (you want your forms to be "sticky" on errors, right?)
'curr_value' => 'current_value',
@@ -89,14 +90,13 @@ Example:
<OPTION VALUE=""><% $opt{'empty_label'} || 'all' %>
% }
-%# foreach my $record ( sort { $a->$name_col() cmp $b->$name_col()
-%# || $a->$key() <=> $b->$key()
-%# }
-%# @records
-%# )
-%# {
-% # show records in query order, or preserve the order they were given
-% foreach my $record (@records) {
+% # XXX fix this eventually, when we have time to test it
+% foreach my $record ( sort { $a->$name_col() cmp $b->$name_col()
+% || $a->$key() <=> $b->$key()
+% }
+% @records
+% )
+% {
% my $recvalue = $record->$key();
% my $selected;
% if ( $opt{'all_selected'} ) {