multi-select for agent on RADIUS usage report, RT#73050

[freeside.git] / httemplate / search / report_sqlradius_usage.html

diff --git a/httemplate/search/report_sqlradius_usage.html b/httemplate/search/report_sqlradius_usage.html
index 01215e8..38d52e5 100644 (file)
--- a/httemplate/search/report_sqlradius_usage.html
+++ b/httemplate/search/report_sqlradius_usage.html
@@ -1,20 +1,28 @@
+%# some overlap with report_sqlradius_usage_custnum.html
 <& /elements/header.html, mt($title) &>
 
 <FORM ACTION="sqlradius_usage.html" METHOD="GET">
 
-<TABLE BGCOLOR="#cccccc" CELLSPACING=0
+<TABLE BGCOLOR="#cccccc" CELLSPACING=0>
 
 <& /elements/tr-select-agent.html,
-  'empty_label'   => 'all',
+  'multiple'      => 1,
+  'disable_empty' => 1,
+  #'empty_label'   => 'all',
 &>
 
-% my @exporttypes = map { "'$_'" } qw(sqlradius broadband_sqlradius);
+%#more future-proof to actually ask all exports if they ->can('usage_sessions')
+% my @exporttypes = qw( sqlradius sqlradius_withdomain broadband_sqlradius
+%                       phone_sqlradius radiator
+%                     );
 <& /elements/tr-select-table.html,
   'label'         => 'Export',
   'table'         => 'part_export',
   'name_col'      => 'label',
   'hashref'       => {},
-  'extra_sql'     => ' WHERE exporttype IN('.join(',', @exporttypes).')',
+  'extra_sql'     => ' WHERE exporttype IN ( '.
+                                            join(',', map "'$_'", @exporttypes).
+                                          ')',
   'disable_empty' => 1,
   'order_by'      => 'ORDER BY exportnum',
 &>