[freeside-commits] branch FREESIDE_2_3_BRANCH updated. 5a1b5212056dc5f4f497aae8339bed8dde50cb8c

Ivan ivan at 420.am
Sun Mar 10 21:53:57 PDT 2013


The branch, FREESIDE_2_3_BRANCH has been updated
       via  5a1b5212056dc5f4f497aae8339bed8dde50cb8c (commit)
      from  38b842853f3a704a72616ffe080a74a68c87b3aa (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 5a1b5212056dc5f4f497aae8339bed8dde50cb8c
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Sun Mar 10 21:53:56 2013 -0700

    fix radius data usage report w/sqlradius_withdomain export, RT#19964

diff --git a/httemplate/search/report_sqlradius_usage.html b/httemplate/search/report_sqlradius_usage.html
index 01215e8..7e54465 100644
--- a/httemplate/search/report_sqlradius_usage.html
+++ b/httemplate/search/report_sqlradius_usage.html
@@ -8,13 +8,18 @@
   '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',
 &>

-----------------------------------------------------------------------

Summary of changes:
 httemplate/search/report_sqlradius_usage.html |    9 +++++++--
 1 files changed, 7 insertions(+), 2 deletions(-)




More information about the freeside-commits mailing list